Working with IMCE translated to hebrew, the word 'KB' is translated to 'ק"ב' (notice the double quotes inside the string).

This breaks the code that inserts the image into the image selection window after selecting it in the image browser. see this comment and the two that follows.

The attached patch (against DRUPAL-5 branch) fixes this issue.

CommentFileSizeAuthor
imce_string_escape.patch2.82 KByhager

Comments

ufku’s picture

ufku commented

thanks for the patch.
Is it possible that filenames contain this quote char?
If so we need to change the method for calling imceFinitor.
In the attached patch it is called with only the row index of the file and properties of the file is resolved inside imceFinitor. Moreover eval is no more used.
please try it. if you find any errors you're wellcome to submit another.
I will commit this soon.

liorkesos’s picture

liorkesos commented

Hi ufku,
Filenames could theoretically include the '"' but I think it's wise to address that scenario as well.
We could have a filename called תשובות מהע"וד and it could retrigger the bug.
This is important because we're manually changing the translation these days to overide the bug.
What needs to be done in order to commit the patch? Can we help in something?
Lior

ufku’s picture

ufku commented

i'm going to create a bugfix release soon. this patch is going to be included.
thanks.

ufku’s picture

ufku commented
Version: 5.x-1.0 » 5.x-1.x-dev
Status: Needs review » Fixed

commited. thanks

Anonymous’s picture

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

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