[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
RE: [bpel-dev] patch6=add-rename-design-element-affect-source-tab
|
Done: https://bugs.eclipse.org/bugs/show_bug.cgi?id=197468
>-----Original Message-----
>From: bpel-dev-bounces@xxxxxxxxxxx [mailto:bpel-dev-bounces@xxxxxxxxxxx] On
>Behalf Of Simon D Moser
>Sent: Monday, July 23, 2007 2:45 PM
>To: BPEL Designer project developer discussions.
>Cc: BPEL Designer project developer discussions.; bpel-dev-bounces@xxxxxxxxxxx
>Subject: RE: [bpel-dev] patch6=add-rename-design-element-affect-source-tab
>
>Hi Vitaly,
>
>one big "collection" defect sounds great! It's probably faster then putting
>something up on the site, too, so go ahead and file one!
>
>Cheers
>Simon
>
>Simon Moser, M.Eng.
>
>
>
> Websphere Integration Mail: IBM Deutschland Entwicklung
> Developer Development smoser@xxxxxx. GmbH
> Team Lead BPEL Editor com Vorsitzender des
> Dept. 4722, Bldg. Phone: Aufsichtsrats: Martin Jetter
> 71032-01, Room 086 +49-7031-16-43 Geschäftsführung: Herbert
> Websphere Solutions and 04 Kircher
> Services Fax: Sitz der Gesellschaft:
> IBM Deutschland +49-7031-16-48 Böblingen
> Entwicklung GmbH 90 Registergericht: Amtsgericht
> Schönaicherstr. 220, D - Stuttgart, HRB 243294
> 71032 Boeblingen
>
>
>
>
>
>
>
>
>
>
> "Tishkov, Vitaly
> V"
> <vitaly.v.tishkov To
> @intel.com> "BPEL Designer project developer
> Sent by: discussions."
> bpel-dev-bounces@ <bpel-dev@xxxxxxxxxxx>
> eclipse.org cc
> <bpel-dev-bounces@xxxxxxxxxxx>
> Subject
> 07/23/2007 12:41 RE: [bpel-dev]
> PM patch6=add-rename-design-element-af
> fect-source-tab
>
> Please respond to
> "BPEL Designer
> project developer
> discussions."
> <bpel-dev@eclipse
> .org>
>
>
>
>
>
>
>Hi Simon,
>
>Is there any good place on the Eclipse BPEL site to store this TODO list?
>I'm afraid that it will get lost in the mail sooner or later.
>
>I'd still propose to file a bug but not for this one problem but for all
>the problems related to our branch. The bug synopsis summary could look
>like "TODO list for the MultiTab-DomFacade branch".
>
>What do you think?
>
>Thanks,
> Vitaly.
>
>>-----Original Message-----
>>From: bpel-dev-bounces@xxxxxxxxxxx [mailto:bpel-dev-bounces@xxxxxxxxxxx]
>On
>>Behalf Of Simon D Moser
>>Sent: Monday, July 23, 2007 2:29 PM
>>To: BPEL Designer project developer discussions.
>>Cc: BPEL Designer project developer discussions.;
>bpel-dev-bounces@xxxxxxxxxxx
>>Subject: RE: [bpel-dev] patch6=add-rename-design-element-affect-source-tab
>>
>>Hi Vitaly,
>>
>>looks good - I am happy with the solution. I think that we wont need to
>>file a bug yet, since this is only applicable (yet) to the DOMFacade
>>Branch, and the bugs so far all relate to the main-branch, but we should
>>keep it on our TODO list
>>
>>Cheers
>>Simon
>>
>>
>>Simon Moser, M.Eng.
>>
>>
>>
>> Websphere Integration Mail: IBM Deutschland Entwicklung
>> Developer Development smoser@xxxxxx. GmbH
>> Team Lead BPEL Editor com Vorsitzender des
>> Dept. 4722, Bldg. Phone: Aufsichtsrats: Martin Jetter
>> 71032-01, Room 086 +49-7031-16-43 Geschäftsführung: Herbert
>> Websphere Solutions and 04 Kircher
>> Services Fax: Sitz der Gesellschaft:
>> IBM Deutschland +49-7031-16-48 Böblingen
>> Entwicklung GmbH 90 Registergericht: Amtsgericht
>> Schönaicherstr. 220, D - Stuttgart, HRB 243294
>> 71032 Boeblingen
>>
>>
>>
>>
>>
>>
>>
>>
>>
>>
>> "Tishkov, Vitaly
>> V"
>> <vitaly.v.tishkov To
>> @intel.com> "BPEL Designer project developer
>> Sent by: discussions."
>> bpel-dev-bounces@ <bpel-dev@xxxxxxxxxxx>
>> eclipse.org cc
>> <bpel-dev-bounces@xxxxxxxxxxx>
>> Subject
>> 07/23/2007 11:12 RE: [bpel-dev]
>> AM patch6=add-rename-design-element-af
>> fect-source-tab
>>
>> Please respond to
>> "BPEL Designer
>> project developer
>> discussions."
>> <bpel-dev@eclipse
>> .org>
>>
>>
>>
>>
>>
>>
>>Hi Simon,
>>
>>My answer for the first question is "yes and no" :)
>>
>>If we make it public then we won't need to write such inner classes.
>>But in order to use it activity2xml() from outside BPELWriter the newly
>>added constructor BPELWriter(BPELResource resource, Document document)
>must
>>be called - it does some initializations usually made in
>write(BPELResource
>>resource, OutputStream out, Map args).
>>
>>On another hand, we'll probably need to change the signature of write()
>>method in the future because we won't need to create the DOM from EMF but
>>just serialize DOM which we're to have synchronized with EMF.
>>
>>So, I'd propose the following now: leave activity2xml() protected now and
>>file a bug "remove InsertInContainerCommand.MyBPELWriter after
>>BPELWriter.write() is refactored".
>>
>>Does it look reasonable?
>>
>>Cheers
>>Vitaly.
>>
>>>-----Original Message-----
>>>From: bpel-dev-bounces@xxxxxxxxxxx [mailto:bpel-dev-bounces@xxxxxxxxxxx]
>>On
>>>Behalf Of Simon D Moser
>>>Sent: Monday, July 23, 2007 11:49 AM
>>>To: BPEL Designer project developer discussions.
>>>Cc: BPEL Designer project developer discussions.;
>>bpel-dev-bounces@xxxxxxxxxxx
>>>Subject: RE: [bpel-dev]
>patch6=add-rename-design-element-affect-source-tab
>>>Importance: High
>>>
>>>Hi Vitaly,
>>>
>>>I see. However, I am asking myself whether increasing the visibility to
>>>'public' you help here ?
>>>I think that could potentially save us from carrying all these inner
>>>classes just to get access (where in the first place I understand why
>>>activity2xml was protected in the past, however, I am not so sure this is
>>>the right qualifier for the future?)
>>>Opinions ?
>>>
>>>Cheers
>>>Simon
>>>
>>>Simon Moser, M.Eng.
>>>
>>>
>>>
>>> Websphere Integration Mail: IBM Deutschland Entwicklung
>>> Developer Development smoser@xxxxxx. GmbH
>>> Team Lead BPEL Editor com Vorsitzender des
>>> Dept. 4722, Bldg. Phone: Aufsichtsrats: Martin Jetter
>>> 71032-01, Room 086 +49-7031-16-43 Geschäftsführung: Herbert
>>> Websphere Solutions and 04 Kircher
>>> Services Fax: Sitz der Gesellschaft:
>>> IBM Deutschland +49-7031-16-48 Böblingen
>>> Entwicklung GmbH 90 Registergericht: Amtsgericht
>>> Schönaicherstr. 220, D - Stuttgart, HRB 243294
>>> 71032 Boeblingen
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> "Tishkov, Vitaly
>>> V"
>>> <vitaly.v.tishkov
>To
>>> @intel.com> "BPEL Designer project developer
>>> Sent by: discussions."
>>> bpel-dev-bounces@ <bpel-dev@xxxxxxxxxxx>
>>> eclipse.org
>cc
>>> <bpel-dev-bounces@xxxxxxxxxxx>
>>>
>Subject
>>> 07/23/2007 09:22 RE: [bpel-dev]
>>> AM
>patch6=add-rename-design-element-af
>>> fect-source-tab
>>>
>>> Please respond to
>>> "BPEL Designer
>>> project developer
>>> discussions."
>>> <bpel-dev@eclipse
>>> .org>
>>>
>>>
>>>
>>>
>>>
>>>
>>>Hi Simon,
>>>
>>>>I reviewed the code and I am happy with it - only I haven't fully
>>>>understood the purpose of the myBPELWriter inner class that you define
>in
>>>>the commands that you changed - maybe you can explain a word w.r.t. that
>>>?!
>>>
>>>I need to access
>>>org.eclipse.bpel.model.resource.BPELWriter.activity2XML()
>>>which is protected and thus inaccessible from
>>>org.eclipse.bpel.ui.commands.InsertInContainerCommand
>>>
>>>So, I extended BPELWriter and made activity2XML() public. This trick is
>>>often used in the project (e.g. see BPELEditor.setGraphicalViewer()).
>>>
>>>Does it explains the need of MyBPELWriter?
>>>
>>>Thanks,
>>> Vitaly.
>>>
>>>>-----Original Message-----
>>>>From: bpel-dev-bounces@xxxxxxxxxxx [mailto:bpel-dev-bounces@xxxxxxxxxxx]
>>>On
>>>>Behalf Of Simon D Moser
>>>>Sent: Sunday, July 22, 2007 8:34 PM
>>>>To: BPEL Designer project developer discussions.
>>>>Cc: BPEL Designer project developer discussions.;
>>>bpel-dev-bounces@xxxxxxxxxxx
>>>>Subject: Re: [bpel-dev]
>>patch6=add-rename-design-element-affect-source-tab
>>>>
>>>>Hi Vitaly,
>>>>
>>>>first: I am deeply impressed!! This is great work so far!
>>>>I tried the patch on a sanity check, and it seems to work - although I
>>>>found some more bugs - e.g.: don't try to undo deleted things etc, but
>>>that
>>>>is just normal at that stage of the game - actually, I expected to see
>>>>things more fragile then they actually are !!
>>>>
>>>>I reviewed the code and I am happy with it - only I haven't fully
>>>>understood the purpose of the myBPELWriter inner class that you define
>in
>>>>the commands that you changed - maybe you can explain a word w.r.t. that
>>>?!
>>>>Either way, I have committed the code - so you can get it from the known
>>>>branch ;-)
>>>>
>>>>Cheers
>>>>Simon
>>>>
>>>>
>>>>Simon Moser, M.Eng.
>>>>
>>>>
>>>>
>>>> Websphere Integration Mail: IBM Deutschland Entwicklung
>>>> Developer Development smoser@xxxxxx. GmbH
>>>> Team Lead BPEL Editor com Vorsitzender des
>>>> Dept. 4722, Bldg. Phone: Aufsichtsrats: Martin
>Jetter
>>>> 71032-01, Room 086 +49-7031-16-43 Geschäftsführung: Herbert
>>>> Websphere Solutions and 04 Kircher
>>>> Services Fax: Sitz der Gesellschaft:
>>>> IBM Deutschland +49-7031-16-48 Böblingen
>>>> Entwicklung GmbH 90 Registergericht:
>Amtsgericht
>>>> Schönaicherstr. 220, D - Stuttgart, HRB 243294
>>>> 71032 Boeblingen
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> "Tishkov, Vitaly
>>>> V"
>>>> <vitaly.v.tishkov
>>To
>>>> @intel.com> "BPEL Designer project developer
>>>> Sent by: discussions."
>>>> bpel-dev-bounces@ <bpel-dev@xxxxxxxxxxx>
>>>> eclipse.org
>>cc
>>>>
>>>>
>>Subject
>>>> 07/20/2007 07:15 [bpel-dev]
>>>> PM
>>patch6=add-rename-design-element-af
>>>> fect-source-tab
>>>>
>>>> Please respond to
>>>> "BPEL Designer
>>>> project developer
>>>> discussions."
>>>> <bpel-dev@eclipse
>>>> .org>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>Hi Simon,
>>>>
>>>>Please find attached the 6th patch for the MultiTab-DomFacade branch.
>>>>
>>>>The first step towards real EMF and DOM models synchronization is done:
>>>>- renaming existent elements and adding new ones in the design tab now
>>>>reflect on the DOM model and the source tab contents.
>>>>
>>>>Classes added.
>>>> 1. org.eclipse.bpel.ui.BPELModelAdapter.
>>>> 2. org.eclipse.bpel.ui.BPELModelReconcileAdapter.
>>>> 3. org.eclipse.bpel.ui.util.BPELEditorUtil.
>>>> 4. org.eclipse.bpel.ui.util.BPELNodeAssociationManager.
>>>>All these classes were created from the corresponding wsdl project
>>>classes.
>>>>There is quite a lot of garbage in the code. I decided not to clean it
>up
>>>>until the next patch.
>>>>
>>>>New dependencies.
>>>> 1. org.eclipse.bpel.model now depends on org.eclipse.wst.xml.core and
>>>> org.elipse.wst.sse.core.
>>>> 2. org.eclipse.bpel.ui now depends on org.eclipse.wst.xsd.ui and
>>>> org.eclipse.wst.wsdl.ui
>>>>
>>>>
>>>>Known bugs and limitations.
>>>> 1. Do not edit source in the source tab and then save the file.
>>Changes
>>>> in the DOM model don't affect EMF model.
>>>> 2. Elements are added into the DOM model (and thus in the source tab)
>>>> without the name attribute.
>>>> 3. Renaming just added elements in the design tab doesn't affect the
>>>> corresponding nodes in the source tab until the file is saved.
>>>> 4. The source tab content is not formatted after adding new elements
>>in
>>>> the design tab.
>>>>
>>>>Thanks,
>>>> Vitaly.(See attached file:
>>>>patch6=add-rename-design-element-affect-source-tab.txt)
>>>>_______________________________________________
>>>>bpel-dev mailing list
>>>>bpel-dev@xxxxxxxxxxx
>>>>https://dev.eclipse.org/mailman/listinfo/bpel-dev
>>>_______________________________________________
>>>bpel-dev mailing list
>>>bpel-dev@xxxxxxxxxxx
>>>https://dev.eclipse.org/mailman/listinfo/bpel-dev
>>_______________________________________________
>>bpel-dev mailing list
>>bpel-dev@xxxxxxxxxxx
>>https://dev.eclipse.org/mailman/listinfo/bpel-dev
>_______________________________________________
>bpel-dev mailing list
>bpel-dev@xxxxxxxxxxx
>https://dev.eclipse.org/mailman/listinfo/bpel-dev