Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
migration system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
6 Jun 2014 at 21:02 UTC
Updated:
7 Oct 2014 at 04:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Anonymous (not verified) commentedHere is a patch that changes the sort order of the migration query. It works for the "Manage form display" tab so for content editors it may solve this issue. The weights all reflected the same values as D6 in my (somewhat limited) testing.
The other referenced issue in the description will probably be needed for the "Manage display" tab as currently all of my fields are set to disabled there.
Comment #2
Anonymous (not verified) commentedComment #4
ultimikeI just retested Ryan's patch from comment 1 and it still applies cleanly. Looking at the test results, I'm not sure FieldInstancePerViewModeTest still exists, so I submitted the test to be re-run.
Thanks,
-mike
Comment #7
benjy commentedFixed the unit test and a few bugs in the sources relating to weight. Not sure if we want to do it here or in a follow-up but FieldInstancePerFormDisplay could do with a unit test.
Comment #8
chx commentedThanks for fixing this. Please file a followup for the unit test.
Comment #9
webchickGood catch. Thanks for testing the D8 migration path! :)
Committed and pushed to 8.x. Thanks!