The return value from the function DrupalStorageStreamWrapper::getExternalUrl shold be a web accessible URL, according to the documentation here. It currently only returns the Drupal path instead. This results in URLs to files looking like this:


I am using Drupal 7.8 and Storage API 7.x-1.2. This should be a small fix, and I will supply a patch shortly.

#1 external_url.patch353 bytesmwallenberg
Members fund testing for the Drupal project. Drupal Association Learn more


mwallenberg’s picture

Status: Active » Needs review
353 bytes

Only a small fix is needed, in storage_core_bridge.module, line 620. The following diff shows the difference:

@@ -617,7 +617,7 @@
-      return $storage->serveURL();
+      return $storage->serveURL(TRUE);
     catch (StorageException $e) {
       return FALSE;

I've made an attempt at a patch file, though I don't think it can be applied directly. As I said, it's a small change - just supply TRUE as the argument to serveUrl.

jbrown’s picture

Assigned: mwallenberg » Unassigned
Status: Needs review » Fixed

Thanks, but this is already fixed in 7.x-1.2:

mwallenberg’s picture


Indeed it is. I'll just update my 7.x-1.1 version to 7.x-1.2, and I'll be fine. :P

Status: Fixed » Closed (fixed)

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