Work around errors in patch bundle URL resolution.

Review Request #9917 — Created May 8, 2018 and submitted

Information

Review Board
release-3.0.x
058b475...

Reviewers

Web crawlers have a nasty habit of trying to index diff fragments
(somehow), and don't always pass in all the right data in the URL. When
we encounter the unsurprising errors, we'd try to show a link to the
patch bundle, but reversing this URL would fail.

This change works around that by catching the NoReverseMatch and just
including an empty link instead.

Ran unit tests.

brennie
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (8abb9ab)
Loading...