Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 354292

Summary: Actions, Handlers and Filters not shown @ Node Level
Product: z_Archived Reporter: Kamesh Sampath <kamesh.sampath>
Component: SapphireAssignee: Konstantin Komissarchik <konstantin>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: kamesh.sampath, konstantin, ling.hao
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard: kamesh
Attachments:
Description Flags
Patch v1
none
mylyn/context/zip
none
Patch v2
konstantin: iplog+
mylyn/context/zip none

Description Kamesh Sampath CLA 2011-08-09 15:07:43 EDT
Build Identifier: Eclipse 3.7 - Indigo -  Sapphire Latest Stream Build

The Action Handlers, and Filters are not shown at the Node level of the definition, though actions, handlers and filters could be added at any level of the UI 

Reproducible: Always

Steps to Reproduce:
1.Create an sdef file 
2.Create new part and add a section 
3.You will not see actions under the section
Comment 1 Konstantin Komissarchik CLA 2011-09-05 14:44:56 EDT
Note that the fix for this should be based on 0.3.x branch rather than HEAD.
Comment 2 Kamesh Sampath CLA 2011-09-06 04:52:11 EDT
Right now I am workin on the HEAD how do i switch t 0.3.x branch, when i tried Team ->Swith to another branch or version it not working it says "chosen operation is not enabled" it means I need to make my working copy in fully commited state before I can do that ?

I also need a small help, sometimes ite becoming really diffuclt for me to switch back and forth with versions as with my working copy I am not able to commit . Do you have any idea or suggestion to overcome this ? I see lot of eclipse projects are migrating to GIT , are we also planning to do that ?
Comment 3 Konstantin Komissarchik CLA 2011-09-06 23:52:05 EDT
I recommend that you setup a complete new environment for working on different branches. Your dev-eclipse/target need to be from the branch you are working on. At that point, you might as well have separate workspaces instead of trying to switch projects and configuration around.

> I also need a small help, sometimes ite becoming really diffuclt for me to
> switch back and forth with versions as with my working copy I am not able to
> commit . Do you have any idea or suggestion to overcome this ? 

Once you've exported the patch and attached it to a bug, you can revert your local changes if you need to work on another bug. You can always re-apply the patch when you are ready to resume.

> I see lot of eclipse projects are migrating to GIT , are we also planning to 
> do that ?

Git migration will have to happen at some point as that is the direction across Eclipse Foundation, but we have no immediate plans to do that with Sapphire. There have been a lot of pains in adopting Git across Eclipse and frankly I am content to let others work the kinks out.
Comment 4 Kamesh Sampath CLA 2011-09-08 14:42:22 EDT
(In reply to comment #3)
When I browse the CVS repository i see R_0_3_X, if am not wrong i need to use this source base for this fix. 

Correct me if I am wrong.
Comment 5 Konstantin Komissarchik CLA 2011-09-08 14:55:46 EDT
Yes, R_0_3_X is the branch to use.
Comment 6 Kamesh Sampath CLA 2011-09-09 01:18:54 EDT
(In reply to comment #5)
> Yes, R_0_3_X is the branch to use.
Can you please change the status to Assigned so that I can work on this bug in parllel to other bugs that I am working on 0_4_x(HEAD)
Comment 7 Kamesh Sampath CLA 2011-09-09 12:20:48 EDT
My understanding is that for this change i need to touch only one file DefinitionEditor.sdef. 

Correct me if am wrong and any other project or file that I might need touch.
Comment 8 Konstantin Komissarchik CLA 2011-09-09 13:07:06 EDT
> My understanding is that for this change i need to touch only one file
> DefinitionEditor.sdef. 

Yes, that should be all that's required. There is an existing definition for this triplet that you would need to reference in the right spot in sdef to resolve this.
Comment 9 Kamesh Sampath CLA 2011-09-09 13:15:01 EDT
Great! My Sapphire skills is improving i feel :)

yeah in my initial analysis i found the I need to insert ActionsOverallNode - which provides the Actions, Handlers and Filters UI part.  Am doing my second part of the analysis  where to put insert it  mostly here (MasterDetailsContentNodeFactoryCaseDef), let me try to put that right place to make it work and send you a patch for review and merge.

Also in this context I would like to provide a small suggestion to improve the documentation, some of the vital things like what is ContentFactoryNode, Content Factory , various presentation hints and how it will be useful and when/where to use. If these can get in much detail on to the document, that will increase the usability of the framework and will lessen your load of some basic questions in the forum which you might need to answer. 

Please excuse me if these are already addressed am just requoting them :(
Comment 10 Kamesh Sampath CLA 2011-09-10 03:00:58 EDT
Created attachment 203103 [details]
Patch v1

this fixes the issues for adding the action parts (actions, handlers, filters etc.,) and content outline node level for the Master Details Editor Page.
Comment 11 Kamesh Sampath CLA 2011-09-10 03:01:03 EDT
Created attachment 203104 [details]
mylyn/context/zip
Comment 12 Kamesh Sampath CLA 2011-09-10 03:02:37 EDT
From my analysis this bug fix is needed only at the places where the content outline node is present typicallu Master Details Page.

Let me know if you feel there are other palces where I need to apply this fix, if not please move the bug to next stage.
Comment 13 Konstantin Komissarchik CLA 2011-09-10 12:52:45 EDT
There are two places where this change is needed. You got one of them (under IMasterDetailsContentNodeDef). The other one is under IMasterDetailsContentNodeFactoryCaseDef (which itself is under IMasterDetailsContentNodeFactoryDef). 

What you have so far will cover statically-defined node. The other construct covers nodes that are dynamically created based on the contents of a list.
Comment 14 Kamesh Sampath CLA 2011-09-11 11:52:27 EDT
thanks for pointing and explaining the same . I have made the change and am attaching the new patch for your review and merge. 

Please review and once you are fine please move the bug to next stage and le me know if I can take the update for this branch.

If possible can you pelase put up a small WIKI article on the Sapphire SDEF elements and its usage ? e.g. IMasterDetailsContentNodeDef, IMasterDetailsContentNodeFactoryCaseDef, that will come handy for adopters and later we can merge in to the documentation as that will be very useful for the community.
Comment 15 Kamesh Sampath CLA 2011-09-11 11:55:28 EDT
Created attachment 203127 [details]
Patch v2

contains the addition of ActionsOverall Node at the two places wahre its required to be attached
Comment 16 Kamesh Sampath CLA 2011-09-11 11:55:34 EDT
Created attachment 203128 [details]
mylyn/context/zip
Comment 17 Konstantin Komissarchik CLA 2011-09-12 10:10:25 EDT
Patch committed to 0.3.1 and 0.4 streams.
Comment 18 Kamesh Sampath CLA 2011-09-12 13:32:53 EDT
thanks and i see the changes and updated my local R_0_3_x copy.
Comment 19 Ling Hao CLA 2011-09-28 19:46:18 EDT
Verify "Actions and Handlers" is added to content outline node, content outline node factory, and case.