See patch.

CommentFileSizeAuthor
img_assist_8.patch1.48 KBkeve

Comments

zoo33’s picture

zoo33 commented

Interesting. From what I can tell, this would require that the module is installed in the sites directory (which is indeed the preferred location). On the other hand, the current code assumes that the module is in /modules, which of course is not good. But isn't there a better way to find the base_url of the Drupal installation in javascript?

keve’s picture

keve commented

You are right. But i am not that good in javascript to offer other solution. :(

zoo33’s picture

zoo33 commented
Status: Needs review » Closed (duplicate)

I've followed this up in #123203. Thanks for pointing out the problem!

jurriaanroelofs’s picture

jurriaanroelofs commented

Thanks.
this should be commited to the module. Facilitating inline images in drupal is hard enough as it is!

zoo33’s picture

zoo33 commented

That other issue I mentioned has had a patch committed. This problem should be gone in current 5.x-1.x-dev versions as well as in the 5.x-1.5 release. If not, please re-open that issue.