Michal Sniatala
Michal Sniatala
It seems that this setting is only related to the model, so adding this setting to the database configuration doesn't seem right to me. I would rather see an additional...
We could use this new config to handle other things in a model, like: https://github.com/codeigniter4/CodeIgniter4/pull/8455
@neznaika0 Please check #8112 when you have time.
I wonder about the overall usefulness of this helper method... I'm not entirely convinced, but if others see potential in it, I won't be opposed.
@ddevsr I do not doubt that you have a use case for this, but I'm not sure if this is such a common problem to add it to the framework.
+1 for binds.
TBH, I have never created or worked with table names that contain special characters. Will the same problem occur if we try to create a normal query with our Model...
Cases like this are already supported: ```php $builder->orderBy('year IS NULL', '', false)->orderBy('year', 'asc'); ```
@amigoscs I don't think allowing the `IS NULL` and `IS NOT NULL` as valid parameters would be a good decision. Every DB engine supports the case you're dealing with differently....
@amigoscs In general, we don't accept features that would have support in only one DB engine. And allowing different valid `$direction` values for each engine doesn't make sense either. What...