Very excited about the existance of this project, I'd love to support it where I can.

As a start, I've installed XAMPP on my local maschine and Oracle 10g Express Edition, Drupal 7.x-dev and the latest version of the Oracle project. I followed the instructions in INSTALL.oracle.txt and created a settings.php with the following settings:

$databases = array (
  'default' => 
  array (
    'default' => 
    array (
      'driver' => 'oracle',
      'database' => 'system',
      'username' => 'drupal',
      'password' => '',
      'host' => 'localhost',
      'port' => '',
    ),
  ),
);
$db_prefix = 'test1_';

When I go to http://www.example.com/install.php I can select the install profile, select the language and next I get the screen to input database parameters, hence the defined settings are invalid.

Now, there are several notices and warnings displayed on that screen:

    *  Notice: Use of undefined constant LOG_LOCAL0 - assumed 'LOG_LOCAL0' in include_once() (line 17  of C:\Develop\Drupal7\includes\database\oracle\database.inc).
    * Warning: openlog() expects parameter 3 to be long, string given in include_once() (line 17 of C:\Develop\Drupal7\includes\database\oracle\database.inc).
    * Notice: Undefined index: distribution_name in drupal_install_profile_distribution_name() (line 191 of C:\Develop\Drupal7\includes\install.inc).
    * Notice: Undefined index: use_cache in DatabaseConnection_oracle->__construct() (line 73 of C:\Develop\Drupal7\includes\database\oracle\database.inc).

Underneath the first field in the form is Database type and the only two options I get are "MySQL" and "SQLite", no PostgreSQL and no Oracle.

Any idea what may be causing this?

Comments

aaaristo’s picture

aaaristo commented

it looks like on windows you get some problem... i just committed some patches..
you should use LOG_USER instead of LOG_LOCAL0 in database.inc
and add 'use_cache' => false to your conf like this:

$databases = array (
'default' =>
array (
'default' =>
array (
'driver' => 'oracle',
'database' => 'system',
'username' => 'drupal',
'password' => '',
'host' => 'localhost',
'port' => '',
'use_cache' => false
),
),
);
$db_prefix = 'test1_';

i've actually fixed those things in the last commit.
Don't use the $db_prefix unless you really need it.. Because you will need a more powerful oracle user to do that.
DB prefixes are translated in schemas.

For the rest, you should see the Oracle option in your list, try to check if now you get it.

Which release of the 7.x Drupal are you using?

Andrea

aaaristo’s picture

aaaristo commented

It would be great to have someone supporting the driver other than me so please feel free to submit patches when you find bugs, or ask for informations opening support requests / feature requests so that others can learn from your questions.

Andrea

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented

Hi @andrea, thanks for your support. I've downloaded your latest oracle module and also the latest Drupal 7.x-dev as of yesterday (before I used the dev version from April 11).

First of all, changing LOG_LOCAL0 to LOG_USER had to be done manually, I guess this should be changed in your code, shouldn't it?

After that I ran install.php again and still don't get Oracle as an option for database types. Instead I receive the following error/warnings:

    *  Notice: Undefined index: distribution_name in drupal_install_profile_distribution_name()  (line 191 of C:\Develop\Drupal7\includes\install.inc).
    * Strict warning: Declaration of DatabaseConnection_oracle::lastInsertId() should be compatible with that of PDO::lastInsertId() in Database::openConnection() (line 1510 of C:\Develop\Drupal7\includes\database\database.inc).
    * Strict warning: Declaration of DatabaseConnection_oracle::quote() should be compatible with that of PDO::quote() in Database::openConnection() (line 1510 of C:\Develop\Drupal7\includes\database\database.inc).
aaaristo’s picture

aaaristo commented

The last is 30 Apr... May be you downloaded it before the packaging system repackage...

Which version of PHP are you using? The errors that you are receiving looks like php 5.3 that is not yet supported (http://drupal.org/requirements).

And probably you don't get the oracle option because you do not have the pdo_oci extension active and working.

Try creating a phpinfo(); page and search for "PDO drivers" you should have 'oci' listed.

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented

You're right, the latest XAMPP came with PHP 5.3.1, I now changed back to XAMPP 1.7.1 which comes with PHP 5.2.9

Then I activated extension=php_pdo_oci.dll in the php.ini but that produces an error message because oci.dll can not be found. This oci.dll is obviously required by php_pdo_oci.dll but I have no idea where to get that dll from and where to put it then. Any ideas? Ah, yes, I'm on Windows with my test maschine.

aaaristo’s picture

aaaristo commented

You should install an oracle instant client:

http://www.oracle.com/technology/software/tech/oci/instantclient/index.html

-basic
-sdk

aaaristo’s picture

aaaristo commented
Category: bug » support

changing the type of the thread

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented
Category: support » bug

Perfect, I downloaded that and copied the "Instant Client Package - Basic Lite" and copied the DLL files into the apache/bin directory. Now I do have OCI up and running.

So back to the install.php in Drupal. Still getting lots of errors:

# Strict warning: Declaration of DatabaseConnection_oracle::lastInsertId() should be compatible with that of PDO::lastInsertId() in Database::openConnection()  (line 1510 of C:\Develop\Drupal7\includes\database\database.inc).
# Strict warning: Declaration of DatabaseConnection_oracle::quote() should be compatible with that of PDO::quote() in Database::openConnection() (line 1510 of C:\Develop\Drupal7\includes\database\database.inc).
# Strict warning: Declaration of DatabaseStatement_oracle::getStatement() should be compatible with that of DatabaseStatementPrefetch::getStatement() in require_once() (line 612 of C:\Develop\Drupal7\includes\database\oracle\database.inc).

Also, as I'm not quite familiar with Oracle administration yet, I'm not quite sure what to use for databse name, username and password. I tried several variations with no luck, so I'm keen to get some hints.

EDIT: When Drupal tries to connect to the database (during installation) the following error is reported:

Failed to connect to your Oracle database server. Oracle  reports the following message: SQLSTATE[HY000]: pdo_oci_handle_factory: ORA-12514: TNS:listener does not currently know of service requested in connect descriptor (ext\pdo_oci\oci_driver.c:581).
aaaristo’s picture

aaaristo commented
Category: bug » support

jurgenhaas, this is a support request there is actually no bug in what you are facing...

it look like you are still using php 5.3. You should not get those strict warnings in 5.2.

ORA-12514 means that drupal was able to connect to the port of your oracle listener, but you gave a wrong SID in the database name field... (you should use 'XE' for the Express Edition)
Or you instance is not registered with the listener...
try:
sqlplus drupal/drupal@host:port/xe

to see if you can connect with sqlplus.

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented

Appologize for setting tha category back to "bug report" in #8, that happened unintentionally because I had the form open in my browser while you correctly changed category to "support request" and then submitting the form used the previous value.

Back to the topic: the tip with "Datebase name" as XE did the trick, thanks a lot.

However, I'm now getting Fatal error: Call to undefined function _db_check_install_needed() in C:\Develop\Drupal7\includes\database\oracle\database.inc on line 166 but I got over that form, so connecting to the databse worked just fine.

aaaristo’s picture

aaaristo commented
Status: Active » Closed (fixed)
aaaristo’s picture

aaaristo commented

function _db_check_install_needed() should be defined in includes/database/database.inc... but i removed the call because it is pretty useless.