Bug 197484 - SystemView does not provide IContextObject for filter levels deeper than 1
Summary: SystemView does not provide IContextObject for filter levels deeper than 1
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 2.0   Edit
Hardware: All All
: P1 major (vote)
Target Milestone: 2.0.1   Edit
Assignee: Martin Oberhuber CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks: 197550
  Show dependency tree
 
Reported: 2007-07-23 09:46 EDT by Tobias Schwarz CLA
Modified: 2011-05-25 07:53 EDT (History)
2 users (show)

See Also:
mober.at+eclipse: pmc_approved+
dmcknigh: review+


Attachments
provide IContextObject for all children (1.73 KB, patch)
2007-07-23 10:15 EDT, Tobias Schwarz CLA
mober.at+eclipse: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tobias Schwarz CLA 2007-07-23 09:46:35 EDT
Build ID: rse 2.0

in our own subsystem, when refreshing a filter with folders, the content of the folders isn't filtered.
getChildren() of the element adapter doesn't get a IContextObject.
the context is only provided for the first filter level (in our subsystem the folder).

i located the problem inside SystemView.doOurInternalRefresh() and internalRSERefreshStruct(), where the input for the refresh of the children wasn't translated to an IContextObject.

i attach a patch, that solves this problem.
Comment 1 Tobias Schwarz CLA 2007-07-23 10:15:29 EDT
Created attachment 74357 [details]
provide IContextObject for all children

Legal Message:
I, Tobias Schwarz, 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 Martin Oberhuber CLA 2007-07-23 10:27:29 EDT
Patch looks good to me.
I committed it since we urgently need this:
[197484] Fix for SystemView to provide ContextObject for queries on all levels

DaveM can you please review it, thanks.
Comment 3 David McKnight CLA 2007-07-23 10:42:25 EDT
Looks okay to me.
Comment 4 Martin Oberhuber CLA 2007-07-23 11:02:32 EDT
Released for I20070723