issues/395 (#396)

Co-authored-by: Vasiliy Ditsyak <vasilich6107@users.noreply.github.com>

Author: vasilich6107

packages/graphql_codegen/lib/src/printer/base/input.dart

@@ -356,29 +356,59 @@ Code _printToJson(PrintContext context, Iterable<ContextProperty> properties) {
         ))
         .statement,
     for (final property in properties) ...[
-      if (!property.isRequired)
-        Code(
-            'if(${kDataVariableName}.containsKey(\'${property.name.value}\')) {'),
-      declareFinal(context.namePrinter.printLocalPropertyName(property.name))
-          .assign(refer(context.namePrinter.printPropertyName(property.name)))
-          .statement,
-      refer(resultDataVariableName)
-          .index(literalString(property.name.value))
-          .assign(
-            printToJsonValueOnExpression(
-              context,
-              property,
-              printMaybeAddCast(
-                refer(
-                  context.namePrinter.printLocalPropertyName(property.name),
+      if (context.context.config.allowMissingNullableKeysInFromJson &&
+          !property.isNonNull) ...[
+        // For nullable properties when allowMissingNullableKeysInFromJson is true
+        declareFinal(context.namePrinter.printLocalPropertyName(property.name))
+            .assign(refer(kDataVariableName)
+                .property('containsKey')
+                .call([literalString(property.name.value)]).conditional(
+              refer(context.namePrinter.printPropertyName(property.name)),
+              literalNull,
+            ))
+            .statement,
+        refer(resultDataVariableName)
+            .index(literalString(property.name.value))
+            .assign(
+              printToJsonValueOnExpression(
+                context,
+                property,
+                printMaybeAddCast(
+                  refer(
+                    context.namePrinter.printLocalPropertyName(property.name),
+                  ),
+                  property.hasDefaultValue && property.isNonNull,
+                  printClassPropertyType(context, property),
                 ),
-                property.hasDefaultValue && property.isNonNull,
-                printClassPropertyType(context, property),
               ),
-            ),
-          )
-          .statement,
-      if (!property.isRequired) Code('}'),
+            )
+            .statement,
+      ] else ...[
+        // Original logic for required properties or when allowMissingNullableKeysInFromJson is false
+        if (!property.isRequired)
+          Code(
+              'if(${kDataVariableName}.containsKey(\'${property.name.value}\')) {'),
+        declareFinal(context.namePrinter.printLocalPropertyName(property.name))
+            .assign(refer(context.namePrinter.printPropertyName(property.name)))
+            .statement,
+        refer(resultDataVariableName)
+            .index(literalString(property.name.value))
+            .assign(
+              printToJsonValueOnExpression(
+                context,
+                property,
+                printMaybeAddCast(
+                  refer(
+                    context.namePrinter.printLocalPropertyName(property.name),
+                  ),
+                  property.hasDefaultValue && property.isNonNull,
+                  printClassPropertyType(context, property),
+                ),
+              ),
+            )
+            .statement,
+        if (!property.isRequired) Code('}'),
+      ]
     ],
     refer(resultDataVariableName).returned.statement,
   ]);

packages/graphql_codegen/test/assets/issue_395/options.json

@@ -0,0 +1,3 @@
+{
+  "allowMissingNullableKeysInFromJson": true
+}

packages/graphql_codegen/test/assets/issue_395/schema.graphql

@@ -0,0 +1,4 @@
+input CarInput {
+  idNullable: ID
+  idNonNullable: ID!
+}