Page MenuHomePhabricator

Maniphest - add support for !assign command
ClosedPublic

Authored by btrahan on Oct 14 2013, 7:18 PM.
Tags
None
Referenced Files
F13366621: D7307.id16457.diff
Wed, Jun 26, 9:01 PM
F13324285: D7307.diff
Fri, Jun 14, 11:30 PM
F13311455: D7307.diff
Tue, Jun 11, 4:59 AM
F13297473: D7307.diff
Fri, Jun 7, 5:36 AM
F13280672: D7307.diff
Sun, Jun 2, 8:59 AM
F13251188: D7307.id16456.diff
May 24 2024, 8:57 PM
F13241404: D7307.id16457.diff
May 22 2024, 8:47 PM
F13241403: D7307.id16456.diff
May 22 2024, 8:47 PM

Details

Reviewers
epriestley
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rPd0127f95e541: Maniphest - add support for !assign command
Summary

also try to centralize some of the command parsing logic. note that differential is still an exception here. it uses a whitelist-style regex. i think long-term we should have this for every app but changing it seemed too big for this diff.

Fixes T3937.

Test Plan

echo '!assign btrahan' | ./bin/mail receive-test --as xerxes --to T22 ; echo '!claim' | ./bin/mail receive-test --as xerxes --to T22

unit tests passed, though my new one is silly

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Nice, this is definitely a lot cleaner. Agreed on the long term vision and also on there being more mess here than fits in one diff.

src/applications/maniphest/mail/ManiphestReplyHandler.php
95–102

Maybe just execute() and then check if you got something and head() it, to avoid the PolicyException business.

btrahan updated this revision to Unknown Object (????).Oct 14 2013, 7:26 PM

avoid try / catch by doing an execute()