I don't mind hard-coding the location of a library, as long as that location is sites/all/libraries and not the module folder. I saw a similar issue to this tagged d7, I'm tagging this one d6 because I don't think you need the libraries module, at least not for this quick fix. Please just alter your one line of code that opens the library, and change the location to sites/all/libraries. Yes, it will break people's current installs, but that's going to happen anyway with the library in the modules folder. This way it only breaks once instead of every single time there's a new dev release.

Comments

greggmarshall’s picture

greggmarshall
Location Aurora/Denver, Colorado
CreditAttribution: greggmarshall commented

I have to agree, I like the idea of having all my third party libraries in one place, not scattered around in various modules' directories.

daniorama’s picture

daniorama CreditAttribution: daniorama commented

Using libraries folder is the logic way. I hope they fix it so upgrading would be faster and easier.

rickvug’s picture

rickvug CreditAttribution: rickvug commented
Status: Active » Closed (duplicate)

I think we're all in agreement here. Marking as a duplicate of #902410: plupload_library() requires plupload library to be in module folder rather than flexible or sites/all/libraries.

justintime’s picture

justintime CreditAttribution: justintime commented
Category: task » feature
Status: Closed (duplicate) » Active

Re-opening because the 6.x and 7.x branches are basically separate projects. I'm about to resolve this for 6.x but don't want to close it for 7.x.

justintime’s picture

justintime CreditAttribution: justintime commented
Status: Active » Needs review

Please apply the patch in this comment. Along with working with new versions of plupload, it has support for sites/all/libraries as well as libraries module

justintime’s picture

justintime CreditAttribution: justintime commented
Status: Needs review » Fixed

Since patch from #1124476: plupload library versions >= 1.4.3 change file paths has been committed, this issue should be fixed as well.

carwin’s picture

carwin CreditAttribution: carwin commented

probably need to update the readme

justintime’s picture

justintime CreditAttribution: justintime commented

@carwin - I already did: http://drupalcode.org/project/plupload.git/blobdiff/ed3cea20d4c299e97303...

rickvug’s picture

rickvug CreditAttribution: rickvug commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Fixed » Patch (to be ported)

This hasn't been ported to the 7.x-1.x-dev branch yet.

justintime’s picture

justintime CreditAttribution: justintime commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Patch (to be ported) » Fixed

Please do a quick search before making changes to existing fixed issues. D7's issue for this is over at #902410: plupload_library() requires plupload library to be in module folder rather than flexible or sites/all/libraries - it's even mentioned in #3 above.

rickvug’s picture

rickvug CreditAttribution: rickvug commented

Sorry about that. I should have known better with the D7 version being quite different and all.

dddave’s picture

dddave CreditAttribution: dddave commented
Status: Fixed » Needs work

Sorry for reopening but there is a small yet dangerous typo in the Readme. You want users to create a "libaries" folder.

justintime’s picture

justintime CreditAttribution: justintime commented
Status: Needs work » Fixed

No worries, good catch. Fixed that, and also fixed the typo on 'dowload'.

Status: Fixed » Closed (fixed)

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