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

Removed unnecessary check for "/".

Browse Source 
The DisplayAlias of a content item will not be set to "" or "/" for the item to become the homepage, so there's no need to check for the display alias to be "/".
This commit is contained in:
Sipke Schoorstra
2015-07-26 17:16:41 +01:00
parent 8aa2a9f070
commit 805de7d563
1 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/Orchard.Web/Modules/Orchard.Autoroute/Handlers/AutoroutePartHandler.cs
View File

@@ -91,13 +91,11 @@ namespace Orchard.Autoroute.Handlers {
} }
// Check for permalink conflict, unless we are trying to set the home page. // Check for permalink conflict, unless we are trying to set the home page.
if (part.DisplayAlias != "/") { var previous = part.Path;
var previous = part.Path; if (!_autorouteService.Value.ProcessPath(part))
if (!_autorouteService.Value.ProcessPath(part)) _orchardServices.Notifier.Warning(
_orchardServices.Notifier.Warning( T("Permalinks in conflict. \"{0}\" is already set for a previously created {2} so now it has the slug \"{1}\"",
T("Permalinks in conflict. \"{0}\" is already set for a previously created {2} so now it has the slug \"{1}\"", previous, part.Path, part.ContentItem.ContentType));
previous, part.Path, part.ContentItem.ContentType));
}
} }
void RemoveAlias(AutoroutePart part) { void RemoveAlias(AutoroutePart part) {