Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I noticed that the module hardcoded the library location:
function _plupload_library_path() {
return variable_get('plupload_library_path', module_exists('libraries') ? libraries_get_path('plupload') : 'sites/all/libraries/plupload');
}
This creates an issue in how an install profile / distribution can use this module/library.
Comments
Comment #1
markwk CreditAttribution: markwk commentedI'm trying to come up with a distribution using an external library + module so we can follow that example.
Comment #2
slashrsm CreditAttribution: slashrsm commentedHm... this code will use data provided by Libraries module if enabled. If one does not have Libraries it will use variable. How would you make this more flexible?
Comment #3
markwk CreditAttribution: markwk commentedGood point. Was in a hurry when I posted the issue. I'll check this with libraries. I assume that will fix the issue, I hope. Thanks for the quick response.
Comment #4
slashrsm CreditAttribution: slashrsm commentedNP. Closing the issue. Feel free to reopen if you need any other info.
Comment #5
markwk CreditAttribution: markwk commentedThanks.