I am trying to upgrade this module to work with Drupal 7 and the upgrade fails due to this error:
An AJAX HTTP error occurred. HTTP Result Code: 200 Debugging information follows. Path: http://site.com/update.php?id=1014249&op=do StatusText: OK ResponseText: Fatal error: Call to undefined function imagecache_presets() in /home/site/public_html/sites/all/modules/imagecache_actions/canvasactions/imagecache_canvasactions.install on line 38
Any ideas how to get around this?
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 1237226-remove-d6-upgrade-hook.patch | 1.51 KB | mrfelton |
Comments
Comment #1
makeapage commentedHave the same problem....
Comment #2
makeapage commentedFurther info:
Notice: Undefined index: update_success in update_results_page() (line 162 of /home/xxx/public_html/www.xxx.com/update.php).
The update process was aborted prematurely while running update #6201 in imagecache_canvasactions.module. All errors have been logged.
Comment #3
mariano.barcia commentedsame problem here
Comment #4
mrfelton commentedSame here.
it's referring to code in imagecache_canvasactions_update_6201(). Is that update hook still supposed to be in the Drupal 7 version of the module? Looks like a Drupal 6 update hook.
Comment #5
mrfelton commentedI tried upgrading to 2.0 before upgrading to D7, but made no difference. Attached patch seems to resolve it for me:
Comment #6
moices commentedThe function imagecache_presets do not exist anymore in drupal 7.
image_styles should be used.
Replace imagecache_presets for image_styles in imagecache_canvasactions.install.
Comment #7
Aurochs commentedWhy did not they commit this in the latest dev release???
Comment #8
fonant commentedFix in #6 seems to work for me.
Comment #9
fietserwinPatch is OK, even though you should have upgraded in Drupal 6 and thus already be on 6201. But as there is no upgrade path from image presets (D6) to image styles (D7) upgrading does not make any sense. So I will remove this upgrade function (as in the patch).
Note: it looks like the whole install file can be removed, but will check that later.