#306 setSession offset

v2.0.2
closed-fixed
Other (50)
8
2005-11-08
2005-11-07
No

Going back to April 19th 2005 aveitch checked into the CVS revision
1.40.2.9 of MailManager.py stating.
"Batching of results on the ticket screens is now done by the
database with LIMIT and OFFSET rather than on the client"
With this change the key in the session which held the view page
offset was changed from 'start' to 'offset'.

In the MailManager.py in the method setSession there is a trap to
ensure that the page offset is cleared if the user changes views,
however this code still references the 'start' key in

the session and not 'offset'.

Here is the fix to MailManager.py

1205,1206c1205,1206
< if key != 'start':
< SESSION.set('start', 0)
---
> if key != 'offset':
> SESSION.set('offset', 0)

Discussion

  • Peter George

    Peter George - 2005-11-08
    • priority: 5 --> 8
     
  • Graeme Mathieson

    • status: open --> closed
     
  • Graeme Mathieson

    • status: closed --> closed-fixed
     
  • Graeme Mathieson

    Logged In: YES
    user_id=12789

    Hi,

    I've applied this fix to the 2.0.x branch and to the latest
    CVS. Thanks!

    Regards,

    Graeme

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks