Page MenuHomePhabricator

Don't mark subscribes/mentions as "Lock Overridden"
ClosedPublic

Authored by epriestley on May 10 2019, 6:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 8:31 AM
Unknown Object (File)
Sat, Mar 23, 12:45 AM
Unknown Object (File)
Sat, Mar 23, 12:45 AM
Unknown Object (File)
Sat, Mar 23, 12:45 AM
Unknown Object (File)
Sat, Mar 23, 12:45 AM
Unknown Object (File)
Mar 5 2024, 2:54 PM
Unknown Object (File)
Feb 15 2024, 1:24 AM
Unknown Object (File)
Feb 7 2024, 10:54 AM
Subscribers
None

Details

Reviewers
amckinley
Summary

See PHI1209. When a task is in "Hard Lock" mode, it's still possible to apply some changes to it. Notably:

  • You can subscribe/unsubscribe.
  • You can mention it on another object.
  • You can add a relationship from some other object to it (e.g., select it as a "Parent Task" for some other task).

Currently, these types of edits will show a "Lock Overridden" timeline emblem icon. However, they should not: you didn't override a lock to make these changes, they just bypass locks.

For now, special case these cases (self subscribe/unsubscribe + inverse edge edits) so they don't get the little icon, since I think this list is exhaustive today.

Some day we should modularize this, but we'd need code like this anyway (since TYPE_SUBSCRIBE is not modular yet), and this seems unlikely to cause problems even if it's a bit rough.

Test Plan
  • Hard-locked a task.
    • Subscribed/unsubscribed, mentioned, relationship'd it as a non-author. No timeline emblems.
  • Soft-locked a task.
    • Subscribed/unsubscribed, mentioned, relationship'd it, no timeline emblems.
    • Clicked "Edit", answered "yes" to the override prompt, edited it. Got a timeline emblem.
  • Added some comments and stuff to a normal non-locked task, no emblems.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley added inline comments.
src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
5218

"test if"

This revision is now accepted and ready to land.May 14 2019, 2:50 PM