• 
      

    Prevent double publishing from banner

    Review Request #7698 — Created Oct. 14, 2015 and submitted — Latest diff uploaded

    Information

    Review Board
    release-2.6.x

    Reviewers

    Now that the banner is an RB.SplitButtonView, we no longer want to
    register an event handler through the DraftBannerView's events
    object. We just use the event handler given to the RB.SplitButtonView
    constructor. Previously, the handler for publishing was being called
    twice, leading to a confusing permission error. Removing the event
    handler fixed the issue.

    Ran unit tests.
    Ran JS tests.
    Could no longer trigger the permission error after applying this patch.