Similar to #354719: Call to imageapi_get_available_toolkits() breaks install profiles :

The test for function_exists('imageapi_menu') fails when installing in a profile and thus breaks install profiles. It's not obvious to me how to fix this, I will work without this check for the time being...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drewish’s picture

Status: Active » Postponed (maintainer needs more info)

need more info... where is this being called. a patch would be great.

alex_b’s picture

Sorry, called in imagecache.install imagecache_requirements() when installed with an install profile:

  if (!function_exists('imageapi_menu')) {
    $requirements['imagecache_imageapi'] = array(
      'title' => $t('ImageAPI'),
      'value' => $t('Missing'),
      'severity' => REQUIREMENT_ERROR,
      'description' => $t('Imagecache requires ImageAPI to function.'),
    );
  }

I'd be happy to roll a patch but I am not quite sure what approach to take here. It's not clear to me why this test exists at all when imagecache depends on imageapi anyway (see .info file).

alex_b’s picture

Assigned: Unassigned » alex_b

I am going to roll 2 patches: one removes the check, the other one moves it to $phase == 'runtime'

alex_b’s picture

Here you go, both applicable to latest DRUPAL-5--2. Both address the problem that imagecache barfs on install profile installation.

alex_b’s picture

Status: Postponed (maintainer needs more info) » Needs review

Needs review.

drewish’s picture

Status: Needs review » Fixed

thanks, committed to DRUPAL-5--2.

Status: Fixed » Closed (fixed)

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