Install failed due to globalredirect.install calling a function from globalredirect.module, which hasn't been loaded yet.

Workaround follows:


diff -u -r1.4 globalredirect.install
--- globalredirect.install      13 Jul 2010 00:46:27 -0000      1.4
+++ globalredirect.install      10 Dec 2010 20:50:29 -0000
@@ -28,6 +28,9 @@
   $ret = array();
 
   // Get the default settings
+  if (!function_exists('_globalredirect_get_settings')) {
+    require_once(dirname(__FILE__) . '/globalredirect.module');
+  }
   $defaults = _globalredirect_get_settings(TRUE);
 
   // Define a settings array based on the variables already set (or their defaults)
CommentFileSizeAuthor
globalredirect-DRUPAL-7--1.patch707 bytespillarsdotnet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented

this seems like the best place to report this error:

An AJAX HTTP error occurred. HTTP Result Code: 200 Debugging information follows. Path: http://new.openconcept.ca/update.php?id=147&op=do StatusText: OK ResponseText: Fatal error: Call to undefined function _globalredirect_get_settings() in /home/dm7/sites/all/modules/contrib/globalredirect/globalredirect.install on line 31

I don't know if this patch fixes the problem.

adellefrank’s picture

adellefrank CreditAttribution: adellefrank commented
Status: Needs review » Reviewed & tested by the community

This fixes the installation issue, as far as I can tell. You'll need to run cron before global redirect will pick up on your legacy aliases from Drupal 6.

bfroehle’s picture

bfroehle CreditAttribution: bfroehle commented

~

bfroehle’s picture

bfroehle CreditAttribution: bfroehle commented

You'll need to run cron before global redirect will pick up on your legacy aliases from Drupal 6.

Why can't they be processed in a hook_update_N() function? That's what the end user is expecting.

bfroehle’s picture

bfroehle CreditAttribution: bfroehle commented

Already fixed in #861994: Call to undefined function _globalredirect_get_settings().

See #1045842: Development releases link to wrong branch for an explanation of why the Development download links on the main page are broken.

bfroehle’s picture

bfroehle CreditAttribution: bfroehle commented
Status: Reviewed & tested by the community » Closed (duplicate)