Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.7.3

Request #17647 Change some MDB2 query() calls to exec()
Submitted: 2010-08-02 16:27 UTC
From: mckenna Assigned: till
Status: Closed Package: HTTP_Session2 (version 0.7.3)
PHP Version: 5.3.1 OS: Win7
Roadmaps: 0.8.0    
Subscription  


 [2010-08-02 16:27 UTC] mckenna (Bernhard Mayer)
Description: ------------ Some calls to query() of the MDB2 PEAR Package should be changed to use exec(). Especially the calls with a Query that uses INSERT, UPDATE or DELETE to retrieve the affected rows. Sorry no patch file possible. As an example in the mdb2.php container file. At line 245 of write method, change this line: $result = $this->db->query($query); to this line: $result = $this->db->exec($query); The result will contain the number of affected rows or a PEAR Error. Other methods are destroy, gc and replicate.

Comments

 [2010-09-03 18:15 UTC] till (Till Klampaeckel)
-Status: Open +Status: Verified -Assigned To: +Assigned To: till -Roadmap Versions: +Roadmap Versions: 0.8.0
 [2010-09-03 18:21 UTC] till (Till Klampaeckel)
-Status: Verified +Status: Feedback
Please provide feedback: http://svn.php.net/viewvc?view=revision&revision=303027
 [2010-09-03 18:23 UTC] till (Till Klampaeckel)
Just to make sure: To try this: svn co http://svn.php.net/repository/pear/packages/HTTP_Session2/trunk ./HTTP_Session2-trunk/ cd HTTP_Session2-trunk/ sudo pear install -f package.xml ;-)
 [2011-03-11 23:47 UTC] till (Till Klampaeckel)
-Status: Feedback +Status: Closed
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.