[dev] error in sql_file driver for PEAR

Amith Varghese amith at xalan.com
Tue Oct 14 16:03:12 PDT 2003


I just downloaded the VFS package from pear.php.net and noticed that on line 694
it states this

if (!isset($this->params[$val])) {

It however should be this:

if (!isset($this->_params[$val])) {

I generated a patch (might need to be adjusted for correct file path)

-------------- next part --------------
--- /tmp/sql_file.php	2003-10-14 19:00:14.000000000 -0400
+++ sql_file.php	2003-10-14 18:53:32.000000000 -0400
@@ -691,7 +691,7 @@
 
             $required = array('phptype', 'hostspec', 'username', 'password', 'database', 'vfsroot');
             foreach ($required as $val) {
-                if (!isset($this->params[$val])) {
+                if (!isset($this->_params[$val])) {
                     return PEAR::raiseError(sprintf(_("Required '%s' not specified in VFS configuration."), $val));
                 }
             }


More information about the dev mailing list