| Subject: | Patch to make AUTO_INCREMENT work for MySQL |
| Date: | Wed, 6 Jan 2016 18:30:43 -0700 |
| To: | bug-Parse-Dia-SQL [...] rt.cpan.org |
| From: | "Douglas E. Miles" <doug [...] veritablesoftware.com> |
Hi,
This is the same patch in Bug #87327 along with a Dia file and test
file. I know this is way late, but for some reason, I never got to
follow-up emails. I'll also explain the change here: in tedia2sql,
putting 'NOT NULL AUTO_INCREMENT' in the Value field on the Attributes
tab generated the correct SQL. The current code on line 774 in Output.pm
overwrites 'NOT NULL AUTO_INCREMENT' with 'not null', thus not putting
the AUTO_INCREMENT into the generated SQL. The change I made to the line
will allow anything that starts with 'not null' (case insensitive) to be
passed through as is. Anything else is replaced with 'not null'. All the
existing tests still pass with this change in place. Please let me know
if you have any questions, and I'll try not to take 2 years to get back
to you. :)
Message body is not shown because sender requested not to inline it.
Message body is not shown because sender requested not to inline it.
Message body not shown because it is not plain text.