This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 258648 - Missing icons from Explorer
Summary: Missing icons from Explorer
Status: CLOSED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 3.0.4   Edit
Hardware: PC Windows XP
: P3 major (vote)
Target Milestone: 3.0.4   Edit
Assignee: Dimitar Giormov CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard: PMC_approved
Keywords:
: 260583 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-12-12 07:39 EST by Angel Vera CLA
Modified: 2009-02-13 06:42 EST (History)
2 users (show)

See Also:
dimitar.giormov: pmc_approved?
dimitar.giormov: pmc_approved? (raghunathan.srinivasan)
dimitar.giormov: pmc_approved? (naci.dai)
dimitar.giormov: pmc_approved? (deboer)
dimitar.giormov: pmc_approved? (neil.hauge)
kaloyan: pmc_approved+
kaloyan: review+


Attachments
view of missing icons (72.71 KB, image/png)
2008-12-12 07:39 EST, Angel Vera CLA
no flags Details
fix (1.80 KB, patch)
2008-12-15 04:25 EST, Dimitar Giormov CLA
no flags Details | Diff
labels fixed. (1.33 KB, patch)
2008-12-19 04:58 EST, Dimitar Giormov CLA
no flags Details | Diff
screenshot (30.25 KB, image/png)
2009-01-15 13:14 EST, Kaloyan Raev CLA
no flags Details
servlet and servlet mappting. (724 bytes, patch)
2009-01-27 10:09 EST, Dimitar Giormov CLA
no flags Details | Diff
304 patch (1.48 KB, patch)
2009-02-02 07:30 EST, Dimitar Giormov CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Angel Vera CLA 2008-12-12 07:39:09 EST
Created attachment 120310 [details]
view of missing icons

The icons in the project explorer for the Dynamic Web project, deployment descriptor are missing. Refer to the image for more details.
Comment 1 Kaloyan Raev CLA 2008-12-12 09:57:44 EST
I can confirm this behavior on my installation too. 
Comment 2 David Williams CLA 2008-12-12 10:52:45 EST
I looked at this a little, to see if possibly related to my removal of org.eclipse.jem.ui (bug 257980). Doesn't seem to be, that is, the icons didn't live there, or anything. Just thought I'd note it here, in case there's some obscure connection. 
Comment 3 Kaloyan Raev CLA 2008-12-12 10:55:06 EST
It should not be about that icons are missing. 
There is something wrong with the content provider and/or the label provider. In the screenshot it can be seen that the DD tree nests into itself recursively. 
Comment 4 Dimitar Giormov CLA 2008-12-15 04:25:35 EST
Created attachment 120449 [details]
fix

here is a fix.
The problem was that the nodes in the tree were made IAdaptable and in the Content Providers the Root node was taken in really wrong way. I have fixed the root node resolve and the label provider fixed itself.
Comment 5 Dimitar Giormov CLA 2008-12-15 04:31:56 EST
committed and released.
Comment 6 Dimitar Giormov CLA 2008-12-19 04:55:33 EST
icons still missing, though node nesting has disappeared.
The interesting part is with build from last weeks on my laptop, but it should not. Since I have found that the declarations in the plugin.xml are old and not correct.
Comment 7 Dimitar Giormov CLA 2008-12-19 04:58:02 EST
Created attachment 120926 [details]
labels fixed.
Comment 8 David Williams CLA 2009-01-08 13:38:14 EST
Was this fixed in M4? Or should target be changed to M5? 
Comment 9 Angel Vera CLA 2009-01-09 14:25:51 EST
This is still a problem on the latest smoke driver for 3.1
Comment 10 Dimitar Giormov CLA 2009-01-12 09:49:56 EST
it was a wrong declaration in plugin xml, funny enough this should not work for more then 3 months.

I guess that another bug was not considering the possible children correctly and was fixed recently.

However it is fixed and released in 3.1
Comment 11 Dimitar Giormov CLA 2009-01-13 03:04:33 EST
*** Bug 260583 has been marked as a duplicate of this bug. ***
Comment 12 Kaloyan Raev CLA 2009-01-15 13:14:04 EST
"No label provider" is still displayed instead of the subnodes for Servlets and Servlet Mappings
Comment 13 Kaloyan Raev CLA 2009-01-15 13:14:39 EST
Created attachment 122701 [details]
screenshot
Comment 14 Dimitar Giormov CLA 2009-01-27 10:09:09 EST
Created attachment 123873 [details]
servlet and servlet mappting.

hopefully last one.
Comment 15 Dimitar Giormov CLA 2009-01-28 04:01:13 EST
commited and released.
Comment 16 Angel Vera CLA 2009-01-30 11:17:21 EST
I am reopening this bug and changing the version number to 3.0.4, as the same problem is now showing in this weeks smoke test. 
Comment 17 Dimitar Giormov CLA 2009-02-02 07:30:45 EST
Created attachment 124411 [details]
304 patch
Comment 18 Dimitar Giormov CLA 2009-02-02 07:36:32 EST
Chuck can you review the patch.
Comment 19 Dimitar Giormov CLA 2009-02-02 07:45:05 EST
The content and label providers are not functioning well since the declarations of possible children are wrong. The values from earlier refactoring are not updated. This was unnoticed most probably, because of a other bug that allowed not defined possible children to exist in the tree. The outcome is incomplete DD tree and missing icons.

There is no workaround for this issue.

The fix was manually tested.

The fix is being reviewed by Chuck.

the risk is low since the corrections are only in possible children clause in plugin xml.
Comment 20 Angel Vera CLA 2009-02-03 11:09:31 EST
Dimitar, is this only for 3.0.4, or does it also applies to 3.1?
Comment 21 Chuck Bridgham CLA 2009-02-03 13:49:53 EST
looks good - tested fine... approved
Comment 22 Dimitar Giormov CLA 2009-02-04 01:59:28 EST
Angel, this patch is already in 3.1, so this is only for 3.0.4
This wrong declarations are in for some time now, but they were working. With the new platform however "No label provider" is shown.
Comment 23 Kaloyan Raev CLA 2009-02-04 02:37:04 EST
Dimitar has somehow managed to mess the flags...
Chuck has approved review and David has approved by PMC. 
Comment 24 Dimitar Giormov CLA 2009-02-04 03:19:56 EST
committed and released in R3_0_maintenance 
Comment 25 Dimitar Giormov CLA 2009-02-13 06:42:09 EST
checked in latest 3.0.4