I get an "unknown action" error if i trie to edit the action of an preset.
The link looks like:
admin/build/imagecache/2/action/4

In Version 1.14 of imagecache_ui.module path has chenged to
$items['admin/build/imagecache/%imagecache_ui_preset/%imagecache_action']

In think the link has to be changed in :

admin/build/imagecache/2/4

CommentFileSizeAuthor
#4 imagecache_265593.patch2.73 KBdrewish
#1 imagecache_ui.module-unknown-action.patch884 bytestantenic

Comments

tantenic’s picture

tantenic commented
StatusFileSize
new imagecache_ui.module-unknown-action.patch884 bytes

in my opinion

line 307
'#value' => l(t('Configure'), 'admin/build/imagecache/'. $action['presetid'] .'/action/'. $action['actionid'] ),

has to be changed to
'#value' => l(t('Configure'), 'admin/build/imagecache/'. $action['presetid'] .'/'. $action['actionid'] ),

to fit line 71
$items['admin/build/imagecache/%imagecache_ui_preset/%imagecache_action']

I attached my first try to build a patch.

Moonshine’s picture

Moonshine commented
Component: User interface » Code
Status: Active » Needs review

Updating the status...

Your correction looks good to me and does fix the error.

jcfiala’s picture

jcfiala commented

I used the same fix (not having looked for this ticket until after I'd fixed it) and it works fine in the install I'm working with.

So glad this has a D6 version, even if dangerously untried - imagecache is just so darn useful.

drewish’s picture

drewish commented
StatusFileSize
new imagecache_265593.patch2.73 KB

give this a try, it tries to make all the paths a bit saner.

Moonshine’s picture

Moonshine commented

Works fine for me...

Fayna’s picture

Fayna commented

The latest patch worked great. No more error!

drewish’s picture

drewish commented
Status: Needs review » Fixed

great, committed to HEAD.

dopry’s picture

dopry commented

awesome.. good to see those pre d6 menu optimizations go bye bye!!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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