Page MenuHomePhabricator

Migrate old commit saved queries to new audit status constants

Authored by epriestley on Sep 10 2018, 7:50 PM.



Depends on D19651. Ref T13197. The application now accepts either numeric or string values. However, for consistency and to reduce surprise in the future, migrate existing saved queries to use string values.

Test Plan

Saved some queries on master with numeric constants, ran the migration, saw string constants in the database and equivalent behavior in the UI.

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

Owners added a subscriber: Restricted Owners Package.Sep 10 2018, 7:50 PM
amckinley added inline comments.

I sanity-checked this against the code and it's correct, but isn't there an easy way to pull this array out of PhabricatorAuditCommitStatusConstants?


So "statuses": 1 as a constraint doesn't work? Or just malformed because and audit.query are both expecting statuses to be list<string>, and this is just very defensive programming?

This revision is now accepted and ready to land.Sep 10 2018, 9:26 PM
epriestley added inline comments.

I'm hard-coding the list in this migration (and two later migrations) so that I can remove the constants from PhabricatorAuditCommitStatusConstants without retaining any confusion/ambiguity going forward.


Yeah, this is mostly just defensive -- "statuses": 1 should be impossible. It also probably doesn't work.

This revision was automatically updated to reflect the committed changes.