Rather than putting jquery.cycle.all.min.js in with the other javascript files that belong to the module, would it be possible to instruct users to create another sub dir "modules/views_slideshow/contrib/views_slideshow_cycle/js/jquery_cycle".
This would make it a lot easier to deploy sites and unzip the jquery cycle library into it's own dir instead of one that already exists. [At least for my use case when using drush make it would :) .] It looks like this would only be a minor patch to do this although I appreciate that this is probably also affects the 6.x-3.0-dev branch. If you are amenable to this suggestions perhaps the same could be done for the json2 library on github.
Here are two patches, one for each library.
Comment | File | Size | Author |
---|---|---|---|
#10 | 1012468-libraries-1.patch | 4.42 KB | redndahead |
views_slideshow-7.x-3.0-dev-change-path-to-json2-library.patch | 2.25 KB | eft | |
views_slideshow-7.x-3.0-dev-change-path-to-jquery-cycle-library.patch | 2.38 KB | eft | |
Comments
Comment #1
justafishIt'd probably be better to move it to sites/all/libraries and integrate with Libraries API so everything doesn't break when the module is updated
http://drupal.org/project/libraries
Comment #2
eft CreditAttribution: eft commentedagreed - sites/all/libraries or profiles/[profile-name]/libraries or however the Libraries API works
Comment #3
redndahead CreditAttribution: redndahead commentedI am planning on moving to libraries, but I probably won't move to libraries api right now. Things are unstable, not just the modules status, in that module right now so I don't want to implement the api to just have it change.
Comment #4
eft CreditAttribution: eft commentedthanks - what about the minor changes in #0?
Comment #5
redndahead CreditAttribution: redndahead commentedI don't see any other changes than moving the libraries.
Comment #6
eft CreditAttribution: eft commentedthe patches in #0 are simply separating the libraries into different directories - they don't require refactoring to use libraries api
Comment #7
redndahead CreditAttribution: redndahead commentedI'm still going to put them in sites/all/libraries but I'm not going to use the libraries api. Not necessarily how people like to do it, but I think it may be the best approach for now.
Comment #8
eft CreditAttribution: eft commentedwell, that works for me :)
will this get in dev soon?
Comment #9
thill_ CreditAttribution: thill_ commented+1 subscribe
Comment #10
redndahead CreditAttribution: redndahead commentedThis patch changes it to sites/all/libraries/json2 and sites/all/libaries/jquery.cycle
Comment #11
redndahead CreditAttribution: redndahead commentedI went and committed this.
So you should put the files in
sites/all/libraries/json2/json2.js
sites/all/libraries/jquery.cycle/jquery.cycle.all.min.js
Comment #12
thill_ CreditAttribution: thill_ commentedThis isn't very clean but better than nothing.
For those of you using drush make these are the lines you will need to add to your make file assuming your make file is in the profile/[profile-name] directory
Comment #13
eft CreditAttribution: eft commentedthanks redndahead !
Comment #14
redndahead CreditAttribution: redndahead commented@thill_ it's not perfect. I don't think I want to completely reimplement libraries api. As soon as that's stable I will switch to that and then it'll be more robust.
Comment #15
thill_ CreditAttribution: thill_ commentedI didn't mean to switch the status, sorry about that. I agree it is usable like this