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

Bug #798 Auth_Container_DB::_connect doesn't not take care of optional parameters w/Patch
Submitted: 2004-02-19 14:05 UTC
From: tacker Assigned: yavo
Status: Closed Package: Auth
PHP Version: Irrelevant OS: Irrelevant
Roadmaps: (Not assigned)    
Subscription  


 [2004-02-19 14:05 UTC] tacker
Description: ------------ It should connect with optional parameters. Important for poartability. http://pear.php.net/manual/en/package.database.db.db.connect.php

Comments

 [2004-02-19 14:17 UTC] tacker
? Auth.diff Index: Container/DB.php =================================================================== RCS file: /repository/pear/Auth/Container/DB.php,v retrieving revision 1.42 diff -u -r1.42 DB.php --- Container/DB.php 23 Nov 2003 12:56:36 -0000 1.42 +++ Container/DB.php 19 Feb 2004 14:21:00 -0000 @@ -92,7 +92,7 @@ function _connect($dsn) { if (is_string($dsn) || is_array($dsn)) { - $this->db = DB::Connect($dsn); + $this->db = DB::Connect($dsn, $this->options['db_options']); } elseif (get_parent_class($dsn) == "db_common") { $this->db = $dsn; } elseif (DB::isError($dsn)) { @@ -177,6 +177,7 @@ $this->options['dsn'] = ''; $this->options['db_fields'] = ''; $this->options['cryptType'] = 'md5'; + $this->options['db_options'] = array(); } // }}}
 [2004-03-28 23:18 UTC] yavo at php dot net
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.