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

markwk’s picture

markwk commented

I'm trying to come up with a distribution using an external library + module so we can follow that example.

slashrsm’s picture

slashrsm commented

Hm... 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?

markwk’s picture

markwk commented

Good 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.

slashrsm’s picture

slashrsm commented
Status: Active » Fixed

NP. Closing the issue. Feel free to reopen if you need any other info.

markwk’s picture

markwk commented

Thanks.

Status: Fixed » Closed (fixed)

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