See http://drupal.org/node/1811486 and http://drupal.org/node/1356276

wysiwyg is hand rolling its own get_libraries function, so the above patches will not work for wysiwyg when it's looking for libraries if the libraries are located in a base profile (not the current top level install profile).

CommentFileSizeAuthor
#1 wysiwyg-locate-base-profile-libraries-1996626-1.patch3.65 KBdelphian

Comments

delphian’s picture

delphian commented
StatusFileSize
new wysiwyg-locate-base-profile-libraries-1996626-1.patch3.65 KB

Patch is attached. Uses libraries module functions (which are the ones patched above), if they are available, to get all available libraries.

delphian’s picture

delphian commented
Status: Active » Needs review
delphian’s picture

delphian commented
Issue summary: View changes

Adding some clarity to original post.

sonicthoughts’s picture

sonicthoughts commented

Rolling own library was a big problem. Have not yet tested this patch but it is critical to support growing distro base.

dagomar’s picture

dagomar commented

I can confirm the patch in #1 works! Thanks.

dagomar’s picture

dagomar commented
Issue summary: View changes

Fixing typo.

mgifford’s picture

mgifford
he/him
Primary language English
commented
Issue summary: View changes
Parent issue: » #2343445: Stable release of Wysiwyg module, v7.x-2.4 please

  • TwoD committed 3053eeb on 6.x-2.x authored by delphian 
    - #1996626 by delphian: Libraries located in base profiles are not...
  • TwoD committed 711735e on 7.x-2.x authored by delphian
    - #1996626 by delphian: Libraries located in base profiles are not...
twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Status: Needs review » Fixed

Good and simple solution.
I'm truly sorry it took so long for me to get around to it.

Thank you for patching and reviewing!

Status: Fixed » Closed (fixed)

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