Merge pull request #2 from tundebabzy/issue_1

Issue 1

cloudinary_storage/storage.py

@@ -57,6 +57,7 @@ def _upload(self, name, content):
         return cloudinary.uploader.upload(content, **options)
 
     def _save(self, name, content):
+        name = self._normalise_name(name)
         name = self._prepend_prefix(name)
         content = UploadedFile(content, name)
         response = self._upload(name, content)
@@ -117,6 +118,9 @@ def listdir(self, path):
                 files.append(resource_tail)
         return list(directories), files
 
+    def _normalise_name(self, name):
+        return name.replace('\\', '/')
+
 
 class RawMediaCloudinaryStorage(MediaCloudinaryStorage):
     RESOURCE_TYPE = RESOURCE_TYPES['RAW']

tests/tests/test_storage.py

@@ -7,7 +7,7 @@
 from django.conf import settings
 
 from cloudinary_storage.storage import (MediaCloudinaryStorage, ManifestCloudinaryStorage, StaticCloudinaryStorage,
-    StaticHashedCloudinaryStorage)
+                                        StaticHashedCloudinaryStorage)
 from cloudinary_storage import app_settings
 from tests.tests.test_helpers import get_random_name, import_mock
 
@@ -25,8 +25,8 @@ def setUpClass(cls):
         cls.file_name, cls.file = cls.upload_file()
 
     @classmethod
-    def upload_file(cls, prefix=''):
-        file_name = prefix + get_random_name()
+    def upload_file(cls, prefix='', directory_name=''):
+        file_name = prefix + directory_name + get_random_name()
         content = ContentFile(cls.file_content)
         file_name = cls.storage.save(file_name, content)
         return file_name, content
@@ -106,6 +106,14 @@ def test_list_dir(self):
         finally:
             self.storage.delete(file_2_name)
 
+    def test_file_with_windows_path_uploaded_and_exists(self):
+        file_name, file = self.upload_file(directory_name='windows\\styled\\path\\')
+        try:
+            self.assertTrue(self.storage.exists(file_name))
+        finally:
+            self.storage.delete(file_name)
+
+
     @classmethod
     def tearDownClass(cls):
         super(CloudinaryMediaStorageTests, cls).tearDownClass()