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

Bug 324675

Summary: GroupFilterMappingItemProvider ignores servlet filter mappings
Product: [WebTools] WTP Java EE Tools Reporter: Carl Anderson <ccc>
Component: jst.j2eeAssignee: Dimitar Giormov <dimitar.giormov>
Status: RESOLVED FIXED QA Contact: Chuck Bridgham <cbridgha>
Severity: normal    
Priority: P3 CC: dimitar.giormov, raghunathan.srinivasan
Version: 3.2Flags: dimitar.giormov: pmc_approved? (david_williams)
raghunathan.srinivasan: pmc_approved+
dimitar.giormov: pmc_approved? (naci.dai)
dimitar.giormov: pmc_approved? (deboer)
dimitar.giormov: pmc_approved? (neil.hauge)
dimitar.giormov: pmc_approved? (kaloyan)
cbridgha: review+
Target Milestone: 3.2.3   
Hardware: PC   
OS: Windows XP   
Whiteboard: PMC_approved
Attachments:
Description Flags
Added filtermapping to servlet to show in the DD tree. none

Description Carl Anderson CLA 2010-09-07 13:15:47 EDT 
In GroupFilterMappingItemProvider.getFilterMappingDisplay(), if the filter mapping is based on a servlet name instead of a URL mapping, it is discarded.  We should show these filter mappings in the Project Explorer.
Comment 1 Carl Anderson CLA 2010-09-07 13:16:33 EDT 
I am assigning this to Kaloyan for initial investigation.
Comment 2 Dimitar Giormov CLA 2011-01-19 09:58:31 EST 
Created attachment 187110 [details]
Added filtermapping to servlet to show in the DD tree.
Comment 3 Dimitar Giormov CLA 2011-01-19 09:59:03 EST 
Chuck can you review the patch?
Comment 4 Chuck Bridgham CLA 2011-01-25 14:33:21 EST 
approve
Comment 5 Dimitar Giormov CLA 2011-01-26 02:43:38 EST 
Dear PMC members,

This bug causes incomplete representation of the Web model in the deployment descriptor tree. Till now the filter mappings that point to servlet were ignored and not shown.
There is a workaround to check the web.xml
The fix is tested manually. Chuck has reviewed the change.
Risk: Low

Technical details:
The fix actually adds the filter mappings that point to servlet to the list with the mappings that point to URL in the appropriate format.
Comment 6 Carl Anderson CLA 2011-01-27 14:46:46 EST 
Committed to R3_2_maintenance for WTP 3.2.3