PEAR Bug Search Results

MDB2_Driver_sqlsrv: Feature/Change Request 20010 [Open] Charset option in dsn

dastolfo at smart dot it — 2013-07-15T16:08:34+00:00

MDB2_Driver_sqlsrv Feature/Change Request
Reported by dusty_it
2013-07-15T16:08:34+00:00
PHP: Irrelevant OS: Windows Package Version: 1.5.0b5

Description:
------------
I'm working on porting an application to Windows + SQL Server as DB 
Backend. The driver works, I only found that specifying the "charset" 
option with sqlsrv driver raise an error, because the
setCharset function is not defined in sqlsrv driver.

As far as I know, it's not possible to change runtime the charset like 
in mysql, so I think it should suffice to set the option in $params 
array before the call to sqlsrv_connect and comment out the call to 
setCharset. Here follows a patch that I made:

*** sqlsrv.php    2013-07-15 11:40:42.000000000 +0200
--- sqlsrv_patched.php    2013-07-15 11:40:38.000000000 +0200
*************** class MDB2_Driver_sqlsrv extends MDB2_Dr
*** 330,335 ****
--- 330,339 ----
           if ($database) {
               $params['Database'] = $database;
           }
+
+         if (!empty($this->dsn['charset'])) {
+             $params['CharacterSet'] = $this->dsn['charset'];
+         }

           if ($this->dsn['port'] && $this->dsn['port'] != 1433) {
               $host .= ','.$this->dsn['port'];
*************** class MDB2_Driver_sqlsrv extends MDB2_Dr
*** 344,356 ****
               $this->connected_database_name = $database;
           }

-         if (!empty($this->dsn['charset'])) {
-             $result = $this->setCharset($this->dsn['charset'], 
$connection);
-             if (MDB2::isError($result)) {
-                 return $result;
-             }
-         }
-
          if (empty($this->dsn['disable_iso_date'])) {
              @sqlsrv_query($connection,'SET DATEFORMAT ymd');
          }
--- 348,353 ----

MDB2_Driver_sqlsrv: Bug 19737 [Open] fetchrow with MDB2_FETCHMODE_OBJECT

ghada at hpfm dot com dot au — 2012-12-09T14:16:31+00:00

MDB2_Driver_sqlsrv Bug
Reported by ghada
2012-12-09T14:16:31+00:00
PHP: 5.3.8 OS: Windows 7 Package Version: 1.5.0b5

Description:
------------
Get the following error when fetching a row with MDB2_FETCHMODE_OBJECT:

Fatal error: Cannot use object of type stdClass as array in C:\xampp\php\PEAR\MDB2.php on line 1828


This error was worked around using code below:

Line 1824:
        case MDB2_PORTABILITY_RTRIM:
            foreach ($row as $key => $value) {
                if (is_string($value)) {
                    $row[$key] = rtrim($value);
                }
            }
            break;

Should be:

        case MDB2_PORTABILITY_RTRIM:
            foreach ($row as $key => $value) {
                if (is_string($value)) {
                    if (is_array($row))
                    {
                        $row[$key] = rtrim($value);
                    }
                    else
                    {
                        rtrim($row->$key);
                    }
                }
            }
            break;


Other issues still exist in MDB2 - with inserts.  Not sure if related.

Test script:
---------------
$sql = "SELECT col1, col2
		FROM T1";

$result = $dbConn->query($sql);
if (MDB2::isError($result))
{
	die("error");
}

$row =& $result->fetchRow(MDB2_FETCHMODE_OBJECT);	// generates error.


		




Expected result:
----------------
no errors.

Actual result:
--------------
Fatal error: Cannot use object of type stdClass as array in C:\xampp\php\PEAR\MDB2.php on line 1828

MDB2_Driver_sqlsrv: Feature/Change Request 19425 [Open] Add SSL option/functionality

wade at hedgren dot org — 2012-05-18T22:27:01+00:00

MDB2_Driver_sqlsrv Feature/Change Request
Reported by hedroom
2012-05-18T22:27:01+00:00
PHP: 5.3.12 OS: Windows Package Version: 1.5.0b4

Description:
------------
I would like to propose that the ssl option (in the MDB2 documentation) be enabled to turn on encryption for the sqlsrv client.

I have uploaded a patch that has the code.

This would allow encrypted connections to MS-SQL servers that are setup to allow encrypted connections but are not forced...

Thank you,

Wade Hedgren

MDB2_Driver_sqlsrv: Bug 18524 [Open] MDB2_PORTABILITY_FIX_CASE doesn't have any effect

nick at deltamotion dot com — 2012-04-01T20:09:24+00:00

MDB2_Driver_sqlsrv Bug
Reported by nickmc
2011-05-11T00:49:22+00:00
PHP: 5.3.6 OS: Windows Package Version: 1.5.0b3

Description:
------------
MDB2_PORTABILITY_FIX_CASE doesn't seem to do anything. 
I noticed this 
when I switched from the mssql driver to sqlsrv (the only 
change in my code to 
create the problem), and all my associative arrays ended up 
with undefined 
indexes because they now contained mixed case rather than 
being entirely 
lowercase.

I dug around a bit in MDB2\Driver\sqlsrv.php, and I found that 
the check for 
($fetchmode == MDB2_FETCHMODE_ASSOC) before applying 
MDB2_PORTABILITY_FIX_CASE was false although I specified 
MDB2_FETCHMODE_ASSOC. I checked the value for 
$fetchmode and it was 
MDB2_FETCHMODE_DEFAULT.

Adding the following code immediately before that check 
seemed to fix the 
problem, but I'm not sure whether it's the proper way to fix 
the problem, or if 
there are other implications. This is from fetchRow().

if($fetchmode == MDB2_FETCHMODE_DEFAULT)
   $fetchmode = $this->db->fetchmode;


The relevant section of code, altogether, is:
if($fetchmode == MDB2_FETCHMODE_DEFAULT)
			$fetchmode = $this->db->fetchmode;
		
        if ($fetchmode == MDB2_FETCHMODE_ASSOC && 
is_array($row) && 
$this->db->options['portability'] & 
MDB2_PORTABILITY_FIX_CASE) {
			$row = array_change_key_case($row, $this-
>db-
>options['field_case']);
        }

Test script:
---------------
$mdb2 =& MDB2::factory('sqlsrv://user:pass@server/Database');
$mdb2->setFetchMode(MDB2_FETCHMODE_ASSOC);
$mdb2->setOption('field_case', CASE_LOWER);
$mdb2->setOption('portability', MDB2_PORTABILITY_ALL ^ MDB2_PORTABILITY_EMPTY_TO_NULL); 

$statement = $mdb2->prepare('select SomeRow from table');
$result = $statement->execute();

$data = $result->fetchAll();
var_dump($data);

Expected result:
----------------
// this is the output of the above script. notice that 
// SomeRow is not lowercase
array(1) {
  [0]=>
  array(2) {
    ["SomeRow "]=>
    string(4) "1234"
    [0]=>
    string(4) "1234"
  }
}

Actual result:
--------------
array(1) {
  [0]=>
  array(2) {
    ["somerow"]=>
    string(4) "1234"
    [0]=>
    string(4) "1234"
  }
}