Page MenuHomePhabricator

Fix accidental forward dependency in old commit summary migration
ClosedPublic

Authored by epriestley on Feb 5 2014, 10:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 17 2026, 12:11 AM
Unknown Object (File)
Apr 16 2026, 7:57 AM
Unknown Object (File)
Apr 2 2026, 9:34 AM
Unknown Object (File)
Feb 11 2026, 12:13 AM
Unknown Object (File)
Jan 12 2026, 7:09 AM
Unknown Object (File)
Jan 10 2026, 1:07 AM
Unknown Object (File)
Jan 9 2026, 12:59 AM
Unknown Object (File)
Dec 21 2025, 6:01 PM
Subscribers

Details

Summary

See IRC. This migration inadvertently depends on the columns in the commit table, because it calls save(), and thus broke for installs with data after we added the importStatus column. Since that was ~9 months after this patch, probably not many installs are affected.

Test Plan

Ran patch locally with --apply on data. Had user verify fix.

Diff Detail

Repository
rP Phabricator
Branch
savepatch
Lint
Lint Passed
Unit
No Test Coverage