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)
Wed, Nov 20, 9:20 PM
Unknown Object (File)
Fri, Nov 15, 11:06 AM
Unknown Object (File)
Sun, Nov 10, 8:21 PM
Unknown Object (File)
Thu, Nov 7, 1:54 PM
Unknown Object (File)
Oct 18 2024, 8:31 AM
Unknown Object (File)
Oct 14 2024, 9:38 AM
Unknown Object (File)
Oct 13 2024, 9:09 PM
Unknown Object (File)
Oct 2 2024, 4:19 PM
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
Branch
lock1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22779
Build 31245: Run Core Tests
Build 31244: arc lint + arc unit

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