Add logging for language-sanitized properties

Author: Caitlin Bales (MSFT)

src/GraphODataTemplateWriter/CodeHelpers/Android/TypeHelperAndroid.cs

@@ -6,9 +6,11 @@ namespace Microsoft.Graph.ODataTemplateWriter.CodeHelpers.Android
     using Microsoft.Graph.ODataTemplateWriter.Extensions;
     using Vipr.Core.CodeModel;
     using System;
+    using NLog;
 
     public static class TypeHelperAndroid
     {
+        private static Logger logger = LogManager.GetCurrentClassLogger();
         public const string ReservedPrefix = "msgraph_";
         public static HashSet<string> ReservedNames
         {
@@ -95,6 +97,7 @@ public static string SanitizePropertyName(this string property, OdcmObject odcmP
         {
             if (ReservedNames.Contains(property))
             {
+                logger.Info("Property {0} is a reserved word in Android. Converting to {1}{0}", property, ReservedPrefix);
                 return ReservedPrefix + property;
             }
 
@@ -104,6 +107,7 @@ public static string SanitizePropertyName(this string property, OdcmObject odcmP
                 if (odcmProperty.Projection.Type.Name.ToUpperFirstChar() == odcmProperty.Name.ToUpperFirstChar())
                 {
                     // Name the property: {metadataName} + "Property"
+                    logger.Info("Property type {0} has the same name as the class. Converting to {0}Property", property);
                     return string.Concat(property, "Property");
                 }
 

src/GraphODataTemplateWriter/CodeHelpers/CSharp/TypeHelperCSharp.cs

@@ -6,9 +6,12 @@ namespace Microsoft.Graph.ODataTemplateWriter.CodeHelpers.CSharp
     using System.Collections.Generic;
     using Microsoft.Graph.ODataTemplateWriter.Extensions;
     using Vipr.Core.CodeModel;
+    using NLog;
 
     public static class TypeHelperCSharp
     {
+        private static Logger logger = LogManager.GetCurrentClassLogger();
+
         public const string DefaultReservedPrefix = "@";
         public static ICollection<string> GetReservedNames()
         {
@@ -235,6 +238,7 @@ public static string GetSanitizedPropertyName(this string property, OdcmProperty
             {
                 var reservedPrefix = string.IsNullOrEmpty(prefix) ? DefaultReservedPrefix : prefix;
 
+                logger.Info("Property {0} is a reserved word in .NET. Converting to {1}{0}", property.ToUpperFirstChar(), reservedPrefix);
                 return string.Concat(reservedPrefix, property.ToUpperFirstChar());
             }
 
@@ -247,6 +251,7 @@ public static string GetSanitizedPropertyName(this string property, OdcmProperty
                 if (odcmProperty.Projection.Type.Name.ToUpperFirstChar() == odcmProperty.Class.Name.ToUpperFirstChar())
                 {
                     // Name the property: {metadataName} + "Property"
+                    logger.Info("Property type {0} has the same name as the class. Converting to {0}Property", property);
                     return string.Concat(property, "Property");
                 }
 

src/GraphODataTemplateWriter/Extensions/OdcmModelExtensions.cs

@@ -7,9 +7,12 @@ namespace Microsoft.Graph.ODataTemplateWriter.Extensions
     using System.Linq;
     using Microsoft.Graph.ODataTemplateWriter.Settings;
     using Vipr.Core.CodeModel;
+    using NLog;
 
     public static class OdcmModelExtensions
     {
+        private static Logger logger = LogManager.GetCurrentClassLogger();
+
         public static bool IsCollection(this OdcmProperty odcmProperty)
         {
             return odcmProperty.IsCollection;
@@ -152,8 +155,8 @@ public static OdcmProperty GetServiceCollectionNavigationPropertyForPropertyType
                     .First();
             } catch (Exception e)
             {
-                //TODO: Pass the exception to the logger
-                Console.WriteLine("Error: The navigation property \"{0}\" on class \"{1}\" does not specify it is self-contained nor is it defined in an EntitySet", odcmProperty.Name.ToString(), odcmProperty.Class.FullName.ToString());
+                logger.Error("The navigation property \"{0}\" on class \"{1}\" does not specify it is self-contained nor is it defined in an EntitySet", odcmProperty.Name.ToString(), odcmProperty.Class.FullName.ToString());
+                logger.Error(e);
                 return null;
             }
         }