Shouldn't Cake itself take care of useless clauses?

Il giorno lunedì 19 gennaio 2015 16:57:43 UTC+1, José Lorenzo ha scritto:
>
> That's almost right. I would change it a bit by doing this:
>
> public function beforeFind ($event, $query, $options, $primary) {
>         $order = $query->clause('order');
>         if ($order === null || !count($order)) {
>              $query->order("nome");
>         }
> return parent::beforeFind($event, $query, $options, $primary);
> }
>
> That will prevent adding more order clauses than you'd need.
>
> On Monday, January 19, 2015 at 4:04:09 PM UTC+1, Ernesto wrote:
>>
>> Hi all
>>
>> i noticed that the variable "order", used in cake 1.x/2.x models to alter 
>> the default table sort, has gone.
>> i haven't found any hint in the 3.0 cookbook so i'm asking: what's the 
>> best practice to do this in 3.0?
>>
>> i did this:
>>
>> public function beforeFind ($event, $query, $options, $primary) {
>> $query->order("nome");
>> return parent::beforeFind($event, $query, $options, $primary);
>> }
>>
>> is that right?
>>
>> Sorry for my engRish
>>
>

-- 
Like Us on FaceBook https://www.facebook.com/CakePHP
Find us on Twitter http://twitter.com/CakePHP

--- 
You received this message because you are subscribed to the Google Groups 
"CakePHP" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to cake-php+unsubscr...@googlegroups.com.
To post to this group, send email to cake-php@googlegroups.com.
Visit this group at http://groups.google.com/group/cake-php.
For more options, visit https://groups.google.com/d/optout.

Reply via email to