Merge pull request #194 from nextcloud/feat/openapitype/warn-non-requ…

…ired-nullable-properties feat(OpenApiType): Warn about non-required nullable properties
nextcloud · Dec 17, 2024 · 40302d1 · 40302d1
2 parents 2bf65ca + 7c198c8
commit 40302d1
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/OpenApiType.php b/src/OpenApiType.php
@@ -210,6 +210,8 @@ public static function resolve(string $context, array $definitions, ParamTagValu
 				$properties[$name] = $type;
 				if (!$item->optional) {
 					$required[] = $name;
+				} elseif ($type->nullable) {
+					Logger::warning($context, 'Property "' . $name . '" is both nullable and not required. Please consider only using one of these at once.');
 				}
 			}
 			return new OpenApiType(