Load the files before stating the Library is loaded

Hmm... that makes sense. We actually want both, though. We don't do that yet, but a use-case for a 'pre-load' callback is turning filepaths like "jquery-[version]-min.js" into the proper names so they can be loaded. So this is actually a feature request, however a very valid one.

+++ b/libraries.api.php
@@ -275,11 +275,27 @@ function hook_libraries_info() {
+      'info' => array(
+        'mymodule_example_libraries_info_callback',
+      ),

That should probably come first.

+++ b/libraries.module
@@ -596,10 +597,14 @@ function libraries_load($name, $variant = NULL) {
+      // The "loaded" flag is determined by whether all files were loaded.

That's not really correct. 'loaded' currently counts the number of loaded files, so if ($library['loaded']) {}<\code> works, but it's not really a flag. Maybe we can just leave out the comment. :)

This looks really good already. We need to update the hook_libraries_info() documentation, though, and we need some tests for this. The callbacks are already tested in general so we just need to adapt the existing tests.

"pre-load" and "post-load" is seld-documenting and is inline with "pre-detect" and "post-detect", so let's go with that.

Thanks very much for the patch. That was exactly what I was looking for. Below are some nitpicks, but they are really minor. And the tests show that this works as expected. I'll commit this in the next few days, and unless you want to roll another patch (which would be even more awesome) I'll fix the stuff below pre-commit.

+++ b/libraries.api.php
@@ -159,10 +159,14 @@
+ *       after this library is loaded. The library now also contains a "loaded"
+ *       variable, which holds how many files were processed during the load
+ *       sequence.

To be more inline with the other groups, that should be something like: "At this point the library contains a 'loaded' key, which contains the number of files that were loaded."

+++ b/libraries.api.php
@@ -275,11 +279,27 @@ function hook_libraries_info() {
+      // Used to alter the info associated with the Library.

And below: Library -> library.

+++ b/tests/libraries.test
@@ -322,11 +327,12 @@ class LibrariesTestCase extends DrupalWebTestCase {
+    $this->assertEqual($library, $expected, 'Pre-Load callback was applied correctly.');

And below: That should be "Pre-load and post-load callbacks ..."

Thanks for the reroll. Below are some very minor nitpicks. I'm still marking RTBC, though, because (unless you'd like to re-roll again, which would be awesome, of course), I'd happily fix that prior to commit. If I weren't so terribly tired right now, I'd go ahead and commit it right now. I hope to get to that beginning of next week, though.

+++ b/libraries.api.php
@@ -159,10 +159,13 @@
+ *     - post-load: Callbacks registered in this group are applied directly
+ *       after this library is loaded. At this point, the library contains a
+ *       "loaded" key, which contains the number of files that were loaded.

"loaded" -> 'loaded', also I think the first line wraps early.

+++ b/tests/libraries.test
@@ -370,14 +376,16 @@ class LibrariesTestCase extends DrupalWebTestCase {
+    // When the library information is cached only the pre-load callback group
+    // should be invoked.

"... only the pre-load AND POST-LOAD callback groupS ..."

I did some minor documentation fixes pre-commit, and then committed this. Patch is attached.
Thanks again!
http://drupal.org/commitlog/commit/10030/3db1b3ed803d00a4633096fa43faafb...