Add support for setting enums in scope string as well

Author: mythz

src/ServiceStack.Text/JsConfig.cs

@@ -44,7 +44,8 @@ public static JsConfigScope CreateScope(string config, JsConfigScope scope = nul
             {
                 var parts = item.SplitOnFirst(':');
                 var key = parts[0].ToLower();
-                var boolValue = parts.Length == 1 || (parts[1].ToLower() != "false" && parts[1] != "0");
+                var value = parts.Length == 2 ? parts[1].ToLower() : null;
+                var boolValue = parts.Length == 1 || (value != "false" && value != "0");
 
                 switch (key)
                 {
@@ -111,6 +112,51 @@ public static JsConfigScope CreateScope(string config, JsConfigScope scope = nul
                     case "reuseStringBuffer":
                         scope.ReuseStringBuffer = boolValue;
                         break;
+                    case "datehandler":
+                        switch (value)
+                        {
+                            case "timestampoffset":
+                                scope.DateHandler = DateHandler.TimestampOffset;
+                                break;
+                            case "dcjscompatible":
+                                scope.DateHandler = DateHandler.DCJSCompatible;
+                                break;
+                            case "iso8601":
+                                scope.DateHandler = DateHandler.ISO8601;
+                                break;
+                            case "rfc1123":
+                                scope.DateHandler = DateHandler.RFC1123;
+                                break;
+                            case "unixtime":
+                                scope.DateHandler = DateHandler.UnixTime;
+                                break;
+                            case "unixtimems":
+                                scope.DateHandler = DateHandler.UnixTimeMs;
+                                break;
+                        }
+                        break;
+                    case "timespanhandler":
+                        switch (value)
+                        {
+                            case "durationformat":
+                                scope.TimeSpanHandler = TimeSpanHandler.DurationFormat;
+                                break;
+                            case "standardformat":
+                                scope.TimeSpanHandler = TimeSpanHandler.StandardFormat;
+                                break;
+                        }
+                        break;
+                    case "propertyconvention":
+                        switch (value)
+                        {
+                            case "lenient":
+                                scope.PropertyConvention = PropertyConvention.Lenient;
+                                break;
+                            case "strict":
+                                scope.PropertyConvention = PropertyConvention.Strict;
+                                break;
+                        }
+                        break;
                 }
             }
 

tests/ServiceStack.Text.Tests/JsConfigTests.cs

@@ -201,6 +201,12 @@ public void Does_create_scope_from_string()
             Assert.That(!scope.ExcludeDefaultValues.Value);
             Assert.That(scope.IncludeDefaultEnums.Value);
             scope.Dispose();
-        }        
+
+            scope = JsConfig.CreateScope("DateHandler:ISO8601,timespanhandler:durationformat,PropertyConvention:strict");
+            Assert.That(scope.DateHandler, Is.EqualTo(DateHandler.ISO8601));
+            Assert.That(scope.TimeSpanHandler, Is.EqualTo(TimeSpanHandler.DurationFormat));
+            Assert.That(scope.PropertyConvention, Is.EqualTo(PropertyConvention.Strict));
+            scope.Dispose();
+        }
     }
 }