Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.