lsm/Orchard
1
0
Fork 0
mirror of https://github.com/OrchardCMS/Orchard.git synced 2025-10-26 20:16:15 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Removes useless and case sensitive check (#8352)

Browse Source 
That check was useless because we use a int.TryParse. Morevoer queryString.AllKeys.Contains() is case sensitive.
This commit is contained in:
Hermes Sbicego
2020-04-23 19:11:07 +02:00
committed by GitHub
parent 914b5f8391
commit 8a327cf887
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/Orchard.Web/Modules/Orchard.CustomForms/Controllers/ItemController.cs
View File

@@ -72,13 +72,10 @@ namespace Orchard.CustomForms.Controllers {
var customForm = form.As<CustomFormPart>(); var customForm = form.As<CustomFormPart>();
// Retrieve the id of the content to edit // Retrieve the id of the content to edit
int contentId = 0;
var queryString = Services.WorkContext.HttpContext.Request.QueryString; var queryString = Services.WorkContext.HttpContext.Request.QueryString;
if (queryString.AllKeys.Contains("contentId")) { int.TryParse(queryString["contentId"], out int contentId);
int.TryParse(queryString["contentId"], out contentId);
}
ContentItem contentItem; ContentItem contentItem;
if (contentId > 0) { if (contentId > 0) {
contentItem = _contentManager.Get(contentId); contentItem = _contentManager.Get(contentId);