/**
 *  Implementation of hook_requirements().
 */
function hook_requirements($phase) {
  $requirements = array();
  // Ensure translations don't break at install time
  $t = get_t();

  if ($phase == 'install') {
    if (drupal_get_installed_schema_version('content') < 6000) {
      drupal_set_message($t("Some updates are still pending. Please return to !update and run the remaining updates.", array('!update' => l($t(), 'update.php', array('query' => 'op=selection')))), 'warning');
      $requirements['cck'] = array(
        'title' => $t('CCK'),
        'description' => $t('Updates for content.module need to be run first.<br/>Please re-run the update script.'),
        'severity' => REQUIREMENT_ERROR,
      );
    }
  }

  return $requirements;
}

Need I say more?

CommentFileSizeAuthor
#3 509760-embarassing-D6-1.patch688 bytesdave reid

Comments

mikey_p’s picture

Priority: Normal » Critical
Anonymous’s picture

Mikey_p:

I saw your tweet, and I have to say that you're not being very straightforward about the seriousness of this issue. You're assuming that I have some prior knowledge. If this issue is critical to your or other users' use of this module, you should say more.

It would be even better if you provided a patch.

If you cannot provide a patch, you could effectively provoke people like me to action if you give us a clue about why this is critical and what the problem is.

Thanks for your effort to bring an issue to the module maintainers' attention, but you're not helping the cause by being oblique.

(I see that it's the function name. Will you provide the patch or shall I?)

dave reid’s picture

Status: Active » Needs review
StatusFileSize
new688 bytes

Really, this is kind of funny that this hasn't been fixed even though it's been highly noticed on http://api.lullabot.com/hook_requirements. :)

aaron’s picture

Status: Needs review » Fixed

Hilarious! Thanks for the catch; fixed in dev. I hope to roll a new version this week.

Aaron

Status: Fixed » Closed (fixed)

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