Fix fetching original and patched files through the web API.

Review Request #5804 — Created May 12, 2014 and submitted

Information

Review Board
release-2.0.x
793c6b1...

Reviewers

The handlers for getting original/patched files in the API were
expecting a URL parameter that doesn't exist. It would always attempt to
fetch using the draft filediff resource, instead of intelligently
choosing between that and the published filediff resource.

By using the correct URL parameter name, and using that parameter
correctly, we can fetch the original/patched files correctly.

Manually tested by fetching the original/patched files.

Unit tests pass. We actually don't have unit tests for these resources, though.
Bigger task.

Description From Last Updated

This was here for testing. I've reverted it locally.

chipx86chipx86
chipx86
  1. 
      
  2. Show all issues

    This was here for testing. I've reverted it locally.

  3. 
      
david
  1. Ship It!

  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (7fd6b84)
Loading...