Method names confusing in play.db.DB

Bug #650851 reported by Michael Boyd
6
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 "executeWithResult".

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.