Opened 10 years ago

Closed 9 years ago

Last modified 9 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
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: UI/UX:


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@… 10 years ago.

Download all attachments as: .zip

Change History (2)

Changed 10 years ago by tom@…

comment:1 Changed 9 years ago by mtredinnick

  • Resolution set to fixed
  • Status changed from new to closed

(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