Page MenuHomePhabricator

Make dashboard arrange actions (move, add, remove) work again after read locking from "chaos reduction"
ClosedPublic

Authored by epriestley on Feb 21 2018, 1:20 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 7:25 PM
Unknown Object (File)
Thu, Apr 11, 8:16 AM
Unknown Object (File)
Sat, Mar 30, 10:03 AM
Unknown Object (File)
Mar 21 2024, 6:09 AM
Unknown Object (File)
Mar 21 2024, 6:09 AM
Unknown Object (File)
Mar 21 2024, 6:09 AM
Unknown Object (File)
Mar 21 2024, 6:09 AM
Unknown Object (File)
Mar 10 2024, 10:34 PM
Subscribers
None

Details

Summary

See PHI385. Ref T13054. Ref T13083. The dashboard "arrange" operations (add, remove, move) rely on doing $dashboard->setThing(...) and then applying transactions.

This no longer works after the read locking change from T13054. To make this function again, just add an explicit save() after layout adjustment. This should be more nuanced eventually, but all arrange operations are nonfunctional in a corrupting way at HEAD of master/stable, so stop the bleeding first.

Test Plan
  • Created new empty and template dashboards.
  • Moved panels.
  • Added new and existing panels.
  • Removed panels.

Diff Detail

Repository
rP Phabricator
Branch
rmdash1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19630
Build 26576: Run Core Tests
Build 26575: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Feb 21 2018, 1:22 AM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.