I'm getting a lot of these warnings:

Notice: Use of undefined constant PERMISSIONS_VARIABLE_CONFIG_PATH - assumed 'PERMISSIONS_VARIABLE_CONFIG_PATH' in permissions_variable_menu() (line 57 of ...\sites\all\modules\contrib\permissions_variable\permissions_variable.module).

For some reason Drupal's module_load_include function isn't working and https://api.drupal.org/api/drupal/includes!module.inc/function/module_lo... recommended using require_once instead, which worked. See attached patch.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

emmonsaz’s picture

emmonsaz’s picture

Status: Active » Needs review
emmonsaz’s picture

Status: Needs review » Needs work
thedavidmeister’s picture

Hey, thanks for the patch. That behaviour you've reported is pretty weird, but I'm happy to swap it out with require.

Can we please get a re-rolled patch that is generated from inside the module directory rather than outside it?

emmonsaz’s picture

Let's see if this one works...

emmonsaz’s picture

Status: Needs work » Needs review

thedavidmeister’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

emmonsaz’s picture