fix(QueryBuilder): Explicit arguments scoping

Adds explicit argument scope in insert() for compatibility when Lucee is run in "modern" localmode
coldbox-modules · Jan 24, 2022 · b5c1070 · b5c1070
1 parent ced5d2a
commit b5c1070
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/models/Query/QueryBuilder.cfc b/models/Query/QueryBuilder.cfc
@@ -2476,18 +2476,18 @@ component displayname="QueryBuilder" accessors="true" {
      * @return query
      */
     public any function insert( required any values, struct options = {}, boolean toSql = false ) {
-        if ( values.isEmpty() ) {
+        if ( arguments.values.isEmpty() ) {
             return;
         }
 
-        if ( !isArray( values ) ) {
-            if ( !isStruct( values ) ) {
+        if ( !isArray( arguments.values ) ) {
+            if ( !isStruct( arguments.values ) ) {
                 throw(
                     type = "InvalidSQLType",
                     message = "Please pass a struct or an array of structs mapping columns to values"
                 );
             }
-            values = [ values ];
+            arguments.values = [ arguments.values ];
         }
 
         var columns = arguments.values[ 1 ]