[freeside-commits] branch master updated. 4ee96719dd1665ef3694aa983cc9046b53e8cc5d

[Mark Wells]

The branch, master has been updated
       via  4ee96719dd1665ef3694aa983cc9046b53e8cc5d (commit)
      from  f1e4e2c83b33a2c6280073ced388a91742a3bb46 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 4ee96719dd1665ef3694aa983cc9046b53e8cc5d
Author: Mark Wells <mark at freeside.biz>
Date:   Fri Jun 13 12:48:02 2014 -0700

    but don't break Pg cursors in the process, #28895

diff --git a/FS/FS/Cursor.pm b/FS/FS/Cursor.pm
index 3af3c1b..f6d8816 100644
--- a/FS/FS/Cursor.pm
+++ b/FS/FS/Cursor.pm
@@ -77,13 +77,13 @@ sub new {
   }
 
   $sth->execute or die $sth->errstr;
-  # in mysql, make sure we're not holding any locks on the tables mentioned
-  # in the query; in Pg this will do nothing.
-  $dbh->commit;
 
   if ( driver_name() eq 'Pg' ) {
     $self->{fetch} = $dbh->prepare("FETCH FORWARD $buffer FROM ".$self->{id});
   } elsif ( driver_name() eq 'mysql' ) {
+    # make sure we're not holding any locks on the tables mentioned
+    # in the query
+    $dbh->commit if driver_name() eq 'mysql';
     $self->{fetch} = $dbh->prepare("SELECT * FROM $self->{id} ORDER BY rownum LIMIT ?, $buffer");
   }
 

-----------------------------------------------------------------------

Summary of changes:
 FS/FS/Cursor.pm |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)