Page MenuHomePhabricator

When an edit overrides an object lock, note it in the transaction record
ClosedPublic

Authored by epriestley on Feb 8 2019, 12:20 AM.
Tags
None
Referenced Files
F12804363: D20131.diff
Wed, Mar 27, 4:07 PM
Unknown Object (File)
Tue, Mar 26, 1:34 PM
Unknown Object (File)
Tue, Mar 26, 1:33 PM
Unknown Object (File)
Tue, Mar 26, 1:32 PM
Unknown Object (File)
Tue, Mar 26, 1:32 PM
Unknown Object (File)
Tue, Mar 26, 1:31 PM
Unknown Object (File)
Tue, Mar 26, 1:31 PM
Unknown Object (File)
Sun, Mar 10, 11:48 PM
Subscribers

Details

Summary

Ref T13244. See PHI1059. When you lock a task, users who can edit the task can currently override the lock by using "Edit Task" if they confirm that they want to do this.

Mark these edits with an emblem, similar to the "MFA" and "Silent" emblems, so it's clear that they may have bent the rules.

Also, make the "MFA" and "Silent" emblems more easily visible.

Test Plan

Edited a locked task, overrode the lock, got marked for it.

Screen Shot 2019-02-07 at 4.14.09 PM.png (157×317 px, 6 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 8 2019, 12:22 AM
Harbormaster failed remote builds in B21905: Diff 48068!
  • Tailor the interaction check to avoid some complications in Projects unit tests, where membership is sometimes not fully loaded and we're editing on behalf of some random actor.

Wasn't there supposed to be an icon getting upstreamed in this revision? I can see the change to the celerity map but not the associated icon file.

src/applications/transactions/view/PhabricatorApplicationTransactionView.php
420

I'm pretty sure this was not what you meant, since $lock isn't used again in this function and I've never seen the "use assignment operation's side effect as a function argument" idiom in the Phabricator code base.

There's no new icon, just a new way to render them (as "emblems"). The celerity map change is just because the CSS changed.

($icon = ... was debugging code, though.)

This revision is now accepted and ready to land.Feb 8 2019, 10:21 PM
This revision was automatically updated to reflect the committed changes.