Unable to uninstall, PHP Fatal error: Call to undefined function media_vimeo_variable_default() in /drupal-7.2/sites/all/modules/media_vimeo/media_vimeo.install on line 19

What I do not get, the function is included in media_vimeo_module include_once('includes/media_vimeo.variables.inc'); so this should not happen, right? Including media_vimeo.variables.inc one more time works as a work around but that is not the way to do this. What is going on, is this a core issue?

CommentFileSizeAuthor
#5 1180030-uninstall-take2.patch1.77 KBsteinmb
#3 1180030_include_issue.patch1.55 KBsteinmb

Comments

kreynen’s picture

kreynen commented

Duplicate of #1164184: Fatal error while uninstalling: undefined function media_youtube_variable_default()

steinmb’s picture

steinmb commented

Thanks, cross posted to that issue, but leaves this open until we see some progress on the issue and that I'm able to get it confirmed that they are duplicates. That is, I know 99.9% that they are then they way they include functions files are identical.

steinmb’s picture

steinmb commented
StatusFileSize
new 1180030_include_issue.patch1.55 KB

Have been seeing these missing included in a lot of places, media module, media_youtube, media_vimeo. What's going on? The files get included, alright but when the included files/function are no where to be found. In Drupal 7 have there been changes to the "autoloader" http://drupal.org/node/224333#absolute_includes

Rolling a patch that follow these guidelines, and that also actually seems to fix this issue.

steinmb’s picture

steinmb commented

Might the problem originate from this issue #1029606: Regression: Not loading the .module file causes a fatal error when uninstalling some modules (as does loading it) that just got commited by webchick.

steinmb’s picture

steinmb commented
Status: Active » Needs review
StatusFileSize
new 1180030-uninstall-take2.patch1.77 KB

Doh! Never upload patches after been working for 16h. Pls. ignore #3, missed a few but important lines that where uncommitted, also the issue refereed to in #4 contained a patch with hints on best practice during uninstall that I implemented.

steinmb’s picture

steinmb commented
Priority: Major » Critical
Status: Needs review » Reviewed & tested by the community

Anyone with permission, could we get this committed pretty please?

kreynen’s picture

kreynen commented

committed http://drupalcode.org/project/media_vimeo.git/commit/45cd2de

kreynen’s picture

kreynen commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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