Make libraries paths available to scripts in Drupal.settings [#1214056]

The attached patch extends Drupal.settings with the paths of all installed libraries. To get the path of the codemirror library for example, use:

Drupal.settings.libraries['codemirrorPath']

Comment	File	Size	Author
	libraries-js-settings.patch	540 bytes	davidtrainer

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

tstoeckler

he/him

German

Essen, Germany

CreditAttribution: tstoeckler commented 16 July 2011 at 08:50

Status:

Needs review

» Needs work

+++ b/libraries.module
@@ -6,6 +6,18 @@
+  foreach($libraries as $library=>$path) {

This should be:
foreach ($libraries as $library => $path) {
to abide by Drupal coding standards

+++ b/libraries.module
@@ -6,6 +6,18 @@
+    drupal_add_js(array('libraries' => array($library . 'Path' => $path)), 'setting');

I don't really like that string concatenation. I would like it better, if you could access the paths as Drupal.settings.libraries[paths][codemirror] or something like that.

Most importantly, though, what is the use-case for this?
In the 7.x-2.x branch we have a lot more information on the libraries than their path, and we definitely don't want store all of that information in JavaScript, so I would really like to know what you are doing with this information.