Download & Extend
Plupload integration » Issues

Use a proper namespace for file_uri_to_object()

Posted by Dave Reid on August 5, 2011 at 3:39pm
Project:Plupload integration
Version:7.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

Login or register to post comments

Comments

#1

Posted by Dave Reid on August 5, 2011 at 3:40pm
Status:active» needs review
AttachmentSize
1240654-plupload-namespace-file-uri-to-object.patch 2.91 KB

#2

Posted by greggles on August 9, 2011 at 3:26pm

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

#3

Posted by robertgarrigos on September 18, 2011 at 11:06am
Status:needs review» reviewed & tested by the community

Applied this patch with success. Everything works fine.

#4

Posted by greenwe on October 12, 2011 at 8:11pm

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

#5

Posted by gmclelland on November 2, 2011 at 10:05pm

#1 patch worked for me.

#6

Posted by ergophobe on November 10, 2011 at 8:02pm

Patch worked for me, but on

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

Both downloaded 10 Nov 2011.

#7

Posted by nmpribeiro on December 12, 2011 at 7:23pm

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

#8

Posted by slashrsm on February 18, 2012 at 5:22pm
Status:reviewed & tested by the community» fixed

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

Thanks!

#9

Posted by System Message on March 3, 2012 at 5:30pm
Status:fixed» closed (fixed)

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

nobody click here