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

Bug 365838

Summary: rtexprvalue tag attribute default incorrect
Product: [WebTools] WTP Source Editing Reporter: Stephen Williams <gingasteve>
Component: jst.jspAssignee: Nick Sandonato <nsand.dev>
Status: RESOLVED FIXED QA Contact: Nick Sandonato <nsand.dev>
Severity: normal    
Priority: P3 CC: thatnitind
Version: 3.3.1   
Target Milestone: 3.3.2   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Before adding rtexprvalue attribute
none
After adding rtexprvalue attribute none

Description Stephen Williams CLA 2011-12-07 03:50:03 EST 
The custom tag attribute rtexprvalue appears to default to false. Is this correct?

I have migrated an Java EE application to Indigo from Helios and am in the process of tidying up the application. I have at least 10,000 warnings. Many of them are of the form "xxx" does not support runtime expressions.

I think I can fix the problem by adding rtexprvalue="true" to the tag attributes but, not only will this be a long job, I don't think I should have to do it.

I will attach two screen shots. The first shows an example error. The second shows what happens after I have added rtexprvalue="true" to one offending tag attribute, i.e. the disappearance of the error.


-- Configuration Details --
Product: Eclipse 1.4.1.20110909-0613 (org.eclipse.epp.package.jee.product)
Installed Features:
 org.eclipse.platform 3.7.1.r37x_v20110729-9gF7UHOxFtniV7mI3T556iZN9AU8bEZ1lHMcVK
Comment 1 Stephen Williams CLA 2011-12-07 03:53:10 EST 
Created attachment 208040 [details]
Before adding rtexprvalue attribute

In this screen shot, Eclipse is complaining that the buildversion attribute of  containerpage.tag does not support runtime expressions and the usage of the tag in addPostTradeUser.jsp is invalid.
Comment 2 Stephen Williams CLA 2011-12-07 03:55:41 EST 
Created attachment 208041 [details]
After adding rtexprvalue attribute

This screen shot shows that, once I add rtexprvalue="true" to containerpage.tag, the warning goes away.
Comment 3 Stephen Williams CLA 2011-12-07 03:57:13 EST 
Please be mindful that I'm relatively new to JSP development and so might simply be that I've misunderstood something rather than a bug in Eclipse.
Comment 4 Nick Sandonato CLA 2011-12-07 13:56:58 EST 
You are correct. The specification indicates that the attribute directive's rtexprvalue default value is true. I've released changes along with some unit tests for default versus specified values for attribute directives. Thanks, Stephen.