- Restored "active rowcount" fetch before ResultProxy

autocloses the cursor.  This was removed in 0.5rc3.

1 files changed, 8 insertions, 2 deletions

diff --git a/lib/sqlalchemy/engine/base.py b/lib/sqlalchemy/engine/base.py
index 832903a73..d22e21cfe 100644
--- a/lib/sqlalchemy/engine/base.py
+++ b/lib/sqlalchemy/engine/base.py
@@ -1382,7 +1382,10 @@ class ResultProxy(object):
     
     @property
     def rowcount(self):
-        return self.context.get_rowcount()
+        if self._rowcount is None:
+            return self.context.get_rowcount()
+        else:
+            return self._rowcount
 
     @property
     def lastrowid(self):
@@ -1395,10 +1398,13 @@ class ResultProxy(object):
     def _init_metadata(self):
         metadata = self.cursor.description
         if metadata is None:
-            # no results, close
+            # no results, get rowcount (which requires open cursor on some DB's such as firebird),
+            # then close
+            self._rowcount = self.context.get_rowcount()
             self.close()
             return
             
+        self._rowcount = None
         self._props = util.PopulateDict(None)
         self._props.creator = self.__key_fallback()
         self.keys = []