fix: improve error message for unsupported sqlpage function arguments

Update expr_to_stmt_param to correctly bubble up UnsupportedExpr errors with the summary of the larger compound expression. This ensures the error message shows the full expression (like 'x || c') instead of just the inner leaf node ('x').

Author: lovasoa

src/webserver/database/sql/parameter_extraction.rs

@@ -379,7 +379,7 @@ fn expr_to_stmt_param(
     ctx: &ParamExtractContext,
 ) -> Result<StmtParam, ExprToParamError> {
     let line = arg.span().start.line;
-    match arg {
+    let res = match arg {
         Expr::Value(ValueWithSpan {
             value: Value::Placeholder(placeholder),
             ..
@@ -419,11 +419,8 @@ fn expr_to_stmt_param(
             left,
             op: BinaryOperator::StringConcat,
             right,
-        } => {
-            let left = expr_to_stmt_param(left, ctx)?;
-            let right = expr_to_stmt_param(right, ctx)?;
-            Ok(StmtParam::Concat(vec![left, right]))
-        }
+        } => expr_to_stmt_param(left, ctx)
+            .and_then(|l| expr_to_stmt_param(right, ctx).map(|r| StmtParam::Concat(vec![l, r]))),
         Expr::Function(Function {
             name: ObjectName(func_name_parts),
             args:
@@ -446,7 +443,16 @@ fn expr_to_stmt_param(
                 summary: expr_summary(arg),
             },
         }),
-    }
+    };
+
+    res.map_err(|mut e| {
+        if let ExprToParamErrorKind::UnsupportedExpr { ref mut summary } = e.kind {
+            if !matches!(arg, Expr::Function(_)) {
+                *summary = expr_summary(arg);
+            }
+        }
+        e
+    })
 }
 
 fn function_arg_expr(arg: &mut FunctionArg) -> Option<&mut Expr> {