From bed8310f7e2a11614b2a0f28dacd4fe2d9f0bd6e Mon Sep 17 00:00:00 2001
From: starfighter83 <unknown@kiln.example.com>
Date: Wed, 24 Nov 2010 14:34:59 +0000
Subject: [PATCH] Fixes #20562: Please make sure that you do not have 2
 templates with the same alias

[TFS Changeset #80539]
---
 .../cms/businesslogic/template/Template.cs    | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/umbraco/cms/businesslogic/template/Template.cs b/umbraco/cms/businesslogic/template/Template.cs
index 6e26b6debc..be59908b0b 100644
--- a/umbraco/cms/businesslogic/template/Template.cs
+++ b/umbraco/cms/businesslogic/template/Template.cs
@@ -295,6 +295,10 @@ namespace umbraco.cms.businesslogic.template
 
         public static Template MakeNew(string Name, BusinessLogic.User u, Template master)
         {
+            //ensure unique alias 
+            if (GetByAlias(Name) != null)
+                Name = EnsureUniqueAlias(Name, 1);
+
             Template t = MakeNew(Name, u);
             t.MasterTemplate = master.Id;
             t.Design = "";
@@ -317,6 +321,10 @@ namespace umbraco.cms.businesslogic.template
 
         public static Template MakeNew(string Name, BusinessLogic.User u)
         {
+            //ensure unique alias 
+            if (GetByAlias(Name) != null)
+                Name = EnsureUniqueAlias(Name, 1);
+
             // CMSNode MakeNew(int parentId, Guid objectType, int userId, int level, string text, Guid uniqueID)
             CMSNode n = CMSNode.MakeNew(-1, _objectType, u.Id, 1, Name, Guid.NewGuid());
             Name = Name.Replace("/", ".").Replace("\\", "");
@@ -338,6 +346,17 @@ namespace umbraco.cms.businesslogic.template
             return t;
         }
 
+        private static string EnsureUniqueAlias(string alias, int attempts)
+        {
+            if (GetByAlias(alias + attempts.ToString()) == null)
+                return alias + attempts.ToString();
+            else
+            {
+                attempts++;
+                return EnsureUniqueAlias(alias, attempts);
+            }
+        }
+
         public static Template GetByAlias(string Alias)
         {
             try