From d7edef9bb889a8016ab1cbe67bb81d5136fbb9e4 Mon Sep 17 00:00:00 2001 From: Bertrand Le Roy Date: Wed, 3 Dec 2014 22:02:18 -0800 Subject: [PATCH] Use TrimStart rather than a loop to remove leading slashes in autoroute driver. --- .../Orchard.Autoroute/Drivers/AutoroutePartDriver.cs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Orchard.Web/Modules/Orchard.Autoroute/Drivers/AutoroutePartDriver.cs b/src/Orchard.Web/Modules/Orchard.Autoroute/Drivers/AutoroutePartDriver.cs index 59052e4fb..ca4bb47b3 100644 --- a/src/Orchard.Web/Modules/Orchard.Autoroute/Drivers/AutoroutePartDriver.cs +++ b/src/Orchard.Web/Modules/Orchard.Autoroute/Drivers/AutoroutePartDriver.cs @@ -81,9 +81,9 @@ namespace Orchard.Autoroute.Drivers { var previous = part.DisplayAlias; if (updater != null && updater.TryUpdateModel(viewModel, Prefix, null, null)) { - // remove any trailing slash in the permalink - while(!string.IsNullOrEmpty(viewModel.CurrentUrl) && viewModel.CurrentUrl.StartsWith("/")) { - viewModel.CurrentUrl = viewModel.CurrentUrl.Substring(1); + // remove any leading slash in the permalink + if (viewModel.CurrentUrl != null) { + viewModel.CurrentUrl = viewModel.CurrentUrl.TrimStart('/'); } part.DisplayAlias = viewModel.CurrentUrl;