I have the Ace Editor running great on one Drupal 7.31 site, but on different 7.31 site that happens to be a multi-site, I get:

The Ace Editor JS library is missing, please check the README for installation instructions.

The readme says to extract and place the contents of the zip file under sites/all/libraries so that ace.js is located at sites/all/libraries/ace/src/ace.js. What about multi-sites?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

interdruper’s picture

Status: Active » Needs review

Could you please test the last dev version in your multi-site environment ? Particularly the already committed patch #2270799: Not working when library is not in sites/all/libraries may fix your issue.

torotil’s picture

The newest dev-version works just fine for me. :)

[… with the library installed in sites/mysite/libraries/ace/src/…]

interdruper’s picture

Status: Needs review » Closed (fixed)
torotil’s picture

Status: Closed (fixed) » Needs review
FileSize
625 bytes

Seems that there is something missing after all. In ace_editor_library_installed() sites/all is still hardcoded. Here is a patch for that.

The patch also removes the unused function ace_editor_include_installed().

torotil’s picture

Category: Support request » Bug report
interdruper’s picture

Status: Needs review » Closed (fixed)
mkhamash’s picture

Version: 7.x-1.1 » 7.x-1.x-dev
Component: Documentation » Code
Priority: Major » Normal
Status: Closed (fixed) » Needs review
FileSize
554 bytes

There is still a static path ("sites/all/libraries") hanging around in the latest dev in ace_editor_requirements hook implementation, I think this is the last one of them.

  • interdruper committed 418888c on 7.x-1.x authored by mkhamash
    Issue #2330383 by mkhamash,interdruper: Multi-site libraries path.
    
interdruper’s picture

Status: Needs review » Closed (fixed)