nodequeue.install l.369-413 :
Missing parenthesis in the IF condition line.

/**
 * Implementation of hook_requirements().
 *
 * We need the translation_helpers module to enable internationalization for queues.
 */
function nodequeue_requirements($phase) {
  $requirements = array();
  $t = get_t();
  if ($phase == 'install' || $phase == 'runtime' && module_exists('translation') && !module_exists('translation_helpers')) {
    $requirements['nodequeue_translation']['title'] = $t('Nodequeue');
    $requirements['nodequeue_translation']['severity'] = REQUIREMENT_WARNING;
    $requirements['nodequeue_translation']['description'] = $t('To have the nodequeue module work with translations, you need to install and enable the !url
module.', array('!url' => l('translation_helpers', 'http://drupal.org/project/translation_helpers')));
    $requirements['nodequeue_translation']['value'] = $t('Translation helpers module not found.');
  }
  return $requirements;

IF condition :

  if ($phase == 'install' || $phase == 'runtime' && module_exists('translation') && !module_exists('translation_helpers')) {

...should be:

  if ( ($phase == 'install' || $phase == 'runtime') && module_exists('translation') && !module_exists('translation_helpers')) {
CommentFileSizeAuthor
#1 528598.patch1.08 KBrobloach

Comments

robloach’s picture

Status: Active » Needs review
StatusFileSize
new1.08 KB

Reasonable.

ezra-g’s picture

Title: nodequeue_requirements to be fixed » Conditional in nodequeue_requirements needs more parens
Status: Needs review » Fixed

Thanks to both of you!

This is committed.

Status: Fixed » Closed (fixed)

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