• 
      

    Fix stripping of XML-like syntax in comments

    Review Request #5796 — Created May 12, 2014 and submitted — Latest diff uploaded

    Information

    Review Board
    release-1.7.x
    aa44134...

    Reviewers

    If a comment consisted of only XML-like syntax (such as "<test>"), it would be treated as if the comment was completely empty and the comment would be canceled. The issue here is that reviewCommentFormIsEmpty would strip out tags before testing whether the comment was empty or not. Removing the call to stripTags() fixes the bug.

    I manually tried to recreate the bug with different ways.
    By submitting '<sim realtime="true>' or 'something <sim realtime="true>' in the comments you can verify that the bug is no longer exists.