I was uninstalling some modules and when i try to disable Commerce Kickstart Migration module, i get a fatal error page.

The log says:

PHP Fatal error: Class 'CommerceKickstartNode' not found in /xxxxxxxxxxxx/profiles/commerce_kickstart/modules/contrib/migrate/includes/base.inc on line 437, referer: http://xxxxxxxx/admin/modules/uninstall/confirm

Comments

GuGuss’s picture

Assigned: Unassigned » Haza
bojanz’s picture

Priority: Critical » Normal

This might not be an issue anymore, since we changed the migrations a lot.

Needs retestng.

GuGuss’s picture

Assigned: Haza » vasike

@vasike : Can you test that to see if it's fixed ?

vasike’s picture

Version: 7.x-2.0-alpha1 » 7.x-2.x-dev

i couldn't see this issue on uninstalling.
BUT there are "similar" issues for disabling Commerce Kickstart Migration module.
it seems that all "demo_content" modules that have migration, use migration classes that extends CommerceKickstartMigration class.
which provide an error like:

Fatal error: Class 'CommerceKickstartMigration' not found in drupal_path/profiles/commerce_kickstart/modules/commerce_kickstart/demo_content/commerce_kickstart_merchandising/commerce_kickstart_merchandising.migrate.inc on line 8

questions:
1. should we close this and open a new issue, or we can continue on this.
2. a solution could be a dependency of Commerce Kickstart Migration for these (demo content) modules.

vasike’s picture

Status: Active » Needs review

Commerce Kickstart Migration dependency added for kickstart demo content modules - https://code.drupalcommerce.org/#/c/210/

bojanz’s picture

Status: Needs review » Fixed

That makes sense. Merged.

pedrorocha’s picture

I'm not sure if is about the same question, but i disabled Commerce Kickstart Blog UI and another error message, similar to the previous one, came:

PHP Fatal error: Class 'CommerceKickstartMigration' not found in /home/pedrorocha/Workspace/www-localhost/XXX/profiles/commerce_kickstart/modules/commerce_kickstart/demo_content/commerce_kickstart_blog/commerce_kickstart_blog.migrate.inc on line 8, referer: http://localhost/XXX/admin/structure/features

It was on http://localhost/xxx/admin/structure/features/cleanup/clear

I put "dependencies[] = commerce_kickstart_migrate" on "commerce_kickstart_blog.info" and disabled Commerce Kickstart Blog without the problem, so i guess it's really ok.

This will go on alpha4, right?

bojanz’s picture

Yes, that's the same error, and the fix will be included in the next release (as will all other fixes committed since the last release was tagged)

Status: Fixed » Closed (fixed)

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