Add index generation and change PK generation

[sarunast]

Dumping table with MySQL gave my duplicate Primary Keys so I changed the generation.
Also there was no Index generation so I managed to create function which can do that.

[luislobo]

THanks! I noticed it but handled it on the generated script by hand

[schmunk42]

Why should this be optional?

[sarunast]

I don't use SQLite so decided to disable it. Just makes migration cleaner.

[schmunk42]

Could you set the default to false so we don't break existing commands.

[schmunk42]

Does this still work for SQLite?

[sarunast]

I don't know about SQLite sorry.

[schmunk42]

Is there anything which would break backward compatibility or SQLite support? I left a comment at the code line.

[schmunk42]

This if should be moved to line 301. I got a sytnax error unexpected endif when I ran a test.
Could you also verify the changeset by dumping a rather complex MySQL schema and then insert it into SQLite?

[schmunk42]

Got an error when dumping a table without PK, eg. AuthAssignment

*** applying m130715_184724_dump

    > create table AuthAssignment ...Exception: CDbCommand failed to execute the SQL statement: CDbCommand failed to prepare the SQL statement: SQLSTATE[HY000]: General error: 1 near ")": syntax error. The SQL statement executed was: CREATE TABLE 'AuthAssignment' (
    "itemname" varchar(64) NOT NULL,
    "userid" varchar(64) NOT NULL,
    "bizrule" text,
    "data" text,
    PRIMARY KEY ()
)  

Shouldn't output the last line I think.

[sarunast]

schmunk42 feel free to update however you want it, I don't mind. Here is the rep https://github.com/Stamy/database-command. It does completely everything for MySQL, Full FK generation, multiple-keys.

[schmunk42]

I'll have a detailed look, when I find more time. Thanks.