TinyMCE image assist popup not redirecting to new installation folder 'libraries'

mariagwyn - May 21, 2009 - 20:08
Project:Image Assist
Version:6.x-3.x-dev
Component:Wysiwyg API Plugin
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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

#1

iva2k - May 23, 2009 - 19:28

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?

AttachmentSize
img_assist-librarypath01.patch 732 bytes

#2

sun - May 26, 2009 - 10:37
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...

#3

iva2k - May 27, 2009 - 02:57

>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.

#4

sun - June 9, 2009 - 00:55
Status:needs work» fixed

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.

AttachmentSize
img_assist.wysiwyg-libraries.patch 1.28 KB

#5

tomsm - June 10, 2009 - 07:19

works great!

#6

ansorg - June 23, 2009 - 14:39

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?

#7

Frodo Looijaard - July 1, 2009 - 21:26

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

Frodo

#8

sun - July 2, 2009 - 08:56
Status:fixed» patch (to be ported)

#9

sun - July 4, 2009 - 11:58
Status:patch (to be ported)» fixed

Committed to 2.x as well.

#10

System Message - July 18, 2009 - 12:00
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.