Reset rich text of review request fields when importing from commits.

Review Request #7452 — Created June 24, 2015 and submitted

Information

Review Board
release-2.0.x
403587e...

Reviewers

In our original version of our Markdown support, we had a global
rich_text flag for an entire review request. Because of that, we needed
to handle escaping text coming from commits or change descriptions (when
updating from those), in order to prevent the imported text from
rendering as Markdown if the review request was set to rich text mode.

We don't have this limitation anymore, and have had reports of
frustration where a commit message expected to be plain text ended up
being escaped.

We now reset the flags instead for any field we import, triggering the
expected behavior. The text will still appear escaped for edit if the
user has that option enabled, but won't appear escaped otherwise.

Unit tests were updated to check for the different code paths for this.
They pass.

Description From Last Updated

'markdown_escape' imported but unused

reviewbotreviewbot
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/reviews/models/base_review_request_details.py
        reviewboard/reviews/tests.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/reviews/models/base_review_request_details.py
        reviewboard/reviews/tests.py
    
    
  2. Show all issues
     'markdown_escape' imported but unused
    
  3. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (d2de929)
Loading...