Closed (fixed)
Project:
Migrate Extras
Version:
7.x-2.2
Component:
Miscellaneous
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Dec 2011 at 22:29 UTC
Updated:
20 Mar 2012 at 18:36 UTC
Jump to comment: Most recent file
Comments
Comment #1
StuartDH commentedSame issue with 7.2.2
The error appears following module installation and whenever I clear the cache
Comment #2
StuartDH commentedNo error messages when using 7.x-2.x-dev
Comment #3
mikeryanRight, it was fixed.
Comment #5
lucascaro commentedHi there, this is still an issue in 6.x-dev as of today. Is there a patch that can be ported to 6.x?
thanks!!
Comment #6
lucascaro commentedwell, I just went ahead and backported the __toString method for the three classes that had problems (MigrateDestinationFlagSimple, MigrateDestinationOGMembership, and MigrateDestinationUserRelationships)
here's a patch for those of us stuck with D6 :)
Comment #7
wonder95 commentedPatch from #6 fixed the error for me.
Comment #8
lucascaro commentedstatus update
Comment #9
apotek commentedCould not get the patch to apply cleanly:
I'm probably not applying to latest dev of migrate_extras. However, I applied the changes manually to my migrate_extras module, and the error has disappeared, so here's a +1 on this patch.
Comment #10
apotek commentedTwo tests of this change report it works. Updating to reviewed & tested.
also updating to 6.x-2.x-dev since the issue is fixed in 7, but not in 6.
Comment #11
mikeryanTried applying the patch - but the __toString() implementations are already there in 6.x-2.x-dev.
Comment #13
nerdcore commentedSame error today:
Migrate Extras: 7.x-2.2
Migrate: Upgrading 7.x-2.2 to 7.x-2.3
Fatal error: Class MigrateDestinationFlagSimple contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (MigrateDestination::__toString) in [...]/sites/all/modules/contrib/migrate_extras/flag.inc on line 78Comment #14
nerdcore commentedForgot which module's Issues page I was on, sorry :)
Comment #15
nerdcore commentedSorry for all the comments...
I do realize this is fixed in 7.x-2.x-dev but my organization is trying to implement regular upgrade procedures, and we were looking at upgrade paths to full version releases (no betas, no rcs, no devs). In this process we upgraded Migrate to 7.x-2.3 which exposed this incompatibility in Migrate Extras 7.x-2.2.
It's great to have the fix committed to Migrate Extras 7.x-2.x-dev but it seems like 7.x-2.2 remains broken...
Is it reasonable to expect this fix to be put forward into a 7.x-2.3 release soon?
Comment #16
mikeryanI expect to review the outstanding issues and cut 2.3 between Drupalcon sessions this week.
The correct status for an issue whose fix is committed, regardless of whether it has appeared in an official release, is fixed.