Node Preview Errors, not generating appropriate URLS when rewrite engine off.

The image will not be in the final destination on preview. There is a menu callback that handles displaying the image... I'll see if I can reproduce later. Do you have clean-urls enabled? they shouldn't be required, but something you can test.

.darrel.

Check out an imageapi issue: http://drupal.org/node/236911#comment-777252.

Also check out these imagefield issues:
http://drupal.org/node/229965
http://drupal.org/node/230881
http://drupal.org/node/174075

Could these all be related?

I'm having the same issue. When you upload a picture it does not show on the resulting page, however it does show upon submit. I have drupal handling the images, not open http in file settings.

Same problem as issue 130701 on the DRUPAL-5 branch. When the download method is public and the site does not support clean URLs, file_create_url() will return a path to a file that does not exist and the menu system will not be used. Here's a patch to test.

I cannot reproduce with and with out clean urls.. using the DRUPAL-5--2.

dopry, you can't reproduce this if your site supports URL re-writing. If you use Apache, try disabling mod_rewrite and restarting.

Doesn't work with private files, rewrite enabled and clean urls.

@DarrenOh,
This came up with imagecache I believe and we had to implement a custom file_create_url there. It may be the same here. We should generate a test case for this issue. I'd be happy to write the test, but what we really need to know is how the real and preview URLS should look in the following cases...

public files, clean urls on
public files, clean urls on
private files, clean urls off
private files, clean urls off

If you can create that set of URL's I'll write a test case we can work with to resolve this issue.

I'll work on it.

Correct URLs:

public files, clean urls on

http://example.com/files/directory/image

public files, clean urls off

http://example.com/?q=files/directory/image

private files, clean urls on

http://example.com/system/files/directory/image

private files, clean urls off

http://example.com/?q=system/files/directory/image

The problem with the current code is that file_create_url() will always generate http://example.com/files/directory/image for public files. The solution is to place the menu path in $file['preview'] and use url() instead. Patch attached.

Where is the DEV for 2.1? It only lists DEV for 1.X

Good Job... work so fine for me!

Looks like pedrofaria forgot to change the status.

Thanks Darren Oh. Committed to the 5.x branch.