Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 348000 - [TMF] Filter viewer does not update with new custom parser fields
Summary: [TMF] Filter viewer does not update with new custom parser fields
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: LinuxTools (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Francois Chouinard CLA
QA Contact: Francois Chouinard CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 301610
  Show dependency tree
 
Reported: 2011-06-01 15:25 EDT by Patrick Tasse CLA
Modified: 2022-01-13 14:53 EST (History)
0 users

See Also:


Attachments
Proposed patch in org.eclipse.linuxtools.tmf.ui package (5.56 KB, patch)
2011-06-01 15:27 EDT, Patrick Tasse CLA
jjohnstn: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Tasse CLA 2011-06-01 15:25:58 EDT
Build Identifier: 

Submitting a patch to fix a problem with the filter viewer.

When the custom parser wizard is used to create a new custom parser or edit the field labels of a custom parser, the applied changes are not reflected in the filter viewer.

Since the filter viewer compiles the list of field labels and event types at class loading and stores them in static variables, any changes made by the user to the custom parsers will not be available until the workbench is restarted.

The solution is to regenerate those lists every time in the constructor of the filter node composite. They will then reflect the latest changes made by the user.

Reproducible: Always

Steps to Reproduce:
1. Open the Filters view to set the labels and event types.
2. Create a new custom parser from the Manage Custom Parsers dialog.
3. Add at least one group (Text) or element (XML) in the custom parser.
4. Create a new filter in the Filters view or the Colors view.
5. Add a node to the filter which contains a "field" parameter.
6. Verify that the new custom parser field is available in the combo button.
7. Add an "event type" node to the filter.
8. Verify that the new custom parser type is available in the combo button.
Comment 1 Patrick Tasse CLA 2011-06-01 15:27:18 EDT
Created attachment 197154 [details]
Proposed patch in org.eclipse.linuxtools.tmf.ui package

Legal Message: I, Patrick Tasse, declare that I developed attached code from
scratch, without referencing any 3rd party materials except material licensed
under the EPL. I am authorized by my employer to make this contribution under
the EPL.
Comment 2 Patrick Tasse CLA 2012-09-06 17:55:36 EDT
This change was committed and delivered in 0.10.0.