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
F19511737: D8152.id18441.diff
Mon, Jan 12, 7:09 AM
F19508104: D8152.id18452.diff
Sat, Jan 10, 1:07 AM
F19503659: D8152.diff
Fri, Jan 9, 12:59 AM
F19234375: D8152.diff
Sun, Dec 21, 6:01 PM
F18925819: D8152.id.diff
Nov 9 2025, 7:07 PM
F18907213: D8152.diff
Nov 8 2025, 10:24 PM
F18808093: D8152.diff
Oct 19 2025, 4:49 AM
F18753489: D8152.id.diff
Oct 4 2025, 7:02 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

Lint
Lint Skipped
Unit
Tests Skipped