Fix named transformations with spaces

Author: eyalktCloudinary

lib-es5/utils/index.js

@@ -874,7 +874,7 @@ function url(public_id) {
   var prefix = unsigned_url_prefix(public_id, cloud_name, private_cdn, cdn_subdomain, secure_cdn_subdomain, cname, secure, secure_distribution);
   var resultUrl = [prefix, resource_type, type, signature, transformation, version, public_id].filter(function (part) {
     return part != null && part !== '';
-  }).join('/');
+  }).join('/').replace(' ', '%20');
   if (sign_url && !isEmpty(auth_token)) {
     auth_token.url = urlParse(resultUrl).path;
     var token = generate_token(auth_token);

lib/uploader.js

@@ -336,7 +336,7 @@ function parseResult(buffer, res) {
   let result = '';
   try {
     result = JSON.parse(buffer);
-    if(result.error && !result.error.name) {
+    if (result.error && !result.error.name) {
       result.error.name = "Error";
     }
   } catch (jsonError) {

lib/utils/index.js

@@ -872,7 +872,7 @@ function url(public_id, options = {}) {
   );
   let resultUrl = [prefix, resource_type, type, signature, transformation, version, public_id].filter(function (part) {
     return (part != null) && part !== '';
-  }).join('/');
+  }).join('/').replace(' ', '%20');
   if (sign_url && !isEmpty(auth_token)) {
     auth_token.url = urlParse(resultUrl).path;
     let token = generate_token(auth_token);

test/utils_spec.js

@@ -527,6 +527,11 @@ describe("utils", function () {
           transformation: ["blip", "blop"],
         }, `http://res.cloudinary.com/${cloud_name}/image/upload/t_blip.blop/test`, {});
       });
+      it("should support named transformations with spaces", function () {
+        test_cloudinary_url("test", {
+          transformation: "blip blop",
+        }, `http://res.cloudinary.com/${cloud_name}/image/upload/t_blip%20blop/test`, {});
+      });
       it("should support base transformation", function () {
         test_cloudinary_url("test", {
           transformation: {