Closed (fixed)
Project:
Recipe
Version:
8.x-2.x-dev
Component:
Code
Priority:
Critical
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
14 Sep 2017 at 15:21 UTC
Updated:
31 Mar 2020 at 04:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
dcam commentedAdded ingredient and ingredient settings migrations.
Comment #3
dcam commentedRefactored the 1.x branch test code to use base classes since the 6.x and 7.x test classes were exactly the same apart from the fixture file that was imported.
Comment #5
dcam commentedTrying to fix the test errors that started popping up with the retests that I just did.
Comment #6
dcam commentedHey, how about a patch that isn't empty?
Comment #8
dcam commentedThe last patch had a bad fixture. It was complaining about not having the system module installed, so I copied the table row from the D7 fixture. The problem was that it didn't contain all of the same fields as the D6 table.
Comment #9
dcam commentedComment #10
dcam commentedI refactored the Recipe tests in preparation for making the 6.x migration tests run off the same code as the 7.x tests.
Comment #11
dcam commentedI decided to commit the ingredient migration changes so that I don't have as much uncommitted code around. So here's one last test of those changes just for the sake of due diligence.
Comment #13
dcam commentedRenaming migrations in Ingredient necessitates a small change in Recipe.
Comment #15
dcam commentedComment #18
dcam commentedI discovered that the changes I committed might have had some problems. So I reverted them.
Comment #19
dcam commentedComment #20
dcam commentedComment #21
dcam commentedAdded the recipe migration.
Comment #23
dcam commentedComment #24
Lebron84 commented