Hopefully, a lot of the work that has been done so far for the 7.x-1.3 migration can be re-used. I'm not very familiar with the 6.x data table structures, but I expect that they are pretty similar to what went into 7.x initially. If we're really lucky, the 7.x migration plugins can be used exactly as they are for 6.x, in which case I'll want to rename them to something more generic to the 1.x branch. Right now everything is named like "recipe713" to denote that it was written for the 7.x-1.3 branch.

Comments

dcam created an issue. See original summary.

dcam’s picture

Status: Active » Needs review
StatusFileSize
new19.5 KB

Added ingredient and ingredient settings migrations.

dcam’s picture

StatusFileSize
new22.39 KB

Refactored 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.

Status: Needs review » Needs work

The last submitted patch, 3: 2908876-3.patch, failed testing. View results

dcam’s picture

Status: Needs work » Needs review
StatusFileSize
new0 bytes

Trying to fix the test errors that started popping up with the retests that I just did.

dcam’s picture

StatusFileSize
new23.76 KB

Hey, how about a patch that isn't empty?

Status: Needs review » Needs work

The last submitted patch, 6: 2908876-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

dcam’s picture

StatusFileSize
new23.78 KB

The 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.

dcam’s picture

Status: Needs work » Needs review
dcam’s picture

StatusFileSize
new32.39 KB

I refactored the Recipe tests in preparation for making the 6.x migration tests run off the same code as the 7.x tests.

dcam’s picture

StatusFileSize
new22.91 KB

I 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.

Status: Needs review » Needs work

The last submitted patch, 11: 2908876-11.patch, failed testing. View results

dcam’s picture

Status: Needs work » Needs review
StatusFileSize
new29.18 KB

Renaming migrations in Ingredient necessitates a small change in Recipe.

Status: Needs review » Needs work

The last submitted patch, 13: 2908876-13.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

dcam’s picture

Status: Needs work » Needs review
StatusFileSize
new29.86 KB

  • dcam committed 761203e on 8.x-2.x
    Issue #2908876 by dcam: Migrate ingredient data from 6.x-1.5 to 8.x-2.x
    

  • dcam committed 1e97aab on 8.x-2.x
    Revert "Issue #2908876 by dcam: Migrate ingredient data from 6.x-1.5 to...
dcam’s picture

I discovered that the changes I committed might have had some problems. So I reverted them.

dcam’s picture

StatusFileSize
new21.12 KB
dcam’s picture

StatusFileSize
new23.52 KB
dcam’s picture

StatusFileSize
new88.41 KB

Added the recipe migration.

  • dcam committed a10aae3 on 8.x-2.x
    Issue #2908876 by dcam: Migrate data from 6.x-1.5 to 8.x-2.x
    
dcam’s picture

Status: Needs review » Fixed
Lebron84’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.