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.