Page MenuHomePhabricator

Let PhabricatorSearchCheckboxesField survive saved query data with mismatched types
ClosedPublic

Authored by epriestley on Jun 18 2017, 8:16 AM.
Tags
None
Referenced Files
F15465005: D18132.id43628.diff
Wed, Apr 2, 1:34 PM
F15455478: D18132.id43675.diff
Sun, Mar 30, 12:40 AM
F15449468: D18132.id43642.diff
Fri, Mar 28, 10:30 AM
F15446376: D18132.id.diff
Thu, Mar 27, 6:46 PM
F15442062: D18132.diff
Wed, Mar 26, 9:10 PM
F15427898: D18132.id.diff
Sun, Mar 23, 4:45 PM
F15415487: D18132.id43642.diff
Thu, Mar 20, 6:12 AM
F15385482: D18132.id43642.diff
Fri, Mar 14, 10:42 PM
Subscribers
Tokens
"Love" token, awarded by 20after4.

Details

Summary

Fixes T12851.

This should fix the error I'm seeing, which is:

  • Argument 1 passed to array_fuse() must be of the type array, boolean given

There may be a better way to patch this up than overriding the getValue() method,
however.

Test Plan
  • Changed the default "Tags" filter to specify true instead of array('self'), then viewed that filter in the UI.
  • Before patch: fatal.
  • After patch: page loads. Note that true is not interpreted as array('self'), but the page isn't broken, which is a big improvement.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley edited reviewers, added: 20after4; removed: epriestley.
epriestley edited edge metadata.

After unraveling things, I have arrived at a similar but not-quite-identical patch.

  • Pretty much the same patch.
epriestley retitled this revision from Fix T12851 to Let PhabricatorSearchCheckboxesField survive saved query data with mismatched types.Jun 19 2017, 10:06 PM
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, amckinley.
epriestley edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Jun 23 2017, 7:13 PM
This revision was automatically updated to reflect the committed changes.