From e667cdbc66041ae8d8a3196d81e36da07604522e Mon Sep 17 00:00:00 2001
From: Shannon <sdeminick@gmail.com>
Date: Wed, 4 Jun 2014 20:32:51 +1000
Subject: [PATCH] Removes UI for adding a segment value for a configured
 segment since it doesn't make sense, a configurable segment is always just a
 boolean match.

---
 .../views/dashboard/settings/segmentdashboard.html |  6 +-----
 src/Umbraco.Web/Models/Segments/Segment.cs         | 14 --------------
 2 files changed, 1 insertion(+), 19 deletions(-)

diff --git a/src/Umbraco.Web.UI.Client/src/views/dashboard/settings/segmentdashboard.html b/src/Umbraco.Web.UI.Client/src/views/dashboard/settings/segmentdashboard.html
index 7cba5633ad..36b0942be9 100644
--- a/src/Umbraco.Web.UI.Client/src/views/dashboard/settings/segmentdashboard.html
+++ b/src/Umbraco.Web.UI.Client/src/views/dashboard/settings/segmentdashboard.html
@@ -75,11 +75,7 @@
                             <input type="text" name="key" ng-model="$parent.item.key" required />
                             <span class="help-inline" val-msg-for="key" val-toggle-msg="required">Required</span>
                         </umb-control-group>
-
-                        <umb-control-group label="Segment Value" alias="value" description="The optional value to store for this segment match">
-                            <input type="text" name="value" ng-model="$parent.item.value" />
-                        </umb-control-group>
-
+                        
                         <umb-control-group label="Persist segment" alias="value" description="This might be useful if you want to track an assigned segment">
                             <input type="checkbox" name="persist" ng-model="$parent.item.persist" title=" For example, if it's a referral provider a segment might be set when coming from 'mysite.com' and you want to know about that later on when the segment no longer exists in the request." />
                         </umb-control-group>
diff --git a/src/Umbraco.Web/Models/Segments/Segment.cs b/src/Umbraco.Web/Models/Segments/Segment.cs
index b0cacc6a6a..ae378eae7b 100644
--- a/src/Umbraco.Web/Models/Segments/Segment.cs
+++ b/src/Umbraco.Web/Models/Segments/Segment.cs
@@ -22,14 +22,12 @@ namespace Umbraco.Web.Models.Segments
 
         public Segment(string key, object value)
         {
-            //ProviderTypeHashCode = providerType.GetHashCode();
             Key = key;
             Value = value;
         }
 
         public Segment(string key, object value, bool persist)
         {
-            //ProviderTypeHashCode = providerType.GetHashCode();
             Key = key;
             Value = value;
             Persist = persist;
@@ -37,7 +35,6 @@ namespace Umbraco.Web.Models.Segments
 
         internal Segment(string key, object value, bool persist, int? slidingDays)
         {
-            //ProviderTypeHashCode = providerType.GetHashCode();
             Key = key;
             Value = value;
             Persist = persist;
@@ -46,23 +43,12 @@ namespace Umbraco.Web.Models.Segments
 
         internal Segment(string key, object value, bool persist, DateTime? absoluteExpiry)
         {
-            //ProviderTypeHashCode = providerType.GetHashCode();
             Key = key;
             Value = value;
             Persist = persist;
             AbsoluteExpiry = absoluteExpiry;
         }
 
-        ///// <summary>
-        ///// The md5 of the type of provider that created this segment
-        ///// </summary>
-        ///// <remarks>
-        ///// it will take up 
-        ///// too much space. 
-        ///// </remarks>
-        //[DataMember(Name = "h", IsRequired = true)]
-        //public string ProviderTypeMd5Hash { get; set; }
-
         /// <summary>
         /// The name of the segment
         /// </summary>