• 
      

    Fix a regression disabling syntax highlighting in the API.

    Review Request #12887 — Created March 14, 2023 and submitted — Latest diff uploaded

    Information

    Review Board
    release-5.0.x

    Reviewers

    We just recently moved to a new internal configuration model for
    controlling the rendering pipeline, but one call location (the FileDiff
    API) was still using the old method. That'd be fine, except the old
    method wasn't properly converting a request to disable syntax
    highlighting.

    This led to requests for diff data to return HTML versions of lines
    instead of plain text.

    The settings migration code now allows for disabling settings, and the
    API has been updated for the new settings code.

    Unit tests check to ensure that we're properly getting plain-text back
    from it, unless requested otherwise.

    Verified that the API returned plain text data for any diffs I queried.

    A customer verified this fixed a regression they noticed.

    Unit tests passed.

    Commits

    Files