Page MenuHomePhabricator

Force 'changes' to a string in hunk migration
ClosedPublic

Authored by epriestley on Jan 10 2017, 7:55 PM.
Tags
None
Referenced Files
F18821238: D17168.id41293.diff
Wed, Oct 22, 8:53 PM
F18794729: D17168.diff
Fri, Oct 17, 12:34 AM
F18772081: D17168.id.diff
Thu, Oct 9, 12:10 AM
F18669317: D17168.id.diff
Wed, Sep 24, 8:01 PM
F18626023: D17168.diff
Sep 16 2025, 12:11 AM
F18614050: D17168.diff
Sep 14 2025, 12:00 PM
F18559655: D17168.diff
Sep 8 2025, 10:16 PM
F18508401: D17168.id.diff
Sep 5 2025, 2:47 AM
Subscribers
None

Details

Summary

Fixes T12090. In obscure situations lost to the mists of time, the changes column could be null. Force a string cast so the migration finishes, even though these changesets are likely meaningless.

Test Plan

I did a force-reapply as a sanity check:

$ ./bin/storage upgrade -f --apply phabricator:20161213.diff.01.hunks.php 

That went cleanly; it would only have caught dramatic errors, but I didn't completely butcher things.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable