-
-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[NFR]: Add method to Mvc/Model/CriteriaInterface #14818
Comments
Also you can use
|
Yes, I agree. And method query (Phalcon/Mvc/Model) return CriteriaInterface query(\Phalcon\Di\DiInterface $container = null): CriteriaInterface |
What IDE are you using? |
I use Intellij IDEA, with your plugin Phalcon 4 (thanks @ruudboon) Today I'm overloading method query in IDEA to return Criteria after query() This code works Model::query()
->columns('var2, var3')//return CriteriaInterface
->where('var = :var:')//return CriteriaInterface
->bind([
'var' => Model::VAR_YES
])//return CriteriaInterface Not this one Model::query()
->where('var = :var:')//return CriteriaInterface
->bind([
'var' => Model::VAR_YES
])//return CriteriaInterface
->columns('var2, var3')//Error function not exists on IDE |
Same on Visual Studio Code. |
Tracked in #14608 |
Is your feature request related to a problem? Please describe.
When I create a query, the IDE never shows me the "columns" method
Describe the solution you'd like
So I checked the "CriteriaInterface" class, and I don't see this function
So I compared all the functions between "CriteriaInterface" and "Criteria" and found some differences:
So I don't know, if we need to add all this functions. I think we need to add "columns" and "join" (even if we have leftJoin and innerJoin)
The text was updated successfully, but these errors were encountered: