Page MenuHomePhabricator

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

Authored by epriestley on Fri, Feb 8, 12:20 AM.

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.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Fri, Feb 8, 12:20 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Fri, Feb 8, 12:22 AM
Harbormaster failed remote builds in B21905: Diff 48068!
epriestley updated this revision to Diff 48069.Fri, Feb 8, 12:44 AM
  • 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.
epriestley requested review of this revision.Fri, Feb 8, 12:46 AM
aeiser added a subscriber: aeiser.Fri, Feb 8, 1:17 AM

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.)

amckinley accepted this revision.Fri, Feb 8, 10:21 PM
This revision is now accepted and ready to land.Fri, Feb 8, 10:21 PM
epriestley updated this revision to Diff 48072.Sat, Feb 9, 1:57 PM
  • Remove stray $lock = .
This revision was automatically updated to reflect the committed changes.