Value() should be defined on a non-pointer receiver

Taik · Taik · commit dfd5d81213b2 · 2016-11-22T15:46:09.000-05:00
diff --git a/types/types.go b/types/types.go
@@ -140,7 +140,7 @@ func (n *NullJSONText) Scan(value interface{}) error {
 }
 
 // Value implements the driver Valuer interface.
-func (n *NullJSONText) Value() (driver.Value, error) {
+func (n NullJSONText) Value() (driver.Value, error) {
 	if !n.Valid {
 		return nil, nil
 	}

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,7 @@ func (n *NullJSONText) Scan(value interface{}) error {`
`140`	`140`	`}`
`141`	`141`
`142`	`142`	`// Value implements the driver Valuer interface.`
`143`		`-func (n *NullJSONText) Value() (driver.Value, error) {`
	`143`	`+func (n NullJSONText) Value() (driver.Value, error) {`
`144`	`144`	`if !n.Valid {`
`145`	`145`	`return nil, nil`
`146`	`146`	`}`