el_to should be varchar in oracle
authorRiver Tarnell <river@users.mediawiki.org>
Wed, 28 Mar 2007 18:26:28 +0000 (18:26 +0000)
committerRiver Tarnell <river@users.mediawiki.org>
Wed, 28 Mar 2007 18:26:28 +0000 (18:26 +0000)
check return from oci_new_descriptor

includes/DatabaseOracle.php
maintenance/ora/tables.sql

index 7560039..20aff7f 100644 (file)
@@ -331,7 +331,11 @@ class DatabaseOracle extends Database {
                        }
                }
 
-               $bval = oci_new_descriptor($this->mConn, OCI_D_LOB);
+               if (($bval = oci_new_descriptor($this->mConn, OCI_D_LOB)) === false) {
+                       $e = oci_error($stmt);
+                       throw new DBUnexpectedError($this, "Cannot create LOB descriptor: " . $e['message']);
+               }
+
                if (strlen($returning))
                        oci_bind_by_name($stmt, ":bval", $bval, -1, SQLT_BLOB);
 
index f8ffc55..7273436 100644 (file)
@@ -163,7 +163,7 @@ CREATE INDEX cl_sortkey     ON categorylinks (cl_to, cl_sortkey);
 
 CREATE TABLE externallinks (
   el_from   INTEGER  NOT NULL  REFERENCES page(page_id) ON DELETE CASCADE,
-  el_to     CLOB     NOT NULL,
+  el_to     VARCHAR(2048) NOT NULL,
   el_index  CLOB     NOT NULL
 );
 -- XXX CREATE INDEX externallinks_from_to ON externallinks (el_from,el_to);