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

Bug 311568

Summary: Composite Component Locator doesn't detect file/folder renames and moves
Product: [WebTools] Java Server Faces Reporter: Cameron Bateman <cameron.bateman>
Component: CoreAssignee: Cameron Bateman <cameron.bateman>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: deboer, raghunathan.srinivasan
Version: 3.2Flags: cameron.bateman: pmc_approved?
raghunathan.srinivasan: pmc_approved? (david_williams)
raghunathan.srinivasan: pmc_approved? (naci.dai)
deboer: pmc_approved+
raghunathan.srinivasan: pmc_approved? (neil.hauge)
raghunathan.srinivasan: pmc_approved? (kaloyan)
raghunathan.srinivasan: review+
Target Milestone: 3.2 RC1   
Hardware: PC   
OS: Windows XP   
Whiteboard: PMC_approved
Attachments:
Description Flags
Fixes the issue.
cameron.bateman: review+
Test coverage for changes none

Description Cameron Bateman CLA 2010-05-04 11:21:30 EDT
When a folder in WebContent/resources is moved or renamed or when a file in one of these folders is renamed or moved, the changes are reflected by events fired by the JSF resource framework and are not handled perfectly by the Facelet tag registry.
Comment 1 Cameron Bateman CLA 2010-05-04 11:23:08 EDT
Created attachment 166966 [details]
Fixes the issue.
Comment 2 Cameron Bateman CLA 2010-05-04 11:24:53 EDT
Created attachment 166967 [details]
Test coverage for changes
Comment 3 Raghunathan Srinivasan CLA 2010-05-04 13:35:55 EDT
* Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug" (requested by an adopter) please document it as such. 
Locating composite components is a key feature of the JSF 2.0 support in Helios. This bug will break that feature and hence is a stop-ship defect.
* Is there a work-around? If so, why do you believe the work-around is insufficient? 
No reasonable workaround
* How has the fix been tested? Is there a test case attached to the bugzilla record? Has a JUnit Test been added? 
New Junit test to cover this use case is included
* Give a brief technical overview. Who has reviewed this fix? 
See description.
* What is the risk associated with this fix?
low-medium
Comment 4 Cameron Bateman CLA 2010-05-04 18:21:47 EDT
Patches applied to HEAD (3.2RC1).