Took me a while, and I went deeper into menu router debugging than I really had to ...
But :
If you install the module and immediately try it out, the upload works just dandy (much nicer than I'd expected!) but then it said it would redirect to a caption editor ... and broke 404.

It turns out that you need to visit /admin/settings/image/image_fupload and save at least once before using the system, otherwise an internal setting in a local variable does not get initialized, and the logic breaks.

$image_node_types[$node_type]['type'] is NULL if you haven't saved the settings at least once.

There is probably a place in the code for this to be tidied up, not sure where.
Only replicable on a brand new system, as I guess variable_get() is persistant forever after.

Comments

grandcat’s picture

grandcat commented
Title: File not found at /node/add/image/list_images if you haven't changed the settings. » [image module part] Preview page error if no settings changes were done

Thank you for your research.
I know this bug but I thought it was fixed with the update.php and module_..install procedure.

According to your issue, it doesn't seem to be fixed correctly.
Another solution would be to introduce "hook_init" to create it dynamically if it does not exist.
What do you think?

dman’s picture

dman commented

looks like image_fupload_install() is trying to do the right thing, but the value that was missing for me was $image_node_types[$node_type]['type'], as required later in image_node_type_load

    } elseif (arg(3) == "list_images" && $image_node_types[$node_type]['type'] == "image") {

I don't understand all the logic there, but I imagine that setting 'type' => 'image' in image_fupload_install() would be enough.

Delphine Lepers’s picture

Delphine Lepers commented

Thank you so much for sharing this !

cayenne’s picture

cayenne commented

Wow! Just ran into this problem here. Glad there's an easy workaround. Great module apart from that....