Improvements to MySQL producers foreign key and comment handling

Fix mysql_table_type .. (where did this go, or why was it not there?)

Fixed handling of NULLs.

Fixed a problem with the trigger_steps where it used to expect a string when it was getting an array.

Add test to prove insert statements with quoted semi-colons can still be skipped

Allow skipped insert statements and trigger bodies to contain quoted semi-colons

SQLite  and YAML tests broken, fixed
Also made SQLite parser actually used the parsed table naem

Typo fixing

Add tests for new alter_field, add_field, drop_field methods

Update Trigger to insist on a valid table for on_table
on_table now stores a known table obj in the table() attribute, which can also be used directly
Updated tests to match

Produce either a list of statements or a string
Split produce() into multiple methods
Add alter_field, add_field, drop_field for use with Diff

Document producer methods

Recognise & skip DROP statements when parsing
Recognise CURRENT_TIMESTAMP (SQLite v3+)

Added quote-field-names, quote-table-names options

Producers can now return individual statements as s list, if wantarray set
DB2 implements this, produces each individual statement on demand as well
Also implements alter_field, add_field etc for use by Diff
Added quote-field-names, quote-table-names params

More DB2 producing

DB2 producer tests

Ignore Oracle system constraints on source DB schema, also

Modified parse() to instantiate a separate RecDescent parser like other Parsers do

Added tests to catch Oracle problem with parser being reused

Removed conflict markers

Use DB2 Parser

Initial devision of sqlt-diff: Most of the functionality into a module

Added use of graph.

Commiting.

Fixed up ON DELETE parsing for FKs

Added ON DELETE and ON UPDATE clauses to FK output

Added 'default null' parsing

Fix field default value output bug I introduced

Minor refactor to avoid uninitialized value warning

Fixed preservation of function call as field in function-based indices

Parser changes to accomodate function-based indexes and optional size modifier for data type

Removed SIGNATURE.

Removing.

Globals filter now copies extra

new: makes tables in latex format

Applying the patch submitted with this bug report
https://rt.cpan.org/Public/Bug/Display.html?id=18096 by Nigel Metheringham.

Fixes a bug where it wouldn't parse create statements that included "timestamp without time zone".

Added tests for "unique" and "key" qualifiers to field definitions.

Added grammar for handling "unique" and "key" qualifiers to field definition.

Added constraint copy to Globals filter. (Seem to be getting lots of warnings from YAML now...)

Fixed warning for tables with no order attrib

-O::Is, accidentally committed some debug, thanks to Darren Chamberlain [dlc@sevenroot.org] for pointing it out

adding in Oracle driver support and making the DRIVER keys alphabetical :)

initial adds for the oracle dbi parser and a simple test to make sure the use works

Added Globals filter.

Removed un-used stuff left from CutNPaste of t/36-filters.t

Backed out M::B ConfigData based installed templates and moved DiaUml
producer to using evil hack of templates in INC.

Updates.

Changed to build up schema from "show create table" statements and then
pass to regular MySQL parser.

Changed my name.

Changes to make more efficient use of memory with large tables.

Moved YAML to a build requirement.

Just committing, but is it necessary?  It will be regenerated by Build.PL.

Intial code for Dia producer

Build support for installing templates.

Fixed missing schema table comments

Experimental filters

Added mysql_character_set for 4.1+

Applied changes submitted by Paul Makepeace <sourceforge.net@paulm.com>.

His notes:

I have added the --use-same-auth switch to provide the producers with the same DSN/user/password as the parser gets. Since these options sort of existed already for the Dumper producer so I renamed them to more generic --producer-X; now they all have them.

I also added a "--package My::Class::DBI" option for specifying the base class. One thing I wasn't quite sure on was that this base class is being stripped out - confused me at first but it seems to have been taken out in CVS v the version that's on CPAN -- maybe something in the works? Anyway, I put it back just for now as I needed it for my current project :-)

