Uninstall problem

Hi,

Which version of Drupal 7 are you using? this is a problem identified in #1029606: Regression: Not loading the .module file causes a fatal error when uninstalling some modules (as does loading it) and solved both for D7 and D8 in June http://drupal.org/node/1029606#comment-4603656

There is not solution.

My solution is here:

function commerce_extra_panes_termsofservice_uninstall() {
/* New line here */
  module_load_include('module', 'commerce_extra_panes');
/* End new line */
  $extra_panes = commerce_extra_panes_get_panes();
  foreach ($extra_panes as $extra_pane) {
    $pane_id = 'extra_pane__' . $extra_pane->extra_type . '__' . $extra_pane->extra_id;
    variable_del('cep_tos_' . $pane_id);
    variable_del('cep_tos_required_' . $pane_id);
  }
}

As I posted in #2 I can't reproduce this and if you're using a Drupal version older than June (and you should), this should be safe.

Anyways we could be really sure that the module file is being included by doing a drupal_load.

I'll be happy to commit this when some getting the error confirms it fixes the problem.

The patch does not solve the issue. I'm on D7.10 with commerce 1.2 and the revision of commerce extra panes currently for download (20 Jan).

I have the same kind of error but i don't know if i can reproduce it.

The module is currently not working for me and i can't uninstall it.

Is there any manual way of doing it so that i can try working on a clean install?

#5 there's little to do if you can't reproduce a problem, how do you know that you've got a problem then?

Not sure of what does "can't uninstall it" mean, you should be fine just deleting the variables.

I know i had a problem cause i could not uninstall the module, since i was receiving a fatal error similar to the one in the main post :)
The solution proposed by @mpark worked flawlessly, but thanks anyway!

#7 how is it possible that the solution in #3 solves your problem but the same solution in a patch #4 does not?

Setting to RTBC after a chat with D4rKr0w on IRC

And committed!

Thanks everyone for reporting and give feedback.

#3 worked for me

#3 worked for me (drupal 7.12, extra panes 7.x-1.x-dev 2011-Mar-07).

@pcambra, commited? Doesn't seem so in 7.x-1.x-dev of 2011-Mar-07. Am I missing something?

here?
http://drupalcode.org/project/commerce_extra_panes.git/blob/refs/heads/7...

@pcambra,
Sorry, I was either tired or blind yesterday and could not see the change in current dev version...

I get the same problem, #3 works for me too
here is the patch to include in the module

Posted in #16 is the correct patch RTBC. It worked for me.

@pcambra, Patch in 4 was wrong as it try to load the extra_panes_termsofservice module instead of extra_panes module.

Thanks @Tomefa and all ;)

Fixed, thanks for the catch

#16 does not appear to be in the current stable release.

#3 and #16 patch worked for me too. Thank's a lot.

Just using patch from #16 did the trick for me Thanks!!

Patch #16 works for me in Drupal 7.56

Why is this patch not integrated in the module ????????

@remedact my guess is the maintainer is no longer maintaining the module, there have been no commits for about 4 years.

Instead of shouting, perhaps consider applying to be a maintainer of the module? (I'm no longer working on sites that use this or I would do so myself - I was too green to do so when I first commented).