Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1857 closed defect (fixed)

[patch] get_next_by_FIELD and get_previous_by_FIELD don't take params

Reported by: tom@… Owned by: Adrian Holovaty
Component: Core (Other) Version:
Severity: normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


The docs for get_next/previous_by_FIELD state

Both methods accept optional keyword arguments, which should be in the format described in Field lookups above.

It's not clear what should will do, and anyway, they don't take keyword arguments. This patch makes the method do what I'd expect, which is to return the next/previous page that matches the passed filter params.

Attachments (1)

get_next_previous.diff (1.6 KB) - added by tom@… 12 years ago.

Download all attachments as: .zip

Change History (2)

Changed 12 years ago by tom@…

Attachment: get_next_previous.diff added

comment:1 Changed 12 years ago by Malcolm Tredinnick

Resolution: fixed
Status: newclosed

(In [3157]) Fixed #1857 -- Fixed get_previous_by_FIELD and get_next_by_FIELD to work
properly with keyword arguments. Patch from tom@….

Note: See TracTickets for help on using tickets.
Back to Top