Re: Paginator links in 1.2.2

2009-03-24 Thread Mauricio Morales
That´s great. Thanks! On Mar 23, 3:29 pm, Martin Westin wrote: > I created a ticket yesterday and jperras fixed the problem already (!) > > If you have a problem with this in version 8120 you can sync to the > 1.2 branch or get the latest paginator any way you > prefer:https://trac.cakephp.org

Re: Paginator links in 1.2.2

2009-03-23 Thread Martin Westin
I created a ticket yesterday and jperras fixed the problem already (!) If you have a problem with this in version 8120 you can sync to the 1.2 branch or get the latest paginator any way you prefer: https://trac.cakephp.org/changeset/8125 /Martin On Mar 23, 10:41 am, Mauricio Morales wrote: >

Re: Paginator links in 1.2.2

2009-03-23 Thread Mauricio Morales
Hey, I haven't tested it yet. But you can see here the last changes in Paginator Helper: https://trac.cakephp.org/changeset?new=branches%2F1.2.x.x%2Fcake%2Flibs%2Fview%2Fhelpers%2Fpaginator.php%408092&old=branches%2F1.2.x.x%2Fcake%2Flibs%2Fview%2Fhelpers%2Fpaginator.php%407847 https://trac.cak

Re: Paginator links in 1.2.2

2009-03-22 Thread brian
I haven't yet looked into this myself but Gwoo mentioned a security fix for Paginator (Helper or Component, I don't remember) so this may be related. 2009/3/22 Martin Westin : > > I suspect this was not intended since 1.2 is stable and should only > "fix bugs" and not change API requirements. > >

Re: Paginator links in 1.2.2

2009-03-22 Thread Martin Westin
I suspect this was not intended since 1.2 is stable and should only "fix bugs" and not change API requirements. Looks like the changes to the methods sort() and sortkey() in PaginatorHelper caused the change in behaviour. $this->sortKey($options['model']) will always return a value like Modename

Re: Paginator links in 1.2.2

2009-03-22 Thread Sergei
Ok thanks, will try this. On 22 мар, 20:32, Martin Westin wrote: > Looks like a new "requirement" has crept into the core. > > This will give me the results you mention: > $paginator->sort('label, 'field'); > > While this works as expected: > $paginator->sort('label, 'Model.field'); > > /Martin

Re: Paginator links in 1.2.2

2009-03-22 Thread Martin Westin
Looks like a new "requirement" has crept into the core. This will give me the results you mention: $paginator->sort('label, 'field'); While this works as expected: $paginator->sort('label, 'Model.field'); /Martin On Mar 22, 11:09 am, Sergei wrote: > I think paginator links have bug in 1.2.2

Paginator links in 1.2.2

2009-03-22 Thread Sergei
I think paginator links have bug in 1.2.2: When initial sort order is :desc, and then you sort it by :asc, links don't change back to :desc. It' always :asc. Anyone noticed this? --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Goo