Ugh this appears to have broken many things.

I'm concerned that the requirement to use .SQL(...) and the weird dialect dance makes this a bit complex and certainly error prone.

I think that we need to think again about the xorm interface - certainly the session.OrderBy(...) function needs to be changed to take interface{} pairs eg. from:

func (session *Session) OrderBy(order string) *Session { ... }

to:

func (session *Session) OrderBy(order interface{}, args ...interface{}) *Session { ... }

Then we need to think a bit more about how builders could pass down the ordering - but I suspect that keeping these separate from the condition would still be right.