|Summary:||Class cast exception in tagPlugin Set generated code|
|Product:||Tomcat 7||Reporter:||Sheldon Shao <xshao>|
|Component:||Jasper||Assignee:||Tomcat Developers Mailing List <dev>|
Test case for "Set"
Patch for Set.java
Description Sheldon Shao 2012-12-21 12:00:09 UTC
Created attachment 29786 [details] Test case for "Set" It uses "Class.cast" to convert an object to a given class in the code generated by tagPlugin "Set". It causes an exception when the value is primitive type, Here is a sample, Caused by: java.lang.ClassCastException: Cannot cast class java.lang.Integer to int at java.lang.Class.cast(Class.java:1711) at org.apache.jsp.WEB_002dINF.views.error_jsp._jspService(error_jsp.java:108) at org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:70) at javax.servlet.http.HttpServlet.service(HttpServlet.java:668) at org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:432) ... 52 more The correct way is to use "ELSupport.coerceToType". Attached a patch and test case for this issue.
Comment 1 Sheldon Shao 2012-12-21 12:00:56 UTC
Created attachment 29787 [details] Patch for Set.java
Comment 2 Christopher Schultz 2013-01-03 04:00:34 UTC
Is it appropriate to perform a "coerce" instead of a simple cast? (I don't know whether the spec covers this case). If the only problem is primitive types, I think this can be solved without the complexity of formal coercion.
Comment 3 Sheldon Shao 2013-01-04 07:04:05 UTC
Yes. The problem isn't only from primitive types. Perform a "coerce" is a better choice. (In reply to comment #2) > Is it appropriate to perform a "coerce" instead of a simple cast? (I don't > know whether the spec covers this case). If the only problem is primitive > types, I think this can be solved without the complexity of formal coercion.
Comment 4 Mark Thomas 2013-01-04 21:35:14 UTC
It is not a question of what is appropriate or what is best, the JSTL specification requires that an EL coerce is used. I'm looking at this now. The test case is going to need some work.
Comment 5 Mark Thomas 2013-01-04 22:10:32 UTC
Many thanks for the patch. It has been applied to trunk and 7.0.x and will be included in 7.0.35 onwards. I added a different test case to trunk that tests that the coercion works rather than tests the source code for a particular string. Note that the trunk tests for the tag plug-ins depend on the new resources implementation so they have not been back-ported to 7.0.x.