Custom enum description attribute property names

readme.md

@@ -163,10 +163,10 @@ Calling `ToString` directly on enum members usually results in less than ideal o
 ```C#
 public enum EnumUnderTest
 {
-    [Description("Custom description")]
-    MemberWithDescriptionAttribute,
-    MemberWithoutDescriptionAttribute,
-    ALLCAPITALS
+[Description("Custom description")]
+MemberWithDescriptionAttribute,
+MemberWithoutDescriptionAttribute,
+ALLCAPITALS
 }
 ```
 
@@ -186,6 +186,10 @@ EnumUnderTest.MemberWithoutDescriptionAttribute.Humanize().Transform(To.TitleCas
 You are not limited to `DescriptionAttribute` for custom description. Any attribute applied on enum members with a `string Description` property counts.
 This is to help with platforms with missing `DescriptionAttribute` and also for allowing subclasses of the `DescriptionAttribute`.
 
+You can even configure the name of the property of attibute to use as description.
+
+`Configurator.EnumDescriptionPropertyNames[typeof(EnumUnderTest)] = "Info"`
+
 Hopefully this will help avoid littering enums with unnecessary attributes!
 
 ###<a id="dehumanize-enums">Dehumanize Enums</a>

release_notes.md

@@ -1,4 +1,5 @@
 ###In Development
+  - [#257](https://github.com/MehdiK/Humanizer/pull/277): Added support for custom enum description attribute property names
 
 [Commits](https://github.com/MehdiK/Humanizer/compare/v1.26.1...master)
 

src/Humanizer.Tests/ApiApprover/PublicApiApprovalTest.approve_public_api.approved.txt

@@ -75,6 +75,7 @@ public class Configurator
 {
     public Humanizer.Configuration.LocaliserRegistry<Humanizer.Localisation.CollectionFormatters.ICollectionFormatter> CollectionFormatters { get; }
     public Humanizer.DateTimeHumanizeStrategy.IDateTimeHumanizeStrategy DateTimeHumanizeStrategy { get; set; }
+    public System.Collections.Generic.IDictionary<System.Type, string> EnumDescriptionPropertyNames { get; }
     public Humanizer.Configuration.LocaliserRegistry<Humanizer.Localisation.Formatters.IFormatter> Formatters { get; }
     public Humanizer.Configuration.LocaliserRegistry<Humanizer.Localisation.NumberToWords.INumberToWordsConverter> NumberToWordsConverters { get; }
     public Humanizer.Configuration.LocaliserRegistry<Humanizer.Localisation.Ordinalizers.IOrdinalizer> Ordinalizers { get; }

src/Humanizer.Tests/EnumHumanizeWithCustomDescriptionPropertyNamesTests.cs

@@ -0,0 +1,31 @@
+using System;
+using Humanizer.Configuration;
+using Xunit;
+
+namespace Humanizer.Tests
+{
+    public class EnumHumanizeWithCustomDescriptionPropertyNamesTests : IDisposable
+    {
+        public EnumHumanizeWithCustomDescriptionPropertyNamesTests()
+        {
+            Configurator.EnumDescriptionPropertyNames[typeof (EnumUnderTest)] = "Info";
+        }
+
+        public void Dispose()
+        {
+            Configurator.EnumDescriptionPropertyNames.Remove(typeof (EnumUnderTest));
+        }
+
+        [Fact]
+        public void HonorsCustomPropertyAttribute()
+        {
+            Assert.Equal(EnumTestsResources.MemberWithCustomPropertyAttribute, EnumUnderTest.MemberWithCustomPropertyAttribute.Humanize());
+        }
+
+        [Fact]
+        public void CanHumanizeMembersWithoutDescriptionAttribute()
+        {
+            Assert.Equal(EnumTestsResources.MemberWithoutDescriptionAttributeSentence, EnumUnderTest.MemberWithoutDescriptionAttribute.Humanize());
+        }
+    }
+}

src/Humanizer.Tests/EnumUnderTest.cs

@@ -13,6 +13,8 @@ public enum EnumUnderTest
         MemberWithCustomDescriptionAttribute,
         [ImposterDescription(42)]
         MemberWithImposterDescriptionAttribute,
+        [CustomProperty(EnumTestsResources.MemberWithCustomPropertyAttribute)]
+        MemberWithCustomPropertyAttribute,
         MemberWithoutDescriptionAttribute,
         ALLCAPITALS
     }
@@ -23,6 +25,7 @@ public class EnumTestsResources
         public const string MemberWithDescriptionAttributeSubclass = "Description in Description subclass";
         public const string MemberWithCustomDescriptionAttribute = "Description in custom Description attribute";
         public const string MemberWithImposterDescriptionAttribute = "Member with imposter description attribute";
+        public const string MemberWithCustomPropertyAttribute = "Description in custom property attribute";
         public const string MemberWithoutDescriptionAttributeSentence = "Member without description attribute";
         public const string MemberWithoutDescriptionAttributeTitle = "Member Without Description Attribute";
         public const string MemberWithoutDescriptionAttributeLowerCase = "member without description attribute";
@@ -59,4 +62,14 @@ public override string Description
             get { return "Overridden " + base.Description; }
         }
     }
+
+    public class CustomPropertyAttribute : Attribute
+    {
+        public string Info { get; set; }
+
+        public CustomPropertyAttribute(string info)
+        {
+            Info = info;
+        }
+    }
 }

