Merge pull request #510 from Dragwar/dragwar/issue/457

Fix for issue #457

Author: oformaniuk

source/Handlebars.Test/IssueTests.cs

@@ -623,7 +623,19 @@ public void WeirdBehaviour()
             Assert.Equal(expected, actual1);
             Assert.Equal(expected, actual2);
         }
-        
+
+        // Issue: https://github.com/Handlebars-Net/Handlebars.Net/issues/457
+        [Fact]
+        public void StringLength()
+        {
+            var handlebars = Handlebars.Create();
+            var render = handlebars.Compile("{{str.length}}");
+            object data = new { str = "string" };
+
+            var actual = render(data);
+            Assert.Equal("6", actual);
+        }
+
         // Issue: https://github.com/Handlebars-Net/Handlebars.Net/issues/470
         [Theory]
         [ClassData(typeof(EscapeExpressionGenerator))]

source/Handlebars/ObjectDescriptors/ObjectDescriptorProvider.cs

@@ -13,8 +13,6 @@ namespace HandlebarsDotNet.ObjectDescriptors
 {
     public sealed class ObjectDescriptorProvider : IObjectDescriptorProvider
     {
-        private static readonly Type StringType = typeof(string);
-        
         private readonly LookupSlim<Type, DeferredValue<Type, ChainSegment[]>, ReferenceEqualityComparer<Type>> _membersCache = new LookupSlim<Type, DeferredValue<Type, ChainSegment[]>, ReferenceEqualityComparer<Type>>(new ReferenceEqualityComparer<Type>());
         private readonly ReflectionMemberAccessor _reflectionMemberAccessor;
 
@@ -25,17 +23,11 @@ public ObjectDescriptorProvider(IReadOnlyList<IMemberAliasProvider> aliasProvide
 
         public bool TryGetDescriptor(Type type, out ObjectDescriptor value)
         {
-            if (type == StringType)
-            {
-                value = ObjectDescriptor.Empty;
-                return false;
-            }
-            
             value = new ObjectDescriptor(
-                type, 
-                _reflectionMemberAccessor, 
-                GetProperties, 
-                self => new ObjectIterator(self), 
+                type,
+                _reflectionMemberAccessor,
+                GetProperties,
+                self => new ObjectIterator(self),
                 dependencies: _membersCache
             );