| 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) | ||
[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.