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

Bug #12920 add new error info and fix escape method if connection not exist
Submitted: 2008-01-17 13:43 UTC
From: afz Assigned: quipo
Status: Closed Package: MDB2_Driver_pgsql
PHP Version: 5.2.5 OS:
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 14 - 3 = ?

 
 [2008-01-17 13:43 UTC] afz (Ali Fazelzade)
Description: ------------ hi, 1. when we use createDatabase method then escape method return empty for charset because caonnection not exist. 2. add new error info MDB2_ERROR_ALREADY_EXISTS, for "* already exists", this error needed when we try createDatabase that already exist. path for both above bug:

Comments

 [2008-01-17 13:43 UTC] afz (Ali Fazelzade)
Index: pgsql.php =================================================================== RCS file: /repository/pear/MDB2/MDB2/Driver/pgsql.php,v retrieving revision 1.188 diff -u -r1.188 pgsql.php --- pgsql.php 11 Jan 2008 18:48:29 -0000 1.188 +++ pgsql.php 18 Jan 2008 13:31:41 -0000 @@ -119,6 +119,8 @@ $native_msg = 'Database connection has been lost.'; $error_code = MDB2_ERROR_CONNECT_FAILED; } + } else { + $native_msg = @pg_last_error(); } static $error_regexps; @@ -130,6 +132,8 @@ => MDB2_ERROR_NOSUCHTABLE, '/index .* does not exist/' => MDB2_ERROR_NOT_FOUND, + '/database .* already exists/i' + => MDB2_ERROR_ALREADY_EXISTS, '/relation .* already exists/i' => MDB2_ERROR_ALREADY_EXISTS, '/(divide|division) by zero$/i' @@ -202,7 +206,7 @@ if (PEAR::isError($connection)) { return $connection; } - if (version_compare(PHP_VERSION, '5.2.0RC5', '>=')) { + if (is_resource($connection) && version_compare(PHP_VERSION, '5.2.0RC5', '>=')) { $text = @pg_escape_string($connection, $text); } else { $text = @pg_escape_string($text); @@ -1467,4 +1471,4 @@ return $result; } } -?> \ No newline at end of file +?>
 [2008-01-17 20:17 UTC] quipo (Lorenzo Alberton)
This bug has been fixed in CVS. 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.