Fix multi-commit uploading with Mercurial in pre-commit review mode.

Review Request #12623 — Created Sept. 22, 2022 and submitted — Latest diff uploaded

Information

Review Board
release-4.0.x

Reviewers

Currently, uploading a multi-commit review request diffs fails with
"API Error 207: Repository File Not Found" when different files are
changed in different patches.

See the report here:
https://groups.google.com/g/reviewboard/c/FG5TNTNC_fU

It tries to request a raw file from a server repository using the
revision ID of the previous uploaded commit. Sure there's no such
revision on the server as those patches aren't pushed yet to the
upstream repository in a pre-commit review case.

For a proper file request in this case it should use "base_commit_id".

Reproduced the original problem in a new database, and verified that
this fixes the issue.

Commits

Files

    Loading...