src/Humanizer.Tests/Humanizer.Tests.csproj

@@ -63,6 +63,7 @@
     <Compile Include="CasingTests.cs" />
     <Compile Include="DateHumanize.cs" />
     <Compile Include="CollectionHumanizeTests.cs" />
+    <Compile Include="EnumHumanizeWithCustomDescriptionPropertyNamesTests.cs" />
     <Compile Include="Localisation\bg\DateHumanizeTests.cs" />
     <Compile Include="Localisation\bg\TimeSpanHumanizeTests.cs" />
     <Compile Include="Localisation\cs\DateHumanizeTests.cs" />

src/Humanizer/Configuration/Configurator.cs

@@ -1,3 +1,5 @@
+using System;
+using System.Collections.Generic;
 using Humanizer.DateTimeHumanizeStrategy;
 using Humanizer.Localisation.Formatters;
 using Humanizer.Localisation.NumberToWords;
@@ -98,5 +100,20 @@ public static IDateTimeHumanizeStrategy DateTimeHumanizeStrategy
             get { return _dateTimeHumanizeStrategy; }
             set { _dateTimeHumanizeStrategy = value; }
         }
+
+        private const string DefaultEnumDescriptionPropertyName = "Description";
+        private static readonly Dictionary<Type, string> _enumDescriptionPropertyNames = new Dictionary<Type, string>();
+        internal static string EnumDescriptionPropertyNameFor(Type type)
+        {
+            string result = _enumDescriptionPropertyNames.TryGetValue(type, out result) ? result : null;
+            return result ?? DefaultEnumDescriptionPropertyName;
+        }
+        /// <summary>
+        /// The registry of custom attribute property names for Enum.Humanize
+        /// </summary>
+        public static IDictionary<Type, string> EnumDescriptionPropertyNames
+        {
+            get { return _enumDescriptionPropertyNames; }
+        }
     }
 }

src/Humanizer/EnumHumanizeExtensions.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Linq;
 using System.Reflection;
+using Humanizer.Configuration;
 
 namespace Humanizer
 {
@@ -9,7 +10,7 @@ namespace Humanizer
     /// </summary>
     public static class EnumHumanizeExtensions
     {
-        private static readonly Func<PropertyInfo, bool> DescriptionProperty = p => p.Name == "Description" && p.PropertyType == typeof (string);
+        private static readonly Func<PropertyInfo, bool> StringTypedProperty = p => p.PropertyType == typeof(string);
 
         /// <summary>
         /// Turns an enum member into a human readable string; e.g. AnonymousUser -> Anonymous user. It also honors DescriptionAttribute data annotation
@@ -19,28 +20,33 @@ public static class EnumHumanizeExtensions
         public static string Humanize(this Enum input)
         {
             Type type = input.GetType();
-            var memInfo = type.GetMember(input.ToString());
+            var caseName = input.ToString();
+            var memInfo = type.GetMember(caseName);
 
             if (memInfo.Length > 0)
             {
-                var customDescription = GetCustomDescription(memInfo[0]);
+                var propertyName = Configurator.EnumDescriptionPropertyNameFor(type);
+                var customDescription = GetCustomDescription(memInfo[0], propertyName);
 
                 if (customDescription != null)
                     return customDescription;
             }
 
-            return input.ToString().Humanize();
+            return caseName.Humanize();
         }
 
         // I had to add this method because PCL doesn't have DescriptionAttribute & I didn't want two versions of the code & thus the reflection
-        private static string GetCustomDescription(MemberInfo memberInfo)
+        private static string GetCustomDescription(MemberInfo memberInfo, string propertyName)
         {
             var attrs = memberInfo.GetCustomAttributes(true);
 
             foreach (var attr in attrs)
             {
                 var attrType = attr.GetType();
-                var descriptionProperty = attrType.GetProperties().FirstOrDefault(DescriptionProperty);
+                var descriptionProperty =
+                    attrType.GetProperties()
+                        .Where(StringTypedProperty)
+                        .FirstOrDefault(p => p.Name == propertyName);
                 if (descriptionProperty != null)
                     return descriptionProperty.GetValue(attr, null).ToString();
             }