Introduce webapi compatibility fixes for Review Board 1.0.x and a modern Djblets.
Review Request #1485 — Created March 27, 2010 and submitted — Latest diff uploaded
Introduce webapi compatibility fixes for Review Board 1.0.x and a modern Djblets. The Djblets webapi code is changing in many ways, and it breaks compatibility with Review Board 1.0.x. Some changes are internal API changes, while some are external (HTTP status codes). If someone using Review Board 1.0.x installs a more recent Djblets on their system (for bug fixes, development, another package that requires it, etc.), Review Board 1.0.x will break. This change introduces some workarounds for the API changes: 1) We hard-code an HTTP status code of 200 for all errors. 2) All our view functions now take *args, **kwargs as parameters. 3) Handle the api_format kwarg smartly in @webapi_check_login_required., 4) Specifically define api_format as a parameter in review_request_list and count_review_requests so that it's not passed down to the specified function for returning the list. 5) Our own ReviewBoardAPIEncoder now represents a User (as this is changing in Djblets) and we make sure ours runs first. This will not be merged in to master. The master branch will already work with the Djblets changes.
All unit tests pass now.