Bug #52755 for Parse-Dia-SQL: FEATURE: Small Packages :columns stereotype

Mon Dec 14 08:59:58 2009 shemgp [...] gmail.com - Ticket created

Subject:	FEATURE: Small Packages :columns stereotype
Date:	Mon, 14 Dec 2009 21:59:35 +0800
To:	bug-Parse-Dia-SQL [...] rt.cpan.org
From:	shem p <shemgp [...] gmail.com>

Greetings, I would like to use the :columns stereo type in the Small Packages feature. It is currently NOTIMPL at the Parse-Dia-SQL-0.12 package, which I downloaded using cpan into an Ubuntu 9.10 i386 machine which has perl version 5.10.0. Do you have any advice on how to properly go about it? Or do I just hack away. Thanks --shemgp

Mon Dec 14 09:17:10 2009 aff [...] cpan.org - Correspondence added

On Mon Dec 14 08:59:58 2009, shemgp@gmail.com wrote: Show quoted text

> I would like to use the :columns stereo type in the Small Packages > feature.

Could you provide a sample dia file and describe the expected sql output? Do include name/version of OS, platform, perl, and dia used. Best regards, Andreas --

Mon Dec 14 09:17:11 2009 The RT System itself - Status changed from 'new' to 'open'

Tue Dec 15 00:41:58 2009 shemgp [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #52755] FEATURE: Small Packages :columns stereotype
Date:	Tue, 15 Dec 2009 13:41:40 +0800
To:	bug-Parse-Dia-SQL [...] rt.cpan.org
From:	shem p <shemgp [...] gmail.com>

On Mon, Dec 14, 2009 at 10:17 PM, Andreas Faafeng via RT <bug-Parse-Dia-SQL@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=52755 > > > On Mon Dec 14 08:59:58 2009, shemgp@gmail.com wrote: >

>> I would like to use the :columns stereo type in the Small Packages >> feature.

> > Could you provide a sample dia file and describe the expected sql > output? Do include name/version of OS, platform, perl, and dia used.

I've attached the dia file, the output of parsediasql, and the expected sql file, generated from tedia2sql. I'm expecting that the added small package, in the dia file, will add this line: UNIQUE( first, last ) after the last column and before the ')' in the create table statement. Btw, it doesn't also work if i use tedia2sql to convert a dia version 0.97 generated file, but it works for a 0.96.1 dia file. I'm using Ubuntu 9.10, x86, perl v5.10.0, and dia 0.97. Thanks, --shemgp

Download sp_columns.dia-0.97.dia
application/x-dia-diagram 1.3k

Message body not shown because it is not plain text.

Message body is not shown because sender requested not to inline it.

Wed Dec 16 04:55:55 2009 shemgp [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #52755] FEATURE: Small Packages :columns stereotype
Date:	Wed, 16 Dec 2009 17:55:39 +0800
To:	bug-Parse-Dia-SQL [...] rt.cpan.org
From:	shem p <shemgp [...] gmail.com>

Hello, I just noticed that postgres was not spelled correctly in the small package in the dia 0.97 testing file that i sent so I'm sending a corrected and improved version of it. I also tried to add columns support and I hope this patch can be a starting point. So far, it works for me using the command: parsediasql --file sp_columns.dia-0.97.dia --db postgres > sp_columns.dia-0.97.parsediasql_patched.sql Thanks, shemgp

Download sp_columns.dia-0.97.dia
application/x-dia-diagram 1.4k

Message body not shown because it is not plain text.

Message body is not shown because sender requested not to inline it.

Wed Dec 16 05:27:31 2009 shemgp [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #52755] FEATURE: Small Packages :columns stereotype
Date:	Wed, 16 Dec 2009 18:27:14 +0800
To:	bug-Parse-Dia-SQL [...] rt.cpan.org
From:	shem p <shemgp [...] gmail.com>

Hello again, Hope I'm not spamming this feature request but here's an updated patch that enables correct placing of column stereotypes on the proper create table sql statement, instead of lumping all and placing them all into each table. Thanks, shemgp

Download sp_columns.dia-0.97.dia
application/x-dia-diagram 1.6k

Message body not shown because it is not plain text.

Message body is not shown because sender requested not to inline it.

Thu Dec 17 14:09:09 2009 aff [...] cpan.org - Severity Normal added

Thu Dec 17 14:09:09 2009 aff [...] cpan.org - Broken in 0.12 added

Thu Dec 17 14:09:09 2009 aff [...] cpan.org - Broken in 0.12_01 added

Thu Dec 17 14:10:14 2009 aff [...] cpan.org - Correspondence added

On Wed Dec 16 05:27:31 2009, shemgp@gmail.com wrote: Show quoted text

> Hope I'm not spamming this feature request but here's an updated patch > that enables correct placing of column stereotypes on the proper > create table sql statement, instead of lumping all and placing them > all into each table.

Thanks for the patches. I'll have a look. Regards, Andreas --

Fri Dec 18 02:08:52 2009 aff [...] cpan.org - Correspondence added

Please check and see if the 0.12_02 release fixes this issue. Andreas --

Sat Dec 19 07:02:50 2009 shemgp [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #52755] FEATURE: Small Packages :columns stereotype
Date:	Sat, 19 Dec 2009 20:02:35 +0800
To:	bug-Parse-Dia-SQL [...] rt.cpan.org
From:	shem p <shemgp [...] gmail.com>

Just want to confirm that it 0.12_02 works for me as expected. Thanks

Sun Dec 20 07:11:01 2009 aff [...] cpan.org - Status changed from 'open' to 'resolved'

Sun Dec 20 07:11:02 2009 aff [...] cpan.org - Fixed in 0.12_02 added