the most recent updated requires tinymce to be installed in 'sites/all/libararies/tinymce.' However, the Image_assist popup is still look in the old location. As a result, no images appear in the popup window: 'sites/all/modules/wysiwyg/tinymce/jscripts/tiny_mce/tiny_mce_popup.js?'

This does not appear to be an image assist issue as image assist without tinymce works just fine. Perhaps something didn't get updated?

Note, this was true in the dev version of May 16 as well as May 19

CommentFileSizeAuthor
#4 img_assist.wysiwyg-libraries.patch1.28 KBsun
#1 img_assist-librarypath01.patch732 bytesiva2k
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

iva2k’s picture

iva2k CreditAttribution: iva2k commented
FileSize
img_assist-librarypath01.patch732 bytes

I patched img_assist.module to use the right location when working with Wysiwyg TinyMCE.

@sun:
I'm not sure if I have to open a new issue in img_assist queue and post the patch there?

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Project: Wysiwyg » Image Assist
Version: 6.x-2.x-dev » 6.x-3.x-dev
Component: Editor - TinyMCE » Wysiwyg API Plugin
Status: Active » Needs work

Nope, we can simply move this issue. Good catch! :)

Only the patch is a bit dirty...

iva2k’s picture

iva2k CreditAttribution: iva2k commented

>Only the patch is a bit dirty...
Agree. It was indeed a quick hack to just patch 4 sites that I maintain. Given the state of flux for Wysiwyg API and no clear documentation, I guess only sun knows how to fix it so it won't break later.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs work » Fixed
FileSize
img_assist.wysiwyg-libraries.patch1.28 KB

Thanks for reporting, reviewing, and testing! Committed attached patch to all 3.x branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

tomsm’s picture

tomsm CreditAttribution: tomsm commented

works great!

ansorg’s picture

ansorg CreditAttribution: ansorg commented

hello,

I run into the same issue using Image Assist 6.x-2.0-alpha3 and Wysiwyg 6.x-2.0

can this patch be applied there as well?

Frodo Looijaard’s picture

Frodo Looijaard CreditAttribution: Frodo Looijaard commented

I agree: the 2.0 branch has the same issue, and the same patch solves it.

Frodo

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Fixed » Patch (to be ported)
sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Patch (to be ported) » Fixed

Committed to 2.x as well.

Status: Fixed » Closed (fixed)

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