Incompatibility with i18n

I have noticed an incompatibility between image and i18n.

When i18n is active, inline images are displayed with a wrong link to original content. The problem is that this link includes the language prefix. In other words, if the original image URL is http://x.com/files/foo.jpg, the rendered link will be something like http://x.com/en/files/foo.jpg (when english is the active language).

I noticed that this was not the case for inline files. So I had a look at the code to check for differences.

Currently file_create_url() is used in the case of a file, and not in the case of an image.

I tried to use it as well for images (line 334 for version 1.29 of image.module), and it seems to be OK.

329 : sun 1.25 if (variable_get('inline_link_img', '1')) {
330 : $attributes = array(
331 : 'class' => 'inline-image-link',
332 : 'title' => t("View") .': '. $title,
333 : );
334 : $html = l($image, file_create_url($file->filepath), $attributes, NULL, NULL, FALSE, TRUE);
335 : rha 1.12 }
336 : sun 1.24 else {
337 : $html = $image;
338 : matteo 1.1 }

Hope this helps,

-- Denis.