Fix a regression with the Quick Search functionality.

Review Request #8942 — Created May 16, 2017 and submitted

Christian Hammond
Review Board
Some work was done in a recent release to fix issues in Quick Search and
to document the parameters going to this API. The addition of the
documentation (using a decorator that specifies the available list of
parameters) was using strict validation, not allowing for unknown
parameters. This broke Quick Search, which passes in a timestamp
argument (not otherwise used by this API) in all requests.

We now ignore any arguments that aren't used by the API, fixing this

Unit tests pass.

Tested that Quick Search was working properly here. Sent the patch to
a customer and confirmed it fixed the problem for them.

  • 0
  • 0
  • 1
  • 0
  • 1
Description From Last Updated
David Trowbridge
  2. reviewboard/webapi/resources/ (Diff revision 1)

    Trailing comma?

Christian Hammond
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.5.x (30586ed)