Adapter name and sequence problems #3695

Merged
merged 4 commits into from Apr 16, 2013

Projects

None yet

3 participants

@eminetto
Contributor
eminetto commented Feb 6, 2013

Adapter name and sequence problems

@ralphschindler ralphschindler commented on the diff Feb 15, 2013
library/Zend/Db/Adapter/Adapter.php
@@ -329,6 +329,7 @@ protected function createPlatform($parameters)
case 'SqlServer':
return new Platform\SqlServer($options);
case 'Oracle':
+ case 'Oci':
@ralphschindler
ralphschindler Feb 15, 2013 Member

Where is Oci being returned from?

@romulobusatto
romulobusatto Feb 15, 2013 Contributor

when I create a connection to the driver=>Pdo and dns=>oci this is the return.
Example:
'db' => array(
'driver'=> 'Pdo',
'dsn' => 'oci:dbname=example/example;',
)

@ralphschindler
ralphschindler Mar 13, 2013 Member

this is fixed in a separate PR, basically PDO will return 'Oracle' when the pdo driver is 'oci'.
Technically, PDO oracle is not officially supported, its on my todo list.

@ralphschindler ralphschindler added a commit that referenced this pull request Apr 16, 2013
@ralphschindler ralphschindler PR #3695
Merge branch 'eminetto-fix_oracle'
f0e4056
@ralphschindler ralphschindler merged commit cfe24a8 into zendframework:master Apr 16, 2013

1 check failed

default The Travis build could not complete due to an error
Details
@ralphschindler ralphschindler added a commit that referenced this pull request Apr 16, 2013
@ralphschindler ralphschindler Forward #3695
Merge branch 'eminetto-fix_oracle' into develop
8180d10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment