Page MenuHomePhabricator

Fix a warning in BoardLayoutEngine when no objects are being updated

Authored by epriestley on Aug 7 2019, 4:20 PM.



Fixes T13368. Some workflows (like "Move tasks to...") execute board layout without objects to update. In these cases, we can hit a warning because objectPHIDs is not initialized to array().

Test Plan

Went through the "Move tasks to..." workflow on a workboard, no longer saw a warning when trying to iterate over an empty objectPHIDs list.

Diff Detail

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

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Aug 8 2019, 5:18 PM
This revision was automatically updated to reflect the committed changes.