Method names confusing in play.db.DB
Bug #650851 reported by
Michael Boyd
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
play framework |
New
|
Undecided
|
Unassigned |
Bug Description
In the class play.db.DB, there are two method called "execute" and "executeQuery". The first returns a boolean whether the database says the query was successful, so should be used in UPDATE, INSERT and DELETE queries. The second returns a result set, so should be used in SELECT queries.
This is confusing in my opinion, because technically the names mean the same thing, but the methods behave differently.
I think better names would be "execute" (no change) and "executeWithRes
To post a comment you must log in.