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

Bug 327120

Summary: Need to check whether getTextContent() is null before calling getTextContent().trim()
Product: [WebTools] Java Server Faces Reporter: Xiaonan Jiang <xiaonan_jiang>
Component: JSF ToolsAssignee: Ian Trimble <ian.trimble>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: raghunathan.srinivasan, yurykats
Version: unspecified   
Target Milestone: 3.2.3   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
fix none

Description Xiaonan Jiang CLA 2010-10-06 11:08:19 EDT 
Build Identifier: M-3.2.1-20100623085141

There are several places wtp/jsf calls getTextContent().trim() without making sure getTextContent() is not null. 
This causes exceptions when the relevant element is empty. 

Reproducible: Always
Comment 1 Xiaonan Jiang CLA 2010-10-06 11:11:28 EDT 
Created attachment 180340 [details]
fix

Added the necessary null checkings.
Comment 2 Yury Kats CLA 2010-12-01 13:46:12 EST 
Is anything preventing the patch from being applied to 3.2.3? Any other assistance we can offer to help resolve this bug?
Comment 3 Raghunathan Srinivasan CLA 2010-12-01 14:25:04 EST 
Mass update: The JSF Tools team is currently focused on performance and deadlock issues in the product. We plan to address other WTP bugs later this month and will review and checkin the patches soon. Thanks for your contribution and understanding.
-Raghu
Comment 4 Yury Kats CLA 2010-12-01 14:44:47 EST 
Thanks, Raghu. Always glad to see performance improvements in the product!
Do you have specific bug numbers for the work being done currently?
Comment 5 Raghunathan Srinivasan CLA 2010-12-01 17:07:54 EST 
(In reply to comment #4)
> Thanks, Raghu. Always glad to see performance improvements in the product!
> Do you have specific bug numbers for the work being done currently?

Recent fixes include: bug 330976, 330937, 330302 . We are launching bugs as and when we narrow the issues we see in the adopter product to issues in WTP. Hope this helps.
Comment 6 Ian Trimble CLA 2011-01-04 15:48:47 EST 
COmmitted to 3.2.3 and 3.3.0 at 2011/01/04 12:48PM PDT.