Last Comment Bug 916979 - ignores the "limit" parameter
: ignores the "limit" parameter
Product: Bugzilla
Classification: Server Software
Component: WebService (show other bugs)
: 4.5
: x86 Mac OS X
-- normal (vote)
: Bugzilla 5.0
Assigned To: David Lawrence [:dkl]
: default-qa
Depends on:
  Show dependency treegraph
Reported: 2013-09-16 13:39 PDT by Tiziana Sellitto [:tiziana]
Modified: 2013-09-18 08:09 PDT (History)
4 users (show)
glob: approval+
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

916979_1.patch (1.87 KB, patch)
2013-09-17 10:28 PDT, David Lawrence [:dkl]
glob: review+
Details | Diff | Splinter Review

Description User image Tiziana Sellitto [:tiziana] 2013-09-16 13:39:10 PDT
I'm using json-rpc interface to access Bugzilla API. It seems that from yesterday the "limit" and "creation_time" parameters don't work;[{"limit":3,"product":"Firefox","component":"Untriaged","status":"UNCONFIRMED","include_fields":["creation_time","id","status","summary","last_change_time" ]}]

and the "creation_time" doesn't return anything:[{"limit":3,"creation_time":"2011-07-09T17:56:00Z","product":"Firefox","component":"Untriaged","status":"UNCONFIRMED","include_fields":["creation_time","id","status","summary","last_change_time" ]}]

These searches worked fine up to two days ago. 
I've looked at this bug 785565 asking if it could be related.
Comment 1 User image Byron Jones ‹:glob› 2013-09-16 20:57:32 PDT
nothing was changed on production two days ago (the last update was Sep 11 00:15:35 PDT).

as there are two issues reported here, i've split the bug - the creation_time issue is in bug 917157.

the problem with limit is Bugzilla::Bug::map_fields() is converting $params->{limit} to $params->{LIMIT}, however Bugzilla::Search is expecting lowercase {limit} (this also applies to offset).
Comment 2 User image David Lawrence [:dkl] 2013-09-17 10:28:14 PDT
Created attachment 806099 [details] [diff] [review]

Simple remove LIMIT/OFFSET from FIELD_MAP in Bugzilla::Bug. Also cleaned up my variable naming in as it was backwards.

Comment 3 User image Byron Jones ‹:glob› 2013-09-17 23:38:42 PDT
Comment on attachment 806099 [details] [diff] [review]

Comment 4 User image David Lawrence [:dkl] 2013-09-18 08:09:30 PDT
Committing to: bzr+ssh://
modified Bugzilla/
modified Bugzilla/WebService/
Committed revision 8741.

Note You need to log in before you can comment on or make changes to this bug.