(Wasn't particularly excited by either of those switch names; feel free to come up with something better...)

E.g.,

perl -Mlib=blib/lib bin/sqlt --parser DBI –dsn   'dbi:mysql:dbname=X;host=127.0.0.1' --db-user X --db-password X --use-same-auth --producer ClassDBI –package 'My::Class::DBI'

refining the parser, but it still doesn't do everything it should:

- the column order number doesn't make much sense to me--is that number global to the file, or supposed to be relative to the table?

- the data_type and size aren't getting set; perhaps I need to do a mapping of the text output of the query to internal variables?

- is_unique never gets set for any column (because I don't explicitly set it), but the unique indexes and the columns they are over are set correctly.  Is that a problem?  Also, is the is_unique flag only for the simple case for where the column by it self is unique or is it also used for compound unique indexes?

revamped Pg DBI parser to query system tables.  This still doesn't work
as far as I know, but it seems very close.  Perhaps more knowledgable
eye could take a look.

produce DB2 tables

Removed doap.rdf from MANIFEST, since I removed it from CVS a few days earlier.

Moved to htdocs module

Original artwor, courtesy of Jason Williams

Adding non-zero exit status if differences were found

Various tweaks to produce Oracle diffs

Added some constraint tests
Removed YAML prereq, sqlt-diff doesn't use it anymore

Modified equals() to stringify field checks and use more case insensitivity when required

Modified equals() to include more case insensitivity when required

Modified equals() to include case insensitive data type checking and removed is_unique check (better caught by a constraint check)

Modified equals() to include type check, stringify field checks, and more case insensitivity when required

Use the overload::StrVal method to check if references are the same instead of the possibly overloaded stringify

Added negative sign possibility to default number values.
Modified constraints to not create a name if a name is already given.
Commented out PK name creation altogether (and would prefer to do it to all constraints, as well).
Made sure not to create UNIQUE constraints identical to the PRIMARY KEY; Oracle will create the unique constraint automatically when it creates the PK.

Modified "create anything else" rule to not include table and index creates so any syntax errors in those are properly caught
Added RUN rule
Re-enabled constraint states except that hideous monstrosity of a "using index" clause
Modified two word data types to return as two words so it comes out right from the Producer
Added float data type

Removing foreign key check from field equality; foreign key inequality is better caught with the constraint check

It helps if you use the correct version of the case-insensitive field name you're trying to return

Fixed case-insensitivity matching for SQL Server and field names

Removed auto_increment from a field's extra attribute; it's already saved in is_auto_increment and this prevents it unnecessarily showing up in the field's comment when producing MySQL

Removed '#' and '--' comments from being included in table comments; they aren't part of the schema and hose the schema diffing

Re-arranged to put null/not null before default value; looks and parses better

missed a spot

Removed the line that declared YAML to be the producer.  This stopped the YAML warnings from ./Build test and didn't seem to affect anything else.

backing out non-callback package mangling

Changed the second "my $h" to just "$h" because it was throwing warnings.

If a DSN is defined but no 'from' db type given, we infer the 'from' type is DBI

changes to allow subclass tables identical to superclass

Made some changes suggested by Michael Slattery to fix table level comments.  Also added in field level comments.

His remarks follow:

Table level comments weren't populating.  The empty tag is generated in XML, but not populated.  $table->comments() doesn't have a get_comments method, so that explains my if/else around $meth

Right now it produces 'comment' data elements with no attributes in the 'comments' tag.  If this is a bad idea please let me know.

Fixed usage docs.

The short arg for "font-size" was colliding with the "f" for the parser.

Modified test text to reflect that fields() now return empty strings instead of undef when there are no fields

Made sure defined scalar references are passed to _compare_objects(). Removed is_valid comparisons from equals().

Slight adjustment to parsing of identity/auto-increment field qualifier

Fixed bug to exhaustively search for equal constraints and indices
Modified how SQL Server drops a default value constraint on a field

Fixed bug in equals() to exhaustively search sets of indices and constraints

Modified equals() to get scalar references to fields for comparison

Modfied fields() to return empty array when there are no fields
Modified equals() to compare field names, not objects

Removed Data::Compare from requirements list

Modified _compare_objects() to use ref_compare from Class::MakeMethods::Utility::Ref instead of Data::Compare. It seems more reliable and comes in a package we already require.

Changed references to on_delete_do to on_delete and on_update_do to on_update.  I tried to keep it backwards compatible.