nette/database

Missing `havingOr` ?

dakujem opened this issue · 3 comments

I would like to see method Selection::havingOr implemented, as that feature is missing and cumbersome to achieve.

It should work analogously to the Selection::whereOr method.

Let me know if I could work on this. Also, I'd like to port this feature to the 2.x branch.

dg commented

Yes, please send PR.

Well I looked into this issue the other day and found out it would require a deeper rework and a BC break without a doubt.

This is because in the SqlBuilder class, the "where" conditions are represented as an array of conditions, whereas the "having" conditions are represented as a string.

And since I know what the attitude to BC breaks (or general overhauls) is, I'd rather work on it once I know there will be support.

dg commented

Of course, this new feature should not cause a BC break…