Merge pull request #6677 from Skrypt/issue/#6512

Fixing multiple Media Library:clone

Fixes #6512
This commit is contained in:
Sébastien Ros
2016-03-31 12:45:57 -07:00

View File

@@ -213,8 +213,7 @@ namespace Orchard.MediaLibrary.Controllers {
}
return Json(true);
}
catch(Exception e) {
} catch (Exception e) {
Logger.Error(e, "Could not delete media items.");
return Json(false);
}
@@ -228,11 +227,11 @@ namespace Orchard.MediaLibrary.Controllers {
try {
var media = Services.ContentManager.Get(mediaItemId).As<MediaPart>();
if(!_mediaLibraryService.CanManageMediaFolder(media.FolderPath)) {
if (!_mediaLibraryService.CanManageMediaFolder(media.FolderPath)) {
return new HttpUnauthorizedResult();
}
var newFileName = Path.GetFileNameWithoutExtension(media.FileName) + " Copy" + Path.GetExtension(media.FileName);
var newFileName = _mediaLibraryService.GetUniqueFilename(media.FolderPath, media.FileName);
_mediaLibraryService.CopyFile(media.FolderPath, media.FileName, media.FolderPath, newFileName);
@@ -241,13 +240,14 @@ namespace Orchard.MediaLibrary.Controllers {
var clonedTitlePart = clonedContentItem.As<TitlePart>();
clonedMediaPart.FileName = newFileName;
clonedMediaPart.FolderPath = media.FolderPath;
clonedMediaPart.MimeType = media.MimeType;
clonedTitlePart.Title = clonedTitlePart.Title + " Copy";
Services.ContentManager.Create(clonedContentItem);
Services.ContentManager.Publish(clonedContentItem);
return Json(true);
}
catch (Exception e) {
} catch (Exception e) {
Logger.Error(e, "Could not clone media item.");
return Json(false);
}
@@ -255,7 +255,7 @@ namespace Orchard.MediaLibrary.Controllers {
private FolderHierarchy GetFolderHierarchy(IMediaFolder root) {
Argument.ThrowIfNull(root, "root");
return new FolderHierarchy(root) {Children = _mediaLibraryService.GetMediaFolders(root.MediaPath).Select(GetFolderHierarchy)};
return new FolderHierarchy(root) { Children = _mediaLibraryService.GetMediaFolders(root.MediaPath).Select(GetFolderHierarchy) };
}
}
}