• 
      

    Users with no editing ability on a review request should not see issue buttons on inline comments

    Review Request #2419 — Created June 23, 2011 and submitted

    Information

    Review Board
    master

    Reviewers

    I noticed that after I'd created a review with some issues, when I went and viewed my comments in the diff viewer, the buttons for dropping / fixing the issues were visible to me, even though I was not the review requester.
    
    This patch fixes that bug, as well as putting the issue status message in the inline comments.  Not sure how that one slipped by.
    Manual testing.
    Description From Last Updated

    Indentation is wrong.

    chipx86 chipx86

    We don't deal with scoping issues, so no need for self anymore.

    chipx86 chipx86

    You can return this.

    chipx86 chipx86

    First line of a comment should be blank. So: /* * Wraps an ... */

    chipx86 chipx86

    Doesn't look like self is needed, since we never deal with scoping issues from callbacks. You can just use this …

    chipx86 chipx86

    Can just use this.

    chipx86 chipx86
    mike_conley
    chipx86
    1. 
        
    2. reviewboard/htdocs/media/rb/js/reviews.js (Diff revision 1)
       
       
       
       
       
       
      should we even be rendering/creating the buttons in the case where the current user shouldn't see them? Rather than hiding after?
    3. reviewboard/htdocs/media/rb/js/reviews.js (Diff revision 1)
       
       
       
      Show all issues
      Indentation is wrong.
    4. 
        
    chipx86
    1. 
        
    2. Show all issues
      We don't deal with scoping issues, so no need for self anymore.
    3. Show all issues
      You can return this.
    4. Show all issues
      First line of a comment should be blank. So:
      
      /*
       * Wraps an ...
       */
    5. reviewboard/htdocs/media/rb/js/reviews.js (Diff revision 2)
       
       
       
      Show all issues
      Doesn't look like self is needed, since we never deal with scoping issues from callbacks. You can just use this below.
    6. Show all issues
      Can just use this.
    7. reviewboard/htdocs/media/rb/js/reviews.js (Diff revision 2)
       
       
       
      Is this actually replacing issue with some new element? If not, we can just do issue.issueButtons().
    8. reviewboard/htdocs/media/rb/js/reviews.js (Diff revision 2)
       
       
       
       
      Same question here. Though, in this case, I don't see where issue is even used anymore. So I think we can remove the assignment.
    9. 
        
    mike_conley
    Review request changed
    Change Summary:
    Thanks for the review - updated with style changes.
    
    While I was patching this, I also noticed that the issue state for an inline comment wasn't being persisted on closing and re-opening the comment window, so I've fixed that by registering a simple callback with the issue manager.
    david
    1. Ship It!
    2.