From 9ea2f9f6865b4f3073773d2e94739598e09cb88a Mon Sep 17 00:00:00 2001 From: johnnyjoy Date: Wed, 23 Jul 2025 22:13:48 +0800 Subject: [PATCH] fix: linter --- plugin/filter/common_converter.go | 35 +++++++++++++++---------------- 1 file changed, 17 insertions(+), 18 deletions(-) diff --git a/plugin/filter/common_converter.go b/plugin/filter/common_converter.go index 2a9e480b2..098384817 100644 --- a/plugin/filter/common_converter.go +++ b/plugin/filter/common_converter.go @@ -512,27 +512,26 @@ func (c *CommonSQLConverter) handleBooleanComparison(ctx *ConvertContext, field, return err } return nil - } else { - // Other fields use IS TRUE / NOT(... IS TRUE) - var sqlExpr string - if operator == "=" { - if valueBool { - sqlExpr = fmt.Sprintf("%s IS TRUE", c.dialect.GetJSONExtract(jsonPath)) - } else { - sqlExpr = fmt.Sprintf("NOT(%s IS TRUE)", c.dialect.GetJSONExtract(jsonPath)) - } - } else { // operator == "!=" - if valueBool { - sqlExpr = fmt.Sprintf("NOT(%s IS TRUE)", c.dialect.GetJSONExtract(jsonPath)) - } else { - sqlExpr = fmt.Sprintf("%s IS TRUE", c.dialect.GetJSONExtract(jsonPath)) - } + } + // Other fields use IS TRUE / NOT(... IS TRUE) + var sqlExpr string + if operator == "=" { + if valueBool { + sqlExpr = fmt.Sprintf("%s IS TRUE", c.dialect.GetJSONExtract(jsonPath)) + } else { + sqlExpr = fmt.Sprintf("NOT(%s IS TRUE)", c.dialect.GetJSONExtract(jsonPath)) } - if _, err := ctx.Buffer.WriteString(sqlExpr); err != nil { - return err + } else { // operator == "!=" + if valueBool { + sqlExpr = fmt.Sprintf("NOT(%s IS TRUE)", c.dialect.GetJSONExtract(jsonPath)) + } else { + sqlExpr = fmt.Sprintf("%s IS TRUE", c.dialect.GetJSONExtract(jsonPath)) } - return nil } + if _, err := ctx.Buffer.WriteString(sqlExpr); err != nil { + return err + } + return nil } // Special handling for MySQL - use raw operator with CAST