HomePhabricator

Fix accidental forward dependency in old commit summary migration

Description

Fix accidental forward dependency in old commit summary migration

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.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Differential Revision: https://secure.phabricator.com/D8152

Details

Provenance
epriestleyAuthored on
epriestleyPushed on Feb 6 2014, 6:33 PM
Reviewer
btrahan
Differential Revision
D8152: Fix accidental forward dependency in old commit summary migration
Parents
rP4c7acbaface4: Assign PHIDs to calendar events
Branches
Unknown
Tags
Unknown

Event Timeline