You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bu...@apache.org on 2001/02/03 20:29:59 UTC

[Bug 376] New - Check for attribute setter method BugRat Report#665

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=376

*** shadow/376	Sat Feb  3 11:29:59 2001
--- shadow/376.tmp.24460	Sat Feb  3 11:29:59 2001
***************
*** 0 ****
--- 1,24 ----
+ +============================================================================+
+ | Check for attribute setter method BugRat Report#665                        |
+ +----------------------------------------------------------------------------+
+ |        Bug #: 376                         Product: Tomcat 3                |
+ |       Status: NEW                         Version: 3.2.1 Final             |
+ |   Resolution:                            Platform: All                     |
+ |     Severity: Normal                   OS/Version: All                     |
+ |     Priority: High                      Component: Jasper                  |
+ +----------------------------------------------------------------------------+
+ |  Assigned To: bugzilla@apache.org                                          |
+ |  Reported By: anonymous-bug@cortexity.com                                  |
+ |      CC list: Cc:                                                          |
+ +----------------------------------------------------------------------------+
+ |          URL:                                                              |
+ +============================================================================+
+ |                              DESCRIPTION                                   |
+ If you specify an attribute for a JSP tag and then forget to put a setter method in the Tag class, you get a meaningless "Unable to compile class for JSP" error at runtime.
+ 
+ The offending code is lines 195-6 of TagBeginGenerator.java (generateSetters):
+ 
+ Method m = tc.getSetterMethod(attrName);
+ Class c[] = m.getParameterTypes();
+ 
+ Suggested change: check whether m is null and throw a meaningful exception if it is ("No setter method specified for attribute " + attrName).