New SQL classes

[DJ Maze]

I've added a new SQL library.
It tries to be compatible with the old system BUT is more strict when it comes to writing SQL queries.
Correct ANSI/ISO syntax is required to make the most cross-db implementation possible.

So, if you find any SQL errors that weren't there, here's your chance to make them ISO SQL-92 compliant.

New features:

• SQL result iteration and array access
• XML schema using XML XSD
• XML schema export
• Sync DB schema through XML schema
• Transaction management

Example:

PHP:

$result = $db->query("SELECT * FROM {$SQL->TBL->admins}"); $result->num_rows; foreach ($result as $index => $values) { // code here }

[NanoCaiordo]

PHP:

// To sync DB: $db->XML->sync_schema_from_file($file) // To create a XML export scheme header('Content-Type: application/xml'); $db->XML->export_schema(fopen('php://output', 'w')); exit;

[Phoenix]

A tip to existing CVS users - do not update.

CVS is severely broken as a result of this change and is not functional.

[DJ Maze]

Problem currently still relies in 4 methods:

• list_databases
• list_tables
• list_columns
• list_indexes

Working my ass off to get this incompatibility fixed.

[DJ Maze]

Install should work again.
Please inform when something is still fails at install (not after!)

[Phoenix]

Let's build the database gives the impression of nothing happening, which might prompt extra clicking of the button.

Unimpeded, it eventually breaks with:

Code::

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ') VALUES ('26','6','2000','Francisco Burzi registers the PHP-Nuke project at Sou' at line 1

core.xml history data set obviously hasn't converted properly - several fields have extra column(s) of data.
sql/data/core.xml fix committed (twice)

Code::

Fatal error: require() [function.require]: Failed opening required '/install/sql/data/core.php' (include_path='.:/includes:/usr/lib/php:/usr/local/lib/php') in /install/sql/new.php on line 20

[DJ Maze]

Thanks phoenix!

I've added the missing table data and added a unique index on security table to prevent duplicate entries.

[DJ Maze]

Just added XML support for PostgreSQL

[Phoenix]

Code::

Remove install.php and the install directory right now!

Database installed without error, but no forum or PM tables.

Unfortunately, it doesn't proceed to Gather important info & Create super admin account

[Phoenix]

New install now proceeds to completion

Post-install issues here

[movix]

Hello,

Just a question about those new SQL functionalities, would it be possible to have the FULL parameter added to the get_column method in order to retreive comments on the table columns.

i know that the request will be much longer to perform upon using the FULL option, so perhaps it can be optionally set on the get_clolumn method.

Hope the question is clear.

[DJ Maze]

movix wrote:

Just a question about those new SQL functionalities, would it be possible to have the FULL parameter added to the get_column method in order to retreive comments on the table columns.

There is no get_column.
Instead use:

Code::

$db->list_columns(table_name);

It will return a multidimensional array as:

Code::

array( 'column_name' => array( 'type' => (string), 'notnull' => (boolean), 'default' => (string), 'extra' => (string), 'comment' => (string) ), ...... )

[movix]

Hi DZ Maze,

Thanks for your reply.
Yes is is indeed list_columns.
Correct me, but this method did not returned the comment in 9.x

Movix

[DJ Maze]

movix wrote:

Correct me, but this method did not returned the comment in 9.x

Correct, it didn't.
Now it does as it is supported by our XML DB schema.xsd.
Due to that we can add comments to tables and columns in xml db schemas like core.xml

[movix]

what about perfomance when using the FULL option in the query to retrieve comments and extra ?
Is it worth to make this optional upon calling the method ?
For my part i noticed that the query tooks about 10 times longer when using the FULL option in the query.