bugfix: arguments with null default were required

docs/changelog.md

@@ -69,3 +69,4 @@
 - feature: improved descriptions for all internal error states
 
 ## master
+- bugfix: function arguments with a null default value were required instead of optional

src/graphql.rs

@@ -1373,12 +1373,23 @@ fn function_args(schema: &Arc<__Schema>, func: &Arc<Function>) -> Vec<__InputVal
                 (t, arg_name, arg_default)
             })
         })
-        .map(|(arg_type, arg_name, arg_default)| __InputValue {
-            name_: schema.graphql_function_arg_name(func, arg_name),
-            type_: arg_type,
-            description: None,
-            default_value: arg_default,
-            sql_type: None,
+        .map(|(arg_type, arg_name, arg_default)| {
+            let default_value = if let Some((default_value, is_null)) = arg_default {
+                if is_null {
+                    None
+                } else {
+                    Some(default_value)
+                }
+            } else {
+                None
+            };
+            __InputValue {
+                name_: schema.graphql_function_arg_name(func, arg_name),
+                type_: arg_type,
+                description: None,
+                default_value,
+                sql_type: None,
+            }
         })
         .collect()
 }

src/sql_types.rs

@@ -88,7 +88,7 @@ pub struct Function {
 }
 
 impl Function {
-    pub fn args(&self) -> impl Iterator<Item = (u32, &str, Option<&str>, Option<String>)> {
+    pub fn args(&self) -> impl Iterator<Item = (u32, &str, Option<&str>, Option<(String, bool)>)> {
         ArgsIterator::new(
             &self.arg_types,
             &self.arg_type_names,
@@ -200,7 +200,7 @@ struct ArgsIterator<'a> {
     arg_types: &'a [u32],
     arg_type_names: &'a Vec<String>,
     arg_names: &'a Option<Vec<String>>,
-    arg_defaults: Vec<Option<String>>,
+    arg_defaults: Vec<Option<(String, bool)>>,
 }
 
 impl<'a> ArgsIterator<'a> {
@@ -230,7 +230,7 @@ impl<'a> ArgsIterator<'a> {
         arg_defaults: &'a Option<String>,
         num_default_args: usize,
         num_total_args: usize,
-    ) -> Vec<Option<String>> {
+    ) -> Vec<Option<(String, bool)>> {
         let mut defaults = vec![None; num_total_args];
         let Some(arg_defaults) = arg_defaults else {
             return defaults;
@@ -249,23 +249,27 @@ impl<'a> ArgsIterator<'a> {
         debug_assert!(num_default_args <= num_total_args);
         let start_idx = num_total_args - num_default_args;
         for i in start_idx..num_total_args {
-            defaults[i] =
-                Self::sql_to_graphql_default(default_strs[i - start_idx], arg_types[i])
+            defaults[i] = Self::sql_to_graphql_default(default_strs[i - start_idx], arg_types[i])
         }
 
         defaults
     }
 
-    fn sql_to_graphql_default(default_str: &str, type_oid: u32) -> Option<String> {
+    fn sql_to_graphql_default(default_str: &str, type_oid: u32) -> Option<(String, bool)> {
         let trimmed = default_str.trim();
+        if trimmed.starts_with("NULL::") {
+            return Some(("".to_string(), true));
+        }
         match type_oid {
-            21 | 23 => trimmed.parse::<i32>().ok().map(|i| i.to_string()),
-            16 => trimmed.parse::<bool>().ok().map(|i| i.to_string()),
-            700 | 701 => trimmed.parse::<f64>().ok().map(|i| i.to_string()),
-            25 => trimmed
-                .strip_suffix("::text")
-                .to_owned()
-                .map(|i| format!("\"{}\"", i.trim_matches(',').trim_matches('\''))),
+            21 | 23 => trimmed.parse::<i32>().ok().map(|i| (i.to_string(), false)),
+            16 => trimmed.parse::<bool>().ok().map(|i| (i.to_string(), false)),
+            700 | 701 => trimmed.parse::<f64>().ok().map(|i| (i.to_string(), false)),
+            25 => trimmed.strip_suffix("::text").map(|i| {
+                (
+                    format!("\"{}\"", i.trim_matches(',').trim_matches('\'')),
+                    false,
+                )
+            }),
             _ => None,
         }
     }
@@ -276,7 +280,7 @@ lazy_static! {
 }
 
 impl<'a> Iterator for ArgsIterator<'a> {
-    type Item = (u32, &'a str, Option<&'a str>, Option<String>);
+    type Item = (u32, &'a str, Option<&'a str>, Option<(String, bool)>);
 
     fn next(&mut self) -> Option<Self::Item> {
         if self.index < self.arg_types.len() {