- fixes a bug where action/methods for have a reserved name in java

Author: baywet

Templates/Android/BaseModel.template.tt

@@ -43,7 +43,7 @@
     }
 
     public string MethodName(OdcmObject c) {
-        return c.Name.Substring(c.Name.IndexOf(".") + 1).ToUpperFirstChar();
+        return c.Name.Substring(c.Name.IndexOf(".") + 1).SanitizePropertyName(property).ToUpperFirstChar();
     }
 
     public string MethodFullName(OdcmObject c) {

Templates/Java/BaseJavaModel.template.tt

@@ -43,7 +43,7 @@
     }
 
     public string MethodName(OdcmObject c) {
-        return c.Name.Substring(c.Name.IndexOf(".") + 1).ToUpperFirstChar();
+        return c.Name.Substring(c.Name.IndexOf(".") + 1).SanitizePropertyName(c).ToUpperFirstChar();
     }
 
     public string MethodFullName(OdcmObject c) {

src/GraphODataTemplateWriter/CodeHelpers/Java/TypeHelperJava.cs

@@ -10,17 +10,11 @@ namespace Microsoft.Graph.ODataTemplateWriter.CodeHelpers.Java
 
     public static class TypeHelperJava
     {
-        private static Logger logger = LogManager.GetCurrentClassLogger();
+        private static readonly Logger logger = LogManager.GetCurrentClassLogger();
         public const string ReservedPrefix = "msgraph";
-        public static HashSet<string> ReservedNames
-        {
-            get
-            {
-                return new HashSet<string>(StringComparer.OrdinalIgnoreCase) {
+        public static Lazy<HashSet<string>> ReservedNames { get; private set; } = new Lazy<HashSet<string>>(() => new HashSet<string>(StringComparer.OrdinalIgnoreCase) {
                     "abstract", "continue", "for", "new", "switch", "assert", "default", "if", "package", "synchronized", "boolean", "do", "goto", "private", "this", "break", "double", "implements", "protected", "throw", "byte", "else", "import", "public", "throws", "case", "enum", "instanceof", "return", "transient", "catch", "extends", "int", "short", "try", "char", "final", "interface", "static", "void", "class", "finally", "long", "strictfp", "volatile", "const", "float", "native", "super", "while", "true", "false", "null", "import"
-                };
-            }
-        }
+                });
 
         public static string GetReservedPrefix(this OdcmType @type)
         {
@@ -73,7 +67,7 @@ public static string GetTypeString(this OdcmProperty property)
         {
             var propertyType = property.Projection.Type;
             var typeString = GetTypeString(propertyType);
-            if (propertyType.Namespace != OdcmNamespace.Edm && ReservedNames.Contains(typeString))
+            if (propertyType.Namespace != OdcmNamespace.Edm && ReservedNames.Value.Contains(typeString))
             {
                 typeString = "com.microsoft.graph.models.extensions." + typeString;
             }
@@ -95,24 +89,24 @@ public static string GetToLowerFirstCharName(this OdcmProperty property)
 
         public static string SanitizePropertyName(this string property, OdcmObject odcmProperty = null)
         {
-            if (ReservedNames.Contains(property))
+            if (ReservedNames.Value.Contains(property))
             {
                 logger.Info("Property \"{0}\" is a reserved word in Java. Converting to \"{1}{0}\"", property, ReservedPrefix);
-                return ReservedPrefix + property;
+                return ReservedPrefix + property.ToUpperFirstChar();
             }
 
             if (odcmProperty != null && property == odcmProperty.Name.ToUpperFirstChar())
             {
                 // Check whether the property type is the same as the class name.
-                if (odcmProperty.Projection.Type.Name.ToUpperFirstChar() == odcmProperty.Name.ToUpperFirstChar())
+                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");
                 }
 
                 // Name the property by its type. Sanitize it in case the type is a reserved name.  
-                return odcmProperty.Projection.Type.Name.ToUpperFirstChar().SanitizePropertyName(odcmProperty);
+                return odcmProperty.Projection.Type?.Name?.ToUpperFirstChar()?.SanitizePropertyName(odcmProperty) ?? property.SanitizePropertyName();
             }
 
             return property.Replace("@", string.Empty).Replace(".", string.Empty);