diff --git a/src/Orchard.Azure.Tests/Storage/AzureBlobStorageProviderTests.cs b/src/Orchard.Azure.Tests/Storage/AzureBlobStorageProviderTests.cs index 8c43b9a15..6d7053e26 100644 --- a/src/Orchard.Azure.Tests/Storage/AzureBlobStorageProviderTests.cs +++ b/src/Orchard.Azure.Tests/Storage/AzureBlobStorageProviderTests.cs @@ -4,6 +4,7 @@ using NUnit.Framework; using Orchard.Azure.Storage; using Microsoft.WindowsAzure; using System.Linq; +using Orchard.Environment.Configuration; namespace Orchard.Azure.Tests.Storage { [TestFixture] @@ -15,7 +16,7 @@ namespace Orchard.Azure.Tests.Storage { CloudStorageAccount devAccount; CloudStorageAccount.TryParse("UseDevelopmentStorage=true", out devAccount); - _azureBlobStorageProvider = new AzureBlobStorageProvider("default", devAccount); + _azureBlobStorageProvider = new AzureBlobStorageProvider(new ShellSettings { Name = "default" }, devAccount); } [SetUp] diff --git a/src/Orchard.Azure/AzureFileSystem.cs b/src/Orchard.Azure/AzureFileSystem.cs index 4f66b1bc6..97d7eab96 100644 --- a/src/Orchard.Azure/AzureFileSystem.cs +++ b/src/Orchard.Azure/AzureFileSystem.cs @@ -11,19 +11,19 @@ namespace Orchard.Azure { public string ContainerName { get; protected set; } private readonly CloudStorageAccount _storageAccount; - private readonly string _shellName; + private readonly string _root; public CloudBlobClient BlobClient { get; private set; } public CloudBlobContainer Container { get; private set; } - public AzureFileSystem(string containerName, string shellName, bool isPrivate) - : this(containerName, shellName, isPrivate, CloudStorageAccount.FromConfigurationSetting("DataConnectionString")) { + public AzureFileSystem(string containerName, string root, bool isPrivate) + : this(containerName, root, isPrivate, CloudStorageAccount.FromConfigurationSetting("DataConnectionString")) { } - public AzureFileSystem(string containerName, string shellName, bool isPrivate, CloudStorageAccount storageAccount) { + public AzureFileSystem(string containerName, string root, bool isPrivate, CloudStorageAccount storageAccount) { // Setup the connection to custom storage accountm, e.g. Development Storage _storageAccount = storageAccount; ContainerName = containerName; - _shellName = shellName; + _root = String.IsNullOrEmpty(root) || root == "/" ? String.Empty : root + "/"; BlobClient = _storageAccount.CreateCloudBlobClient(); // Get and create the container if it does not exist @@ -47,20 +47,20 @@ namespace Orchard.Azure { public IStorageFile GetFile(string path) { EnsurePathIsRelative(path); - path = String.Concat(_shellName, "/", path); + path = String.Concat(_root, path); Container.EnsureBlobExists(path); return new AzureBlobFileStorage(Container.GetBlockBlobReference(path)); } public bool FileExists(string path) { - path = String.Concat(_shellName, "/", path); + path = String.Concat(_root, path); return Container.BlobExists(path); } public IEnumerable ListFiles(string path) { EnsurePathIsRelative(path); - string prefix = String.Concat(Container.Name, "/", _shellName, "/", path); + string prefix = String.Concat(Container.Name, "/", _root, path); if ( !prefix.EndsWith("/") ) prefix += "/"; @@ -71,7 +71,7 @@ namespace Orchard.Azure { public IEnumerable ListFolders(string path) { EnsurePathIsRelative(path); - path = String.Concat(_shellName, "/", path); + path = String.Concat(_root, path); if ( !Container.DirectoryExists(path) ) { try { @@ -91,7 +91,7 @@ namespace Orchard.Azure { public void CreateFolder(string path) { EnsurePathIsRelative(path); - path = String.Concat(_shellName, "/", path); + path = String.Concat(_root, path); Container.EnsureDirectoryDoesNotExist(path); Container.GetDirectoryReference(path); @@ -99,7 +99,7 @@ namespace Orchard.Azure { public void DeleteFolder(string path) { EnsurePathIsRelative(path); - path = String.Concat(_shellName, "/", path); + path = String.Concat(_root, path); Container.EnsureDirectoryExists(path); foreach ( var blob in Container.GetDirectoryReference(path).ListBlobs() ) { @@ -107,7 +107,7 @@ namespace Orchard.Azure { ( (CloudBlob)blob ).Delete(); if ( blob is CloudBlobDirectory ) - DeleteFolder(blob.Uri.ToString().Substring(Container.Uri.ToString().Length + 2 + _shellName.Length)); + DeleteFolder(blob.Uri.ToString().Substring(Container.Uri.ToString().Length + 1 + _root.Length)); } } @@ -122,7 +122,7 @@ namespace Orchard.Azure { if ( !newPath.EndsWith("/") ) newPath += "/"; - foreach ( var blob in Container.GetDirectoryReference(_shellName + "/" + path).ListBlobs() ) { + foreach ( var blob in Container.GetDirectoryReference(_root + path).ListBlobs() ) { if ( blob is CloudBlob ) { string filename = Path.GetFileName(blob.Uri.ToString()); string source = String.Concat(path, filename); @@ -142,7 +142,7 @@ namespace Orchard.Azure { public void DeleteFile(string path) { EnsurePathIsRelative(path); - path = String.Concat(_shellName, "/", path); + path = String.Concat(_root, path); Container.EnsureBlobExists(path); var blob = Container.GetBlockBlobReference(path); @@ -151,10 +151,10 @@ namespace Orchard.Azure { public void RenameFile(string path, string newPath) { EnsurePathIsRelative(path); - path = String.Concat(_shellName, "/", path); + path = String.Concat(_root, path); EnsurePathIsRelative(newPath); - newPath = String.Concat(_shellName, "/", newPath); + newPath = String.Concat(_root, newPath); Container.EnsureBlobExists(path); Container.EnsureBlobDoesNotExist(newPath); @@ -167,7 +167,7 @@ namespace Orchard.Azure { public IStorageFile CreateFile(string path) { EnsurePathIsRelative(path); - path = String.Concat(_shellName, "/", path); + path = String.Concat(_root, path); if ( Container.BlobExists(path) ) { throw new ArgumentException("File " + path + " already exists"); @@ -180,7 +180,7 @@ namespace Orchard.Azure { public string GetPublicUrl(string path) { EnsurePathIsRelative(path); - path = String.Concat(_shellName, "/", path); + path = String.Concat(_root, path); Container.EnsureBlobExists(path); return Container.GetBlockBlobReference(path).Uri.ToString(); } diff --git a/src/Orchard.Azure/Environment/Configuration/AzureAppDataFolder.cs b/src/Orchard.Azure/Environment/Configuration/AzureAppDataFolder.cs index e108a5f05..540ba18ec 100644 --- a/src/Orchard.Azure/Environment/Configuration/AzureAppDataFolder.cs +++ b/src/Orchard.Azure/Environment/Configuration/AzureAppDataFolder.cs @@ -9,8 +9,8 @@ namespace Orchard.Azure.Environment.Configuration { private readonly AzureFileSystem _fs; - public AzureAppDataFolder(string shellName) { - _fs = new AzureFileSystem("appdata", shellName, true); + public AzureAppDataFolder() { + _fs = new AzureFileSystem("appdata", null, true); } public void CreateFile(string path, string content) { diff --git a/src/Orchard.Azure/Orchard.Azure.Web/Config/Host.config b/src/Orchard.Azure/Orchard.Azure.Web/Config/Host.config index b56b3523b..f18679d61 100644 --- a/src/Orchard.Azure/Orchard.Azure.Web/Config/Host.config +++ b/src/Orchard.Azure/Orchard.Azure.Web/Config/Host.config @@ -16,17 +16,11 @@ - - - - - - diff --git a/src/Orchard.Azure/Storage/AzureBlobStorageProvider.cs b/src/Orchard.Azure/Storage/AzureBlobStorageProvider.cs index 938a16602..39c805c4b 100644 --- a/src/Orchard.Azure/Storage/AzureBlobStorageProvider.cs +++ b/src/Orchard.Azure/Storage/AzureBlobStorageProvider.cs @@ -1,15 +1,16 @@ using Microsoft.WindowsAzure; +using Orchard.Environment.Configuration; using Orchard.Storage; namespace Orchard.Azure.Storage { public class AzureBlobStorageProvider : AzureFileSystem, IStorageProvider { - public AzureBlobStorageProvider(string shellName) - : this(shellName, CloudStorageAccount.FromConfigurationSetting("DataConnectionString")) { + public AzureBlobStorageProvider(ShellSettings shellSettings) + : this(shellSettings, CloudStorageAccount.FromConfigurationSetting("DataConnectionString")) { } - public AzureBlobStorageProvider(string shellName, CloudStorageAccount storageAccount) : base("media", shellName, false, storageAccount) { } + public AzureBlobStorageProvider(ShellSettings shellSettings, CloudStorageAccount storageAccount) : base("media", shellSettings.Name, false, storageAccount) { } } }