Skip Menu |

This queue is for tickets about the Parse-Dia-SQL CPAN distribution.

Report information
The Basics
Id: 66031
Status: resolved
Priority: 0/
Queue: Parse-Dia-SQL

People
Owner: Nobody in particular
Requestors: felix.ostmann [...] thewar.de
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: dia changed naming from aggregate_a/b to assoc_type
in my dia there is no aggregate_a/b anymore, if i change the aggregate in dia the xml-node named assoc_type change his enum value, so aggregate_a/b changed to assoc_type i am not sure about adding the value to $rightEnd{aggregate} or $leftEnd{aggregate}, but i think it is not important. Here is a assoc from my current dia: $ dia -v Dia-Version 0.97.1, kompiliert 22:05:25 Mar 30 2010 <dia:object type="UML - Association" version="2" id="O18"> <dia:attribute name="name"> <dia:string>#fk_domains_domain_informations#</dia:string> </dia:attribute> <dia:attribute name="direction"> <dia:enum val="1"/> </dia:attribute> <dia:attribute name="show_direction"> <dia:boolean val="false"/> </dia:attribute> <dia:attribute name="assoc_type"> <dia:enum val="2"/> </dia:attribute> <dia:attribute name="role_a"> <dia:string>#domain_id#</dia:string> </dia:attribute> <dia:attribute name="multipicity_a"> <dia:string>#0..*#</dia:string> </dia:attribute> <dia:attribute name="visibility_a"> <dia:enum val="3"/> </dia:attribute> <dia:attribute name="show_arrow_a"> <dia:boolean val="false"/> </dia:attribute> <dia:attribute name="role_b"> <dia:string>#domain_id#</dia:string> </dia:attribute> <dia:attribute name="multipicity_b"> <dia:string>#1..1#</dia:string> </dia:attribute> <dia:attribute name="visibility_b"> <dia:enum val="3"/> </dia:attribute> <dia:attribute name="show_arrow_b"> <dia:boolean val="false"/> </dia:attribute> # ... positions ... </dia:object>
Subject: Parse-Dia-SQL-0.17_1.patch
diff -r -u Parse-Dia-SQL-0.17/lib/Parse/Dia/SQL.pm Parse-Dia-SQL-0.17.patched/lib/Parse/Dia/SQL.pm --- Parse-Dia-SQL-0.17/lib/Parse/Dia/SQL.pm 2011-02-16 11:23:11.000000000 +0100 +++ Parse-Dia-SQL-0.17.patched/lib/Parse/Dia/SQL.pm 2011-02-22 11:03:39.230760456 +0100 @@ -1239,6 +1239,8 @@ $leftEnd{aggregate} = $self->{utils}->get_string_from_node($currentNode); } elsif ( $nodeAttrName eq 'aggregate_b' ) { $rightEnd{aggregate} = $self->{utils}->get_string_from_node($currentNode); + } elsif ( $nodeAttrName eq 'assoc_type' ) { + $leftEnd{aggregate} = $self->{utils}->get_num_from_node($currentNode); } elsif ( $nodeAttrName eq 'class_scope_a' ) { $leftEnd{class_scope} = $self->{utils}->get_string_from_node($currentNode); } elsif ( $nodeAttrName eq 'class_scope_b' ) {
From: felix.ostmann [...] thewar.de
Still very important :-/
Subject: Re: [rt.cpan.org #66031] AutoReply: dia changed naming from aggregate_a/b to assoc_type
Date: Tue, 14 Jun 2011 13:09:39 +0200
To: bug-Parse-Dia-SQL [...] rt.cpan.org
From: felix.ostmann [...] thewar.de
No time for this bug? :-/ Have to fix that after every install :-/ Zitat von Bugs in Parse-Dia-SQL via RT <bug-Parse-Dia-SQL@rt.cpan.org>: Show quoted text
> > Greetings, > > This message has been automatically generated in response to the > creation of a trouble ticket regarding: > "dia changed naming from aggregate_a/b to assoc_type", > a summary of which appears below. > > There is no need to reply to this message right now. Your ticket has been > assigned an ID of [rt.cpan.org #66031]. Your ticket is accessible > on the web at: > > https://rt.cpan.org/Ticket/Display.html?id=66031 > > Please include the string: > > [rt.cpan.org #66031] > > in the subject line of all future correspondence about this issue. To do so, > you may reply to this message. > > Thank you, > bug-Parse-Dia-SQL@rt.cpan.org > > ------------------------------------------------------------------------- > in my dia there is no aggregate_a/b anymore, if i change the aggregate > in dia the xml-node named assoc_type change his enum value, so > > aggregate_a/b changed to assoc_type > > i am not sure about adding the value to $rightEnd{aggregate} or > $leftEnd{aggregate}, but i think it is not important. > > > Here is a assoc from my current dia: > > $ dia -v > Dia-Version 0.97.1, kompiliert 22:05:25 Mar 30 2010 > > <dia:object type="UML - Association" version="2" id="O18"> > <dia:attribute name="name"> > <dia:string>#fk_domains_domain_informations#</dia:string> > </dia:attribute> > <dia:attribute name="direction"> > <dia:enum val="1"/> > </dia:attribute> > <dia:attribute name="show_direction"> > <dia:boolean val="false"/> > </dia:attribute> > <dia:attribute name="assoc_type"> > <dia:enum val="2"/> > </dia:attribute> > <dia:attribute name="role_a"> > <dia:string>#domain_id#</dia:string> > </dia:attribute> > <dia:attribute name="multipicity_a"> > <dia:string>#0..*#</dia:string> > </dia:attribute> > <dia:attribute name="visibility_a"> > <dia:enum val="3"/> > </dia:attribute> > <dia:attribute name="show_arrow_a"> > <dia:boolean val="false"/> > </dia:attribute> > <dia:attribute name="role_b"> > <dia:string>#domain_id#</dia:string> > </dia:attribute> > <dia:attribute name="multipicity_b"> > <dia:string>#1..1#</dia:string> > </dia:attribute> > <dia:attribute name="visibility_b"> > <dia:enum val="3"/> > </dia:attribute> > <dia:attribute name="show_arrow_b"> > <dia:boolean val="false"/> > </dia:attribute> > # ... positions ... > </dia:object> >
On Tue Feb 22 05:11:21 2011, Sadrak wrote: Show quoted text
> in my dia there is no aggregate_a/b anymore, if i change the aggregate > in dia the xml-node named assoc_type change his enum value, so
Hi Sadrak/Felix: Thank you for your bug report. I understand your frustration, but I don't understand the problem fully. It would be helpful if you can send me a complete .dia file with a model containing an association illustrating the issue, as well as the actual and the wanted output from PDS. Thank you. Best regards, Andreas --
From: felix.ostmann [...] thewar.de
Here is a short patch with a test (for uml => 1) I copied 951-rt50906.t and only changed the relation (added the multiplicity? and removed the ON DELETE CASCADE) From the docu about tedia2sql (http://tedia2sql.tigris.org/usingtedia2sql.html): UML mode Whichever side is aggregate or composition is the 'one' side in the one-to-many relationship. The multiplicity of the other side may be specified in the usual way. If the relationship is aggregate, a on delete set NULL is added to the foreign key constraint in databases that support it; if it is a composition, then on delete cascade is added instead.
Subject: Parse-Dia-SQL-0.18.patch

Message body is not shown because it is too large.

From: felix.ostmann [...] thewar.de
btw: in uml-mode, the relationship names are generated and not used from the dia ... perhaps you find the failure: NORMAL MODE: ... add constraint fk_detail_main ... VS UML MODE: ... add constraint tbl_detail_fk_Fk_main ... The first is correct.
From: felix.ostmann [...] gmail.com
Did you see the attached .dia from Di 14. Jun 2011, 11:09:19? The problem still exists :-/ i patch every installation here ... no way to get this running correctly? Am Di 14. Jun 2011, 11:11:51, Sadrak schrieb: Show quoted text
> btw: in uml-mode, the relationship names are generated and not used from > the dia ... perhaps you find the failure: > > NORMAL MODE: > ... add constraint fk_detail_main ... > VS UML MODE: > ... add constraint tbl_detail_fk_Fk_main ... > > The first is correct.
On Mon Dec 16 09:35:11 2013, felix.ostmann@gmail.com wrote: Show quoted text
> The problem still exists :-/ i patch every installation here ... no > way to get this running correctly?
Hi Felix Adding the 2 following lines makes other tests fail: } elsif ( $nodeAttrName eq 'assoc_type' ) { $leftEnd{aggregate} = $self->{utils}->get_num_from_node($currentNode); Test Summary Report ------------------- t/950-rt51433.t t/951-rt50906.t t/962-rt57842-postsgres-int.t Could you please look into that? Thanks Andreas
Merged 92ae16e3, released 0.26. Thanks.