discuss: issue #3525 on pgsql, char ? in sql arguments

contrib/drivers/pgsql/pgsql_do_filter.go

@@ -21,6 +21,10 @@ func (d *Driver) DoFilter(
  var index int
  // Convert placeholder char '?' to string "$x".
  newSql, err = gregex.ReplaceStringFunc(`\?`, sql, func(s string) string {
+ if _, ok := args[index].(gdb.NoReplacement); ok {
+ args = append(args[:index], args[index+1:]...)
+ return s
+ }
  index++
  return fmt.Sprintf(`$%d`, index)
  })

database/gdb/gdb.go

@@ -356,12 +356,13 @@ type Counter struct {
 }
 
 type (
- Raw string // Raw is a raw sql that will not be treated as argument but as a direct sql part.
- Value = *gvar.Var // Value is the field value type.
- Record map[string]Value // Record is the row record of the table.
- Result []Record // Result is the row record array.
- Map = map[string]interface{} // Map is alias of map[string]interface{}, which is the most common usage map type.
- List = []Map // List is type of map array.
+ Raw string // Raw is a raw sql that will not be treated as argument but as a direct sql part.
+ Value = *gvar.Var // Value is the field value type.
+ Record map[string]Value // Record is the row record of the table.
+ Result []Record // Result is the row record array.
+ Map = map[string]interface{} // Map is alias of map[string]interface{}, which is the most common usage map type.
+ List = []Map // List is type of map array.
+ NoReplacement bool
 )
 
 type CatchSQLManager struct {

database/gdb/gdb_core.go

@@ -623,6 +623,11 @@ func (c *Core) DoUpdate(ctx context.Context, link Link, table string, data inter
 
  default:
  if s, ok := v.(Raw); ok {
+ placeholders := strings.Index(string(s), "?")
+ if placeholders != -1 {
+ // 插入一个空的
+ args = append([]any{(NoReplacement)(true)}, args...)
+ }
  fields = append(fields, c.QuoteWord(k)+"="+gconv.String(s))
  } else {
  fields = append(fields, c.QuoteWord(k)+"=?")