Page MenuHomePhabricator

Add hard stops on empty batch edit sets
ClosedPublic

Authored by epriestley on Jun 22 2015, 1:06 PM.
Tags
None
Referenced Files
F15529788: D13388.id32420.diff
Tue, Apr 22, 10:29 PM
F15521956: D13388.id.diff
Sun, Apr 20, 4:56 PM
F15517512: D13388.diff
Sat, Apr 19, 3:33 AM
F15486097: D13388.diff
Thu, Apr 10, 4:59 AM
F15459131: D13388.id.diff
Mon, Mar 31, 10:03 AM
F15389053: D13388.id.diff
Mar 15 2025, 4:50 AM
F15378677: D13388.id32408.diff
Mar 13 2025, 4:09 PM
F15353058: D13388.diff
Mar 10 2025, 7:57 PM

Details

Summary

Ref T8637. If a user tries to batch edit a list of tasks which can't be edited, we fall through to withIDs(array()), which can affect everything.

Explicitly stop batch editing if we don't have valid IDs or valid tasks.

The UI sort-of warns you that something is wrong, but this is ultimately a pretty severe UX issue. I'll fix the underlying Query in the next diff.

Test Plan

Tried to batch edit a list of tasks I didn't have permission to edit.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Add hard stops on empty batch edit sets.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 22 2015, 5:28 PM
This revision was automatically updated to reflect the committed changes.