Homec4science

Maniphest - fix corner case where no-op subscriber change had wrong error

Authored by Bob Trahan <btrahan@phacility.com> on Jan 9 2015, 23:14.

Description

Maniphest - fix corner case where no-op subscriber change had wrong error

Summary: Fixes T6732. Fix is to stop trying to catch the error in the controller and let the editor do its job.

Test Plan: tried to add an existing subscriber and got an error message about how that wouldn't do anything

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T6732

Differential Revision: https://secure.phabricator.com/D11303

Details

Committed
Bob Trahan <btrahan@phacility.com>Jan 9 2015, 23:14
Pushed
aubortJan 31 2017, 17:16
Parents
rPH59770443b62f: Projects - tokenize [ProjectX] so "projectX" is a match
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPHef4f1a7f7473: Maniphest - fix corner case where no-op subscriber change had wrong error (authored by Bob Trahan <btrahan@phacility.com>).Jan 9 2015, 23:14