Fix the reply draft banner sticking around after a second publish.

Review Request #5964 — Created June 9, 2014 and submitted — Latest diff uploaded

Information

Review Board
release-2.0.x
33eba27...

Reviewers

Publishing a reply to a comment and then following it up with a second
reply would result in the reply banner sticking around, acting as if the
publish hadn't gone through.

This happened because signal handlers for the "publishing" and
"destroyed" events were registered twice on the first ReviewReply, which
would cause the state to mess up. We only really needed to register
these once.

Posted a reply, published. Posted another, published. Both times, the banner
disappeared.

Also tested with discarding.

Unit tests pass.

    Loading...