Skip to content

Commit

Permalink
fix(QueryUtils): Don't overly specify that grammars must extend `Base…
Browse files Browse the repository at this point in the history
…Grammar`

Lean into more fluid interfaces.  Even `AutoDiscover` doesn't technically
extend `BaseGrammar`.
  • Loading branch information
elpete committed Feb 19, 2025
1 parent 98d6e92 commit c62e301
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions models/Query/QueryUtils.cfc
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ component singleton displayname="QueryUtils" accessors="true" {
*
* @return any
*/
public any function extractBinding( any value, required BaseGrammar grammar ) {
public any function extractBinding( any value, required any grammar ) {
if ( isNull( arguments.value ) ) {
return { "cfsqltype": "CF_SQL_VARCHAR", "value": "", "null": true };
}
Expand Down Expand Up @@ -178,7 +178,7 @@ component singleton displayname="QueryUtils" accessors="true" {
*
* @return string
*/
public string function inferSqlType( any value, required BaseGrammar grammar ) {
public string function inferSqlType( any value, required any grammar ) {
if ( isNull( arguments.value ) ) {
return "CF_SQL_VARCHAR";
}
Expand Down Expand Up @@ -644,7 +644,7 @@ component singleton displayname="QueryUtils" accessors="true" {
return true;
}

public string function serializeBindings( required array bindings, required BaseGrammar grammar ) {
public string function serializeBindings( required array bindings, required any grammar ) {
return serializeJSON(
arguments.bindings.map( function( binding ) {
var newBinding = extractBinding( duplicate( binding ), grammar );
Expand Down

0 comments on commit c62e301

Please sign in to comment.