• 
      

    Fix some bad logic in the change_form_submit_buttons template tag.

    Review Request #11349 — Created Dec. 23, 2020 and submitted — Latest diff uploaded

    Information

    Review Board
    release-4.0.x

    Reviewers

    The new change_form_submit_buttons template tag had some bad logic in
    it that could lead to a failure when adding new objects. It made two bad
    assumptions:

    1. change was an object (it's a bool)
    2. It could always compute a delete URL (it can't if change == False).

    This change fixes up the logic to treat change correctly and to only
    generate a delete URL if we know we're showing the link.

    Re-tested all types of pages, including change forms creating a new object.

    Unit tests pass.

    Commits

    Files