ReviewReplyDraftBannerView now properly removed

Review Request #7564 — Created July 31, 2015 and submitted

Information

Review Board
release-2.5.x

Reviewers

Previously the ReviewReplyDraftBannerView was not correctly removed
when a review reply was published; the element would be removed from
the DOM, but the event listeners would still fire (and cause errors in
the console). Now, the view is correctly removed with
Backbone.View.remove so the elements are removed from the DOM and all
event listeners are disconnects.

Ran JS tests.

With this patch applied, the error no longer appears in the console.

Description From Last Updated

Did you mean to call this this._banner, or use this._bannerView below?

daviddavid

Typo: "SHown"

daviddavid

This should be this._bannerView, right?

daviddavid
reviewbot
  1. Tool: Pyflakes
    Ignored Files:
        reviewboard/static/rb/js/views/reviewBoxView.js
    
    
    
    Tool: PEP8 Style Checker
    Ignored Files:
        reviewboard/static/rb/js/views/reviewBoxView.js
    
    
  2. 
      
david
  1. 
      
  2. Show all issues

    Did you mean to call this this._banner, or use this._bannerView below?

  3. Show all issues

    Typo: "SHown"

  4. 
      
brennie
reviewbot
  1. Tool: PEP8 Style Checker
    Ignored Files:
        reviewboard/static/rb/js/views/reviewBoxView.js
    
    
    
    Tool: Pyflakes
    Ignored Files:
        reviewboard/static/rb/js/views/reviewBoxView.js
    
    
  2. 
      
david
  1. 
      
  2. Show all issues

    This should be this._bannerView, right?

  3. 
      
brennie
reviewbot
  1. Tool: Pyflakes
    Ignored Files:
        reviewboard/static/rb/js/views/reviewBoxView.js
    
    
    
    Tool: PEP8 Style Checker
    Ignored Files:
        reviewboard/static/rb/js/views/reviewBoxView.js
    
    
  2. 
      
david
  1. Ship It!
  2. 
      
brennie
Review request changed
Status:
Completed
Change Summary:
Pushed to release-2.5.x (cac7f06)