consider an alternative query.Where("").Match() about bolthold HOT 4 CLOSED

I agree for the most part however:

• It breaks backwards compatibility
• The RecordAccess type provides access to more than just the record.
• It's yet another reflect call, although that's not too big of a deal IMO

I think it's worth doing. What I'm tentatively picturing is to change the MatchFunc signature to:

MatchFunc(func(value interface{}) (bool, error)

If value is of type RecordAccess, work just like it does now. If value has type compatibility with the entire record, return the record. If it has type compatibility with the currently selected field, return the field.

So you could do something like this:

bolthold.Where("Name").MatchFunc(func(val string) (bool, error) {
			return val != "one" && val != "two" && val != "three"
		})

or

bolthold.Where("Name").MatchFunc(func(val person) (bool, error) {
			return val.Name != "one" && val.Name != "two" && val.Name != "three"
		})

from bolthold.

The changes discussed above have been moved into master.

from bolthold.

maybe you dont break api and you add a new handler ?

from bolthold.

The problem with adding a new handler is that it makes the API more complicated, especially for people coming to it for the first time. What would you name the new handler that made it clear that this new MatchFunc handler also matches on the passed in function, but in a slightly different way than the other handler that matches on a function.

I'd prefer to just extend the behavior of the existing handler without changing the current behavior.

from bolthold.