Skip to content

Commit

Permalink
Merge pull request #793 from yajra/fix-config
Browse files Browse the repository at this point in the history
fix: config structure
  • Loading branch information
yajra authored Jun 29, 2023
2 parents c7cdc13 + f581a16 commit ee66376
Show file tree
Hide file tree
Showing 4 changed files with 26 additions and 27 deletions.
1 change: 0 additions & 1 deletion src/Oci8/Oci8Connection.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@

use Doctrine\DBAL\Driver\OCI8\Driver as DoctrineDriver;
use Doctrine\DBAL\Version;
use Exception;
use Illuminate\Database\Connection;
use Illuminate\Database\Grammar;
use Illuminate\Support\Str;
Expand Down
44 changes: 22 additions & 22 deletions src/config/oracle.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,28 +2,28 @@

return [
'oracle' => [
'driver' => 'oracle',
'tns' => env('DB_TNS', ''),
'host' => env('DB_HOST', ''),
'port' => env('DB_PORT', '1521'),
'database' => env('DB_DATABASE', ''),
'service_name' => env('DB_SERVICE_NAME', ''),
'username' => env('DB_USERNAME', ''),
'password' => env('DB_PASSWORD', ''),
'charset' => env('DB_CHARSET', 'AL32UTF8'),
'prefix' => env('DB_PREFIX', ''),
'prefix_schema' => env('DB_SCHEMA_PREFIX', ''),
'edition' => env('DB_EDITION', 'ora$base'),
'driver' => 'oracle',
'tns' => env('DB_TNS', ''),
'host' => env('DB_HOST', ''),
'port' => env('DB_PORT', '1521'),
'database' => env('DB_DATABASE', ''),
'service_name' => env('DB_SERVICE_NAME', ''),
'username' => env('DB_USERNAME', ''),
'password' => env('DB_PASSWORD', ''),
'charset' => env('DB_CHARSET', 'AL32UTF8'),
'prefix' => env('DB_PREFIX', ''),
'prefix_schema' => env('DB_SCHEMA_PREFIX', ''),
'edition' => env('DB_EDITION', 'ora$base'),
'server_version' => env('DB_SERVER_VERSION', '11g'),
'load_balance' => env('DB_LOAD_BALANCE', 'yes'),
'max_name_len' => env('ORA_MAX_NAME_LEN', 30),
'dynamic' => [],
],
'sessionVars' => [
'NLS_TIME_FORMAT' => 'HH24:MI:SS',
'NLS_DATE_FORMAT' => 'YYYY-MM-DD HH24:MI:SS',
'NLS_TIMESTAMP_FORMAT' => 'YYYY-MM-DD HH24:MI:SS',
'NLS_TIMESTAMP_TZ_FORMAT' => 'YYYY-MM-DD HH24:MI:SS TZH:TZM',
'NLS_NUMERIC_CHARACTERS' => '.,',
'load_balance' => env('DB_LOAD_BALANCE', 'yes'),
'max_name_len' => env('ORA_MAX_NAME_LEN', 30),
'dynamic' => [],
'sessionVars' => [
'NLS_TIME_FORMAT' => 'HH24:MI:SS',
'NLS_DATE_FORMAT' => 'YYYY-MM-DD HH24:MI:SS',
'NLS_TIMESTAMP_FORMAT' => 'YYYY-MM-DD HH24:MI:SS',
'NLS_TIMESTAMP_TZ_FORMAT' => 'YYYY-MM-DD HH24:MI:SS TZH:TZM',
'NLS_NUMERIC_CHARACTERS' => '.,',
],
],
];
4 changes: 2 additions & 2 deletions tests/Database/Oci8ConnectorTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@ public function testOptionResolution()
/**
* @dataProvider tnsDataProvider
*
* @param $dsn
* @param $config
* @param $dsn
* @param $config
*/
public function testOracleConnectCallsCreateConnectionWithProperArguments($dsn, $config)
{
Expand Down
4 changes: 2 additions & 2 deletions tests/Database/OracleEloquentTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@ public function testNewBaseQueryBuilderReturnsOracleBuilderForOracleGrammar()
}

/**
* @param $model
* @param $database
* @param $model
* @param $database
*/
protected function mockConnectionForModel($model, $database)
{
Expand Down

0 comments on commit ee66376

Please sign in to comment.