if (arg(0) == 'node' && is_numeric(arg(1))) {
        $node = node_load(arg(1));
        if ($node) {
          return $node->uid;
        }
      }

This code is always runned, it ignores the option.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dagmar’s picture

Status: Needs review » Needs work

Also, if this settings will be a checkbox, it would be nice to see an 'bool' => TRUE, and 'translatable' => FALSE in the option defintions.

dawehner’s picture

Status: Needs work » Needs review
FileSize
1.37 KB

Fixed this changes + the other issue in the 6.x-3.x patch.

There is no option_definition in 6.x-2.x

merlinofchaos’s picture

Status: Needs review » Fixed

Committed to all branches.

Status: Fixed » Closed (fixed)

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