We're running into conflicts with http://drupal.org/project/file_entity since it is *the* location to have the definitive file_uri_to_object() function until fixed in core. It would be nice if plupload could just namespace its version of the function for its own purposes rather than rely on a function that might get defined after the module is loaded or could change at any time.

CommentFileSizeAuthor
#1 1240654-plupload-namespace-file-uri-to-object.patch2.91 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 1240654-plupload-namespace-file-uri-to-object.patch2.91 KB
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Makes sense to me. I don't maintain the 7.x branch, though.

robertgarrigos’s picture

robertgarrigos commented
Status: Needs review » Reviewed & tested by the community

Applied this patch with success. Everything works fine.

greenwork’s picture

greenwork commented

Worked for me so far. Will update if I run into any problems but defiantly fixed the module enable problem.

gmclelland’s picture

gmclelland commented

#1 patch worked for me.

ergophobe’s picture

ergophobe commented

Patch worked for me, but on

Media 7.x-2.x-dev
File Entity 7.x-2.x-dev

Both downloaded 10 Nov 2011.

Anonymous’s picture

Anonymous (not verified) commented

is this patch in dev version? got the beta and went around with the issue.... solved with the info from here

slashrsm’s picture

slashrsm commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x: http://drupalcode.org/project/plupload.git/commit/abbf0da

Thanks!

Status: Fixed » Closed (fixed)

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

  • Commit abbf0da on 7.x-1.x, 7.x-2.x, 8.x-1.x authored by Dave Reid, committed by slashrsm: 
    Issue #1240654 by Dave Reid: Use a proper namespace for...