| Summary: | Tag library functionality is broken when same library used in multiple projects | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [WebTools] WTP Source Editing | Reporter: | Ian Trimble <ian.trimble> | ||||
| Component: | jst.jsp | Assignee: | Nick Sandonato <nsand.dev> | ||||
| Status: | RESOLVED FIXED | QA Contact: | Nick Sandonato <nsand.dev> | ||||
| Severity: | blocker | ||||||
| Priority: | P3 | CC: | raghunathan.srinivasan, thatnitind, yurykats | ||||
| Version: | 3.2.3 | Flags: | thatnitind:
review+
|
||||
| Target Milestone: | 3.2.3 | ||||||
| Hardware: | PC | ||||||
| OS: | Windows XP | ||||||
| Whiteboard: | |||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 328783 | ||||||
| Attachments: |
|
||||||
|
Description
Ian Trimble
Further investigation indicates the JSF facet *is* required - once one JSF-faceted project has used tag library functionality, subsequent JSF-faceted projects cannot. Re-assigning to JSF Tools. Sorry for the churn back-and-forth, but I am now able to reproduce with no JSF facet. I create two Dynamic Web Projects with JSF libraries added as a User Library. The first project to use ProjectDescription to resolve tag libraries succeeds, subsequent projects do not. A diff of 3.2.2 and 3.3 streams shows fJarRecords has been added to cache one JarRecord per file for all instances, but I'm not sure all necessary members get initialized correctly when JarRecord.isConsistent causes early exit - for example, is fClasspathReferences properly initialized? It appears to be empty later during resolution when the failure is witnessed. NOTE: it may not have been clear - both projects should use the same User Library so the second project finds the JARRecord in the cache. Created attachment 181889 [details]
patch
When finding consistent library records, we need to make sure they're accounted for in the classpath JARs and References.
Approved and released, although I suspect bug 327028 has altered how we handle the visibility of these records across projects. Needs a little more investigation. |