Support bulkRollback in og_membership Migrate destination [#1811408]

Hello,

Please include a bulkRollback method on the MigrateDestinationOGMembership. When rolling back migrations, this is significantly faster than rolling back each migrated membership individually.

Thanks for OG!
-G

Comment	File	Size	Author
#1	og-bulkrollback-1811408.patch	1.09 KB	gdl
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

gdl CreditAttribution: gdl commented 12 October 2012 at 19:42

File	Size
og-bulkrollback-1811408.patch	1.09 KB

Patch attached.

The MigrateDestination migration says that a derived class must impliment either bulkRollback() or rollback(), but says nothing about whether it can/should support both.

Comment #2

gdl CreditAttribution: gdl commented 17 October 2012 at 21:22

Status:

Active

» Needs review

Comment #3

danylevskyi

Myrhorod / Ukraine

CreditAttribution: danylevskyi commented 18 November 2012 at 04:56

Status:

Needs review

» Needs work

Thanks for your patch!
It worked for me, but there is a warning:

Warning: array_flip() [function.array-flip]: Can only flip STRING and INTEGER values! in EntityAPIController->load() (line 184 of entity/includes/entity.controller.inc).