lsm/Orchard
1
0
Fork 0
mirror of https://github.com/OrchardCMS/Orchard.git synced 2026-02-09 09:16:41 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Lists: Adding error handling to item selection

Browse Source
This commit is contained in:
Benedek Farkas
2025-10-02 16:08:16 +02:00
parent c47b55ade0
commit 2046bd2fbd
1 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/Orchard.Web/Modules/Orchard.Lists/Controllers/AdminController.cs
View File

@@ -301,8 +301,17 @@ namespace Orchard.Lists.Controllers {
[HttpPost]
public ActionResult Insert(int containerId, int itemId, PagerParameters pagerParameters) {
var container = _containerService.Get(containerId, VersionOptions.Latest);
ActionResult redirectToList() =>
RedirectToAction("List", new { containerId, page = pagerParameters.Page, pageSize = pagerParameters.PageSize });
var item = _contentManager.Get(itemId, VersionOptions.Latest, new QueryHints().ExpandParts<CommonPart, ContainablePart>());
if (item == null || !item.Has<ContainablePart>()) {
_services.Notifier.Error(T("Item not found or doesn't have ContainablePart."));
return redirectToList();
}
var container = _containerService.Get(containerId, VersionOptions.Latest);
var commonPart = item.As<CommonPart>();
var previousItemContainer = commonPart.Container;
var itemMetadata = _contentManager.GetItemMetadata(item);
@@ -328,7 +337,8 @@ namespace Orchard.Lists.Controllers {
_containerService.MoveItem(item.As<ContainablePart>(), container, position);
_services.Notifier.Information(message);
return RedirectToAction("List", new { containerId, page = pagerParameters.Page, pageSize = pagerParameters.PageSize });
return redirectToList();
}
[HttpPost]