.admin.inc: variable_get('colorbox_auto_image_nodes', 'disabled'), possible values = ['disabled', 'colorbox']

.module: variable_get('colorbox_auto_image_nodes', 0), assumed possible values = [0, 1]

The attached patch changes the form options in admin.inc to match the module's 0 or 1 since there is only 2 options - enabled or disabled. This is more maintainable since checking for == 'colorbox' could lead to trouble if someone changes the form options.

CommentFileSizeAuthor
colorbox.admin_match-image-node-vars.patch684 bytesrecrit
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

recrit’s picture

recrit CreditAttribution: recrit commented

forgot to mention, you will need to resave your settings for the change to take effect.

frjo’s picture

frjo CreditAttribution: frjo commented
Status: Needs review » Fixed

Thanks! Committed to 6-dev.

Status: Fixed » Closed (fixed)

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