Bug 148759 - [package explorer] empty JarPackageFragment nodes fail to get filtered in hierarchical layout of Package Explorer
Summary: [package explorer] empty JarPackageFragment nodes fail to get filtered in hie...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.2   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.2.1   Edit
Assignee: JDT-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 106678 (view as bug list)
Depends on:
Blocks: 106678
  Show dependency tree
 
Reported: 2006-06-26 23:47 EDT by Mik Kersten CLA
Modified: 2010-06-15 10:10 EDT (History)
4 users (show)

See Also:


Attachments
screenshot of filtering problem (11.14 KB, image/gif)
2006-06-26 23:50 EDT, Mik Kersten CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mik Kersten CLA 2006-06-26 23:47:54 EDT
The comment on PackageExplorerPart.createViewer is: 

	 * This viewer ensures that non-leaves in the hierarchical
	 * layout are not removed by any filters.

While this may make sense for many filters, it is preventing Mylar users from using the hierarchical layout because empty nodes end up being shown (see attachment).  Could this rule be updated to "filter *empty* non-leaf" nodes instead, where determining "empty" alllows all the filters applied to the viewer to run?  (Interestingly we have the same rule for Mylar's Task List.)
Comment 1 Mik Kersten CLA 2006-06-26 23:50:40 EDT
Created attachment 45356 [details]
screenshot of filtering problem

Mylar's interest filter determines that "internal" and the 5 nodes under it should be not be shown, but they are forced in.
Comment 2 Martin Aeschlimann CLA 2006-06-27 12:59:48 EDT
I have to find out why this rule was introduced for the hierarchical mode.
Comment 3 Mik Kersten CLA 2006-06-27 15:46:50 EDT
I was wondering that too... perhaps if you have the following structure: one.two.Foo and end up with:
  one
    + two
        + Foo
Then a filter that is intended to filter empty JarPackageFragment's will think that "one" is empty because it has no classes in it.
Comment 4 Mik Kersten CLA 2006-08-01 14:57:16 EDT
Any news on this?  This blocks the single "known limitation" of Mylar, which is currently our longest standing and most-voted open bug (only 3 votes, but unlike enhancements we don't tend to keep bugs with votes alive very long).

  http://wiki.eclipse.org/index.php/Mylar_FAQ#Key_Limitations
Comment 5 Martin Aeschlimann CLA 2006-08-05 12:17:37 EDT
fixed > 20060805

Tested that changing this isn't a performance problem.
If we get some more experience with this I would also be ready to fix that in 3.2.1.
Comment 6 Dani Megert CLA 2006-08-06 10:44:00 EDT
>Tested that changing this isn't a performance problem.
Can you point me to the tests that prooves this? How many items and levels of nesting (i.e. package segments) were part of your tests?
Comment 7 Martin Aeschlimann CLA 2006-08-07 04:23:02 EDT
I ran a profiler on a bigger project. Note that I also improve the hierarchical content provider performance for 3.2.
Comment 8 Mik Kersten CLA 2006-08-25 16:07:19 EDT
*** Bug 106678 has been marked as a duplicate of this bug. ***
Comment 9 Mik Kersten CLA 2006-08-25 16:19:16 EDT
Excellent--I just verified and this gets rid of the limitation in Mylar's interest filtering (comment#4) since the filtering is now consistent .  Sorry for the slow reply, vacation time.  And a pleasant surprise is that while I never used the Hierarchical presentation before Mylar because it forced too many clicks, I'm now finding it be the preferred mode since interest filtering and expansion removes the need for those clicks.

Are you considering fixing this in 3.2.1?
Comment 10 Martin Aeschlimann CLA 2006-08-28 08:10:17 EDT
I misspelled 'candidate', that's why I lost track of the bug.

fix released in 3.2.1 > 20060828
Comment 11 Martin Aeschlimann CLA 2006-08-31 09:56:04 EDT
verified in 3.2.1 M20060830-0800. Mik, can you verify as well?
Comment 12 Mik Kersten CLA 2006-09-03 04:32:37 EDT
Traveling now, will plan to do so mid next week.
Comment 13 Mik Kersten CLA 2006-12-04 15:45:58 EST
This problem now appears for the Project Explorer when in hierarchical mode, with identical symptoms.  Should I open a new bug or shall we use this one to discuss?
Comment 14 Martin Aeschlimann CLA 2006-12-05 03:02:24 EST
A new bug would be better as this one is verified fixed for 3.2.1
Comment 15 Mik Kersten CLA 2006-12-05 18:11:54 EST
Done: bug 166872
Comment 16 Missing name CLA 2010-06-15 09:59:15 EDT
I still have this issue in 3.5.2
Comment 17 Dani Megert CLA 2010-06-15 10:10:58 EDT
>I still have this issue in 3.5.2
Please try again using the latest 3.6 candidate:
http://download.eclipse.org/eclipse/downloads/drops/I20100608-0911/index.php
and if you still see the problem, open a new bug report with steps to reproduce.