Skip to content

Commit ad59b78

Browse files
hishamcogvkries
andauthored
Display the delete button on edit type & part alongside with list page (#18240)
Co-authored-by: Georg von Kries <[email protected]>
1 parent 14e055c commit ad59b78

File tree

3 files changed

+18
-6
lines changed

3 files changed

+18
-6
lines changed

src/OrchardCore.Modules/OrchardCore.ContentTypes/Controllers/AdminController.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -545,7 +545,6 @@ public async Task<ActionResult> EditPart(string id)
545545
}
546546

547547
[HttpPost, ActionName("EditPart")]
548-
[FormValueRequired("submit.Save")]
549548
public async Task<ActionResult> EditPartPOST(string id, string[] orderedFieldNames)
550549
{
551550
if (!await _authorizationService.AuthorizeAsync(User, ContentTypesPermissions.EditContentTypes))

src/OrchardCore.Modules/OrchardCore.ContentTypes/Views/Admin/Edit.cshtml

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@
5555

5656
<zone Name="Title"><h1>@RenderTitleSegments(T["Edit Content Type - {0}", Model.TypeDefinition.DisplayName])</h1></zone>
5757

58-
<form asp-action="Edit">
58+
<form method="post">
5959
@Html.ValidationSummary()
6060

6161
<input type="hidden" asp-for="Name" />
@@ -184,16 +184,28 @@
184184
</div>
185185

186186
<div class="mb-3">
187-
<a class="btn btn-secondary" asp-route-action="List">@T["Cancel"]</a>
188187
<div class="btn-group">
189-
<button class="btn btn-success save" type="submit" name="submit.Save" value="Save">@T["Save"]</button>
188+
<button asp-route-action="Edit" class="btn btn-success save" type="submit" name="submit.Save" value="Save">@T["Save"]</button>
190189
<button type="button" class="btn btn-success dropdown-toggle dropdown-toggle-split" data-reference="parent" data-bs-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
191190
<span class="visually-hidden">@T["Toggle Dropdown"]</span>
192191
</button>
193192
<div class="dropdown-menu">
194-
<button class="dropdown-item save-continue" type="submit" name="submit.Save" value="SaveAndContinue">@T["and continue"]</button>
193+
<button asp-route-action="Edit" class="dropdown-item save-continue" type="submit" name="submit.Save" value="SaveAndContinue">@T["and continue"]</button>
195194
</div>
196195
</div>
196+
@if (contentSettings.IsSystemDefined)
197+
{
198+
<div class="d-inline" data-bs-toggle="tooltip" data-bs-title="@T["System-defined types are integral components of the system and cannot be removed."]">
199+
<button type="button" class="btn btn-danger" disabled>
200+
@T["Delete"]
201+
</button>
202+
</div>
203+
}
204+
else
205+
{
206+
<a role="button" asp-route-action="Delete" asp-route-id="@Model.Name" class="btn btn-danger" data-url-af="RemoveUrl UnsafeUrl" data-title="@T["Delete"]" data-message="@T["Are you sure you want to delete this content type?"]" data-ok-text="@T["Ok"]" data-cancel-text="@T["Cancel"]">@T["Delete"]</a>
207+
}
208+
<a class="btn btn-secondary cancel" role="button" asp-route-action="List">@T["Cancel"]</a>
197209
</div>
198210
</form>
199211
<script asp-name="Sortable" at="Foot"></script>

src/OrchardCore.Modules/OrchardCore.ContentTypes/Views/Admin/EditPart.cshtml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,9 @@
7676
}
7777
}
7878
<div class="mb-3">
79-
<a class="btn btn-secondary" asp-route-action="ListParts">@T["Cancel"]</a>
8079
<button class="btn btn-primary save" type="submit" name="submit.Save" value="Save">@T["Save"]</button>
80+
<a role="button" asp-route-action="DeletePart" asp-route-id="@Model.Name" class="btn btn-danger" data-url-af="RemoveUrl UnsafeUrl" data-title="@T["Delete"]" data-message="@T["Are you sure you want to delete this content part?"]" data-ok-text="@T["Ok"]" data-cancel-text="@T["Cancel"]">@T["Delete"]</a>
81+
<a class="btn btn-secondary cancel" role="button" asp-route-action="ListParts">@T["Cancel"]</a>
8182
</div>
8283
</form>
8384
<script asp-name="Sortable" at="Foot"></script>

0 commit comments

Comments
 (0)