From 06c829eb27e30e10a35a18cef5a56c121edcdb84 Mon Sep 17 00:00:00 2001 From: Claus Date: Wed, 1 Feb 2017 14:03:03 +0100 Subject: [PATCH] initializing stringbuilder with text instead of appending after init. trimming end instead of both beginning and end, since we know whats in the beginning. --- src/Umbraco.Web/Trees/UrlHelperExtensions.cs | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/src/Umbraco.Web/Trees/UrlHelperExtensions.cs b/src/Umbraco.Web/Trees/UrlHelperExtensions.cs index acb10e31c5..4b94f548ca 100644 --- a/src/Umbraco.Web/Trees/UrlHelperExtensions.cs +++ b/src/Umbraco.Web/Trees/UrlHelperExtensions.cs @@ -47,25 +47,21 @@ namespace Umbraco.Web.Trees virtualPath = virtualPath.Replace('\\', '/'); //-1 is the default root id for trees - var sb = new StringBuilder(); - sb.Append("-1"); + var sb = new StringBuilder("-1"); //split the virtual path and iterate through it - string[] pathPaths = virtualPath.Split('/'); + var pathPaths = virtualPath.Split('/'); - for (int p = 0; p < pathPaths.Length; p++) + for (var p = 0; p < pathPaths.Length; p++) { var path = HttpUtility.UrlEncode(string.Join("/", pathPaths.Take(p + 1))); if (string.IsNullOrEmpty(path) == false) { sb.Append(","); - sb.Append(path); + sb.Append(path); } } - - return sb.ToString().Trim(","); - + return sb.ToString().TrimEnd(","); } - } } \ No newline at end of file