• 
      

    Show an error message when trying to compare different file attachment types.

    Review Request #6741 — Created Jan. 6, 2015 and submitted — Latest diff uploaded

    Information

    Review Board
    master
    f587cd4...

    Reviewers

    The file attachment revisioning stuff doesn't actually put any restrictions on
    the file type that's used to replace an attachment. This was fine when just
    displaying individual revisions, but the comparison views would break in fun
    and interesting ways.

    This change fixes up our existing attachment UIs to show an error message in
    the case where the comparison won't work. This currently has to be done
    individually for each different Review UI, which isn't ideal. I'll make sure
    that we get an item added to our backlog to try to clean that up in the future.

    • Set up a file attachment with many different revisions, each of a different
      type. Verified that I saw the correct thing in each of the different Review
      UIs (which get selected based on the "orig" version being shown).
    • Ran unit tests.
    • Ran js-tests.