Fix Search API on 2.5

Review Request #9115 — Created Aug. 1, 2017 and submitted

Information

Review Board
release-2.5.x

Reviewers

Review Board 2.5's webapi_check_local_site passes the local_site
parameter in addition to the local_site_name. This was causing the
_search_* methods to receive two copies of local_site, resulting in
an error. This issue has been resolved by just passing the Local Site
directly in via **kwargs.

Ran unit tests. They all pass with this patch applied and did not
previously.

Description From Last Updated

Can you add unit tests for this?

chipx86chipx86

Can you also make the summary a bit more self-explanatory? It should make a reference to local sites being broken.

chipx86chipx86

I want to see real-world testing in the Testing Done. Can you confirm the fix on global sites, Local Sites, …

chipx86chipx86
brennie
chipx86
  1. 
      
  2. Show all issues

    Can you add unit tests for this?

    1. This patch fixes failing unit tests.

  3. 
      
chipx86
  1. 
      
  2. Show all issues

    Can you also make the summary a bit more self-explanatory? It should make a reference to local sites being broken.

  3. 
      
brennie
brennie
brennie
chipx86
  1. 
      
  2. Show all issues

    I want to see real-world testing in the Testing Done. Can you confirm the fix on global sites, Local Sites, and on RBCommons?

    1. Discussed over Slack. Updating the testing in the commit to reflect the testing.

  3. 
      
chipx86
  1. Ship It!
  2. 
      
brennie
Review request changed
Status:
Completed
Change Summary:
Pushed to release-2.5.x (21da4b4)