[Solar-talk] Some refactoring of "related" methods in
Solar_Sql_Model
Raymond Kolbe
rkolbe at gmail.com
Mon May 26 16:33:05 CDT 2008
There was another piece of code within Solar_Model that is not being used
either...too bad I can't remember what it is right off hand :-( ... I think
it was a call to a model class parent or something and it never gets
used.....
But as for your question above I am good w/the changes.
On 5/26/08, Dmytro Konstantinov <umka.dk at gmail.com> wrote:
>
> Paul,
>
> This will make no difference to me. I'm not using
> Solar_Sql_Model->countPagesRelated() anywhere in my code, so you'll get no
> objections from me! :)
>
> Dmytro
>
>
> On 26 May 2008, at 15:25, Paul M Jones wrote:
>
> Hi guys --
>>
>> I'm preparing to refactor Solar_Sql_Model methods fetchRelatedArray() and
>> fetchRelatedObject() out to the Solar_Sql_Model_Related class, where they
>> seem to make more sense (and will better be able to use versioned data
>> caching from the related model.)
>>
>> I've noticed that although there's a countPagesRelated() method, it's
>> never call from the model class, and in fact is never called from anywhere
>> in the Solar code base. Is anyone actually using that method? If not, are
>> there any objections to me removing it entirely?
>>
>>
>> -- pmj
>> _______________________________________________
>> Solar-talk mailing list
>> Solar-talk at lists.solarphp.com
>> http://mailman-mail3.webfaction.com/listinfo/solar-talk
>>
>
> _______________________________________________
> Solar-talk mailing list
> Solar-talk at lists.solarphp.com
> http://mailman-mail3.webfaction.com/listinfo/solar-talk
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mailman-mail3.webfaction.com/pipermail/solar-talk/attachments/20080526/2fbdc3c3/attachment.html
More information about the Solar-talk
mailing list