Add metadata constants

Author: rasmusjp

src/Our.Umbraco.GraphQL/Constants.cs

@@ -3,5 +3,12 @@ namespace Our.Umbraco.GraphQL
     public class Constants
     {
         public const string ProductName = "Our.Umbraco.GraphQL";
+
+        public class Metadata
+        {
+            public const string AllowedAtRoot = "allowedAtRoot";
+            public const string AllowedChildren = "allowedChildren";
+            public const string ContentTypeAlias = "contentTypeAlias";
+        }
     }
 }

src/Our.Umbraco.GraphQL/GraphQLAuthenticationExtensions.cs

@@ -80,17 +80,17 @@ public static void SetPermissions(this FieldType type, string documentTypeAlias,
         public static void SetPermissions(this FieldType type, GraphType graphType, bool isBuiltInProperty = false)
         {
             // The graph type should have the doc type alias set in the meta data so we're accessing it from that
-            var doctypeAlias = graphType.GetMetadata<string>("documentTypeAlias");
+            var doctypeAlias = graphType.GetMetadata<string>(Constants.Metadata.ContentTypeAlias);
             type.SetPermissions(doctypeAlias, isBuiltInProperty);
         }
 
         public static void SetDoctypeMetadata(this FieldType type, string doctypeAlias)
         {
-            var currentAlias = type.GetMetadata<List<string>>("documentTypeAlias");
+            var currentAlias = type.GetMetadata<List<string>>(Constants.Metadata.ContentTypeAlias);
 
             if (currentAlias == null)
             {
-                type.Metadata["documentTypeAlias"] = doctypeAlias;
+                type.Metadata[Constants.Metadata.ContentTypeAlias] = doctypeAlias;
             }
         }
 

src/Our.Umbraco.GraphQL/Types/PublishedContentAtRootQueryGraphType.cs

@@ -19,9 +19,9 @@ public PublishedContentAtRootQueryGraphType(IEnumerable<IGraphType> documentGrap
                     return userContext.Umbraco.TypedContentAtRoot();
                 });
 
-            foreach (var type in documentGraphTypes.Where(x => x.GetMetadata<bool>("allowedAtRoot")))
+            foreach (var type in documentGraphTypes.Where(x => x.GetMetadata<bool>(Constants.Metadata.AllowedAtRoot)))
             {
-                string documentTypeAlias = type.GetMetadata<string>("documentTypeAlias");
+                string documentTypeAlias = type.GetMetadata<string>(Constants.Metadata.ContentTypeAlias);
 
                 this.AddField(
                     new FieldType

src/Our.Umbraco.GraphQL/Types/PublishedContentByTypeQueryGraphType.cs

@@ -11,7 +11,7 @@ public PublishedContentByTypeQueryGraphType(IEnumerable<IGraphType> documentGrap
 
             foreach (var type in documentGraphTypes)
             {
-                string documentTypeAlias = type.GetMetadata<string>("documentTypeAlias");
+                string documentTypeAlias = type.GetMetadata<string>(Constants.Metadata.ContentTypeAlias);
 
                 Field<PublishedContentInterfaceGraphType>()
                     .Name(type.Name)

src/Our.Umbraco.GraphQL/Types/PublishedContentGraphType.cs

@@ -15,9 +15,9 @@ public PublishedContentGraphType(IContentTypeComposition contentType, PublishedI
             IsTypeOf = content => ((IPublishedContent)content).ContentType.Alias == contentType.Alias;
             Metadata = new Dictionary<string, object>
             {
-                ["contentTypeAlias"] = contentType.Alias,
-                ["allowedAtRoot"] = contentType.AllowedAsRoot,
-                ["allowedChildren"] = contentType.AllowedContentTypes.Select(x => x.Alias).ToArray(),
+                [Constants.Metadata.ContentTypeAlias] = contentType.Alias,
+                [Constants.Metadata.AllowedAtRoot] = contentType.AllowedAsRoot,
+                [Constants.Metadata.AllowedChildren] = contentType.AllowedContentTypes.Select(x => x.Alias).ToArray(),
             };
 
             Interface<PublishedContentInterfaceGraphType>();
@@ -35,7 +35,7 @@ public PublishedContentGraphType(IContentTypeComposition contentType, PublishedI
                 .Name("_contentData")
                 .Description("Built in published content data.")
                 .Resolve(context => context.Source)
-                .SetDoctypeMetadata(GetMetadata<string>("documentTypeAlias"));
+                .SetDoctypeMetadata(GetMetadata<string>(Constants.Metadata.ContentTypeAlias));
 
             this.AddUmbracoContentPropeties(contentType, itemType);
         }

src/Our.Umbraco.GraphQL/Types/PublishedContentInterfaceGraphType.cs

@@ -16,7 +16,7 @@ public PublishedContentInterfaceGraphType()
                 .Name("_contentData")
                 .Description("Built in published content data.")
                 .Resolve(context => context.Source)
-                .SetDoctypeMetadata(GetMetadata<string>("documentTypeAlias"));
+                .SetDoctypeMetadata(GetMetadata<string>(Constants.Metadata.ContentTypeAlias));
         }
     }
 }

src/Our.Umbraco.GraphQL/ValidationRules/RequiresAuthValidationRule.cs

@@ -31,7 +31,7 @@ public INodeVisitor Validate(ValidationContext context)
                     // As a workaround this is set on the parent _conentData field and we use that when working out permissions
                     if (fieldDef.Name == "_contentData")
                     {
-                        var doctype = fieldDef.GetMetadata<string>("documentTypeAlias");
+                        var doctype = fieldDef.GetMetadata<string>(Constants.Metadata.ContentTypeAlias);
                         if (!string.IsNullOrEmpty(doctype))
                         {
                             currentDoctype = doctype;