Added encodeURIComponent() on the file path url

base64_encode() produces "+" that php will treat as whitespace when decoded into $_GET, then base64_decode() fails. i encounterd this when file names contains multibyte characters.
pydio · Jun 3, 2014 · fb909bb · fb909bb
1 parent 102018e
commit fb909bb
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/core/src/plugins/editor.soundmanager/class.SMPlayer.js b/core/src/plugins/editor.soundmanager/class.SMPlayer.js
@@ -123,7 +123,7 @@ function hookToFilesList(){
         $A(fList.getItems()).each(function(row){
             if(!row.ajxpNode || (row.ajxpNode.getAjxpMime() != "mp3" && row.ajxpNode.getAjxpMime() != "wav")) return;
             addVolumeButton();
-            var url = ajxpBootstrap.parameters.get('ajxpServerAccess')+'&get_action=audio_proxy&file=base64encoded:'+base64_encode(row.ajxpNode.getPath())+ '&fake=extension.'+row.ajxpNode.getAjxpMime();
+            var url = ajxpBootstrap.parameters.get('ajxpServerAccess')+'&get_action=audio_proxy&file=base64encoded:'+encodeURIComponent(base64_encode(row.ajxpNode.getPath()))+ '&fake=extension.'+row.ajxpNode.getAjxpMime();
             var player = new Element("div", {className:"ui360 ui360-micro"}).update(new Element("a", {href:url}).update(""));
             row.down("span#ajxp_label").setStyle({backgroundImage:'none'}).insert({top:player});
             threeSixtyPlayer.config.items = [player];
@@ -246,7 +246,7 @@ soundManager.setup({\n\
             return im;
         }
         addVolumeButton();
-        var url = ajxpBootstrap.parameters.get('ajxpServerAccess')+'&get_action=audio_proxy&file=base64encoded:'+base64_encode(ajxpNode.getPath());
+        var url = ajxpBootstrap.parameters.get('ajxpServerAccess')+'&get_action=audio_proxy&file=base64encoded:'+encodeURIComponent(base64_encode(ajxpNode.getPath()));
         if(rich){
             url += '&rich_preview=true&fake=extension.'+ajxpNode.getAjxpMime();
         }else{
@@ -318,4 +318,4 @@ soundManager.setup({\n\
 
     }
 
-});
+});