When using the Aegir hosting system with this module, errors appear when trying to migrate, backup, or clone a site. This is because Drupal is not fully bootstrapped during these operations and the module_load_include function is not defined. There are 2 instances of this function used to include the responsive_navigation_common.inc file. For me replacing these two functions with include_once fixed the issue and allowed Aegir to work properly to manage the Drupal site.

CommentFileSizeAuthor
#7 2018335.8-responsive_navigation-include-once-followup.patch433 bytesmrfelton
#3 include-once-2018335-3.patch1.28 KBadam-delaney

Comments

jrabeemer’s picture

jrabeemer commented

Ok. Please provide a patch. Thanks.

jrabeemer’s picture

jrabeemer commented
Assigned: Unassigned » jrabeemer
adam-delaney’s picture

adam-delaney commented
Status: Active » Needs review
StatusFileSize
new include-once-2018335-3.patch1.28 KB

Submitting a patch to resolve this issue using include_once.

kendouglass’s picture

kendouglass commented

The patch at #3 fixed my problem.

I was getting errors when I try to run drush commands:

Drush command terminated abnormally due to an unrecoverable error.
Error: Call to undefined function module_load_include() in /home/drupal/7/sites/all/modules/contrib/responsive_navigation/responsive_navigation.drush.inc, line 8

Thanks Adam.

blakehall’s picture

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

Working for me too.

jrabeemer’s picture

jrabeemer commented
Status: Reviewed & tested by the community » Closed (fixed)

Committed.

http://drupalcode.org/project/responsive_navigation.git/commit/84f5d0440...

mrfelton’s picture

mrfelton commented
Status: Closed (fixed) » Needs review
StatusFileSize
new 2018335.8-responsive_navigation-include-once-followup.patch433 bytes

Looks like you forgot to make the change in responsive_navigation.drush.inc. Follow u patch attached.

jrabeemer’s picture

jrabeemer commented

Thanks. I'll commit it soon.

scottrigby’s picture

scottrigby
he/him
Primary language English
Location Brooklyn, NY
commented
Status: Needs review » Reviewed & tested by the community

Just sayin'

bjlewis2’s picture

bjlewis2 commented

+1

Manually made the change in #7, and no longer get the error when running "drush updb".

jrabeemer’s picture

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

Committed to 7.x-1.x-dev

http://drupalcode.org/project/responsive_navigation.git/commit/40a1e29

Hopefully this fixes the drush/aegir bugs. Why does this function exist? :-/

Thanks.

v7nguyen’s picture

v7nguyen commented
Version: 7.x-1.x-dev » 7.x-1.0

Hi Newbie,

I am getting into the same error code when running rush with verion 7.x-1.0, is it possible to apply the same patch? Thanks in advance :))

hass’s picture

hass commented
Version: 7.x-1.0 » 7.x-1.x-dev

Status: Fixed » Closed (fixed)

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

nithinkolekar’s picture

nithinkolekar commented
Issue summary: View changes

existing 7.x-1.0 is show error when running drush command but when tried to drush pm-update responsive_navigation it returns No code updates available. .

Is this patch only committed to dev branch?