Use DraftResource.publish from the review dialog.
Review Request #7424 — Created June 13, 2015 and submitted
The review dialog wasn't using the publish method to publish reviews, which
would still publish but would skip emitting the publishing and published
events. This made it hard to do things from extensions that relied on those
events.I've also fixed up an incorrect comment and removed an unused var definition.
Used this from an extension and saw that the publishing event was successfully
triggered.
| Description | From | Last Updated |
|---|---|---|
|
Since this is all pretty much the same except for the function call, maybe we can pre-compute which function we … |
chipx86
|
Review request changed
- Commit:
-
6c46b1c53e56b6d9792435f5fb56d619ba5d6fe0163ab17df6bcc4a4abbc3241043bc2c21a92c746
-
Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js reviewboard/static/rb/js/extensions/models/reviewDraftBannerHookModel.js reviewboard/static/rb/js/resources/models/draftReviewModel.js Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js reviewboard/static/rb/js/extensions/models/reviewDraftBannerHookModel.js reviewboard/static/rb/js/resources/models/draftReviewModel.js
Review request changed
- Commit:
-
163ab17df6bcc4a4abbc3241043bc2c21a92c746ee53a2a4beddda4db34689449748cee0806b2b4d
Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js reviewboard/static/rb/js/resources/models/draftReviewModel.js Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js reviewboard/static/rb/js/resources/models/draftReviewModel.js