http://drupalcode.org/project/xautoload.git/blob/refs/heads/7.x-2.x:/xau... introduced a dependency on the libraries module 2.x+. Using a module_exists() will not work here since libraries 1.x is not compatible. A function_exists() would be helpful.

Love the updates to the module over the past month!

CommentFileSizeAuthor
#1 1833382-libraries.patch1.17 KBindytechcook
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented
Status: Active » Needs review
FileSize
1833382-libraries.patch1.17 KB

Patch Attached

donquixote’s picture

donquixote CreditAttribution: donquixote commented

Are you sure it does?
I assumed the hook would only be invoked if libraries is installed!

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented
Title: Dependency on Libraries » Dependency on Libraries 2.x

You are correct about the hook calling but it still requires 2.x. On new sites this isn't a big deal but I'm trying to implement SPS in open public which uses 1.x version of libraries.

donquixote’s picture

donquixote CreditAttribution: donquixote commented

Ah! This is good to know.
That's going to be a hotfix, next x hours :)

donquixote’s picture

donquixote CreditAttribution: donquixote commented
Status: Needs review » Fixed

Hotfix in 7.x-2.6
(still waiting for the tag to be available for creating a new release)
Thanks!

http://drupalcode.org/project/xautoload.git/commitdiff/d1c1e463c63f59ac5...

donquixote’s picture

donquixote CreditAttribution: donquixote commented

I made this tag from 7.x-2.x-hotfix, not from 7.x-2.x. Maybe this prevents the new release?

donquixote’s picture

donquixote CreditAttribution: donquixote commented

Finally was able to release the tag.
Just a matter of waiting.

Status: Fixed » Closed (fixed)

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