Fixes T3566 List of poll actions should include ability to close an open poll or reopen a closed poll.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Maniphest Tasks
- Restricted Maniphest Task
- Commits
- Restricted Diffusion Commit
rP9a827096a74a: Ability to close poll
Poll author should be able to close/reopen poll. Non-author should get policy screen when attempting to close/reopen poll.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
Handful of minor inlines.
| src/applications/slowvote/controller/PhabricatorSlowvoteCloseController.php | ||
|---|---|---|
| 18 | You can remove this check -- $this->id will always be set. This check is only important for controllers which can both create and edit objects, but this one only edits them. | |
| 61–63 | You can just call $poll->getIsClosed() here and remove the $this->isClosed property. | |
| 67 | The default text for cancel buttons is "Cancel", so just remove this variable. | |
| 79 | This should send the user back to $close_uri, not "/". You can check this behavior with:
(With the Javascript dialog, "cancel" always just closes the dialog.) | |
Comment Actions
| src/applications/slowvote/controller/PhabricatorSlowvoteCloseController.php | ||
|---|---|---|
| 29 | (Looks unused.) | |