- fixes a bug where type specific indexers would fail to build in Go

Signed-off-by: Vincent Biret <[email protected]>

Author: baywet

CHANGELOG.md

@@ -16,6 +16,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Renamed the Shell language to CLI. [#3023](https://github.com/microsoft/kiota/issues/3023)
 - Fixed a bug where and extraneous indexer would be generated for CLI. [#3088](https://github.com/microsoft/kiota/issues/3088)
+- Fixed a bug where type specific indexers would fail to build in Go. [#3090](https://github.com/microsoft/kiota/issues/3090)
 
 ## [1.5.0] - 2023-08-04
 

src/Kiota.Builder/Refiners/GoRefiner.cs

@@ -515,6 +515,9 @@ private static void AddErrorImportForEnums(CodeElement currentElement)
                     method.Parameters.Any(x => x.IsOfKind(CodeParameterKind.Path) &&
                                             !typeToSkipStrConv.Contains(x.Type.Name)),
             "strconv", "FormatBool"),
+        new (static x => x is CodeMethod method && method.IsOfKind(CodeMethodKind.IndexerBackwardCompatibility) &&
+                    method.OriginalIndexer is CodeIndexer indexer && !indexer.IndexParameter.Type.Name.Equals("string", StringComparison.OrdinalIgnoreCase),
+            "strconv", "FormatInt"),
         new (static x => x is CodeMethod method && method.IsOfKind(CodeMethodKind.Serializer),
             "github.com/microsoft/kiota-abstractions-go/serialization", "SerializationWriter"),
         new (static x => x is CodeMethod method && method.IsOfKind(CodeMethodKind.Deserializer, CodeMethodKind.Factory),

src/Kiota.Builder/Writers/Go/GoConventionService.cs

@@ -211,15 +211,21 @@ internal void AddParametersAssignment(LanguageWriter writer, CodeTypeBase? pathP
         foreach (var p in parameters)
         {
             var isStringStruct = !p.Item1.IsNullable && p.Item1.Name.Equals("string", StringComparison.OrdinalIgnoreCase);
-            var defaultValue = isStringStruct ? "\"\"" : "nil";
-            var pointerDereference = isStringStruct ? string.Empty : "*";
-            writer.StartBlock($"if {p.Item3} != {defaultValue} {{");
+            var (defaultValue, pointerDereference, shouldCheckNullability) = (isStringStruct, p.Item1.IsNullable) switch
+            {
+                (true, _) => ("\"\"", string.Empty, true),
+                (_, true) => ("nil", "*", true),
+                (_, false) => (string.Empty, string.Empty, false),
+            };
+            if (shouldCheckNullability)
+                writer.StartBlock($"if {p.Item3} != {defaultValue} {{");
             writer.WriteLine($"{pathParametersTarget}[\"{p.Item2}\"] = {GetValueStringConversion(p.Item1.Name, pointerDereference + p.Item3)}");
-            writer.CloseBlock();
+            if (shouldCheckNullability)
+                writer.CloseBlock();
         }
     }
 #pragma warning restore CA1822 // Method should be static
-    private const string StrConvHash = "i53ac87e8cb3cc9276228f74d38694a208cacb99bb8ceb705eeae99fb88d4d274";
+    internal const string StrConvHash = "i53ac87e8cb3cc9276228f74d38694a208cacb99bb8ceb705eeae99fb88d4d274";
     private const string TimeFormatHash = "i336074805fc853987abe6f7fe3ad97a6a6f3077a16391fec744f671a015fbd7e";
     private static string GetValueStringConversion(string typeName, string reference)
     {