Comment on my ambivalence about db.one

liberapay · Aug 10, 2013 · 0a258b7 · 0a258b7
1 parent 325abc6
commit 0a258b7
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/postgres.py b/postgres.py
@@ -353,6 +353,11 @@ def one_or_zero(self, sql, parameters=None):
 
 
     def one(self, sql, parameters=None, strict=None):
+
+        # I'm half-considering dropping this. Now that one_or_zero exists, this
+        # is really only useful for what should really be called db.first, and
+        # in that case why aren't you using a LIMIT 1?
+
         """Execute a query and return a single result.
 
         :param unicode sql: the SQL statement to execute