DO NOT REPLY [Bug 46086] javax.servlet.jsp.jstl.sql.ResultSupport.java silently swallows SQLException

[email protected]
Newsgroups gmane.comp.jakarta.taglibs.devel
Message-ID <[email protected]>
https://issues.apache.org/bugzilla/show_bug.cgi?id=46086





--- Comment #1 from Henri Yandell <[email protected]>  2008-11-03 15:34:17 PST ---
*grumble*

Easiest fix - throw SQLException from those methods. Can't - it's in the spec.
Plus it would be a bit of a back compat surprise.

Next - create a new file - can't this is spec land. 

Next - use src/org/apache/taglibs/standard/tag/common/sql/ResultImpl.java
directly rather than ResultSupport. Of course - that means people aren't using
the spec anymore even if it is the same class (or copy of that class). Still -
worth doing I think, so I've made that commit.

Next is your RuntimeException, but that's another back compat surprise for
people. 

Here's my change:

svn ci -m "Adding constructors to ResultImpl to work around the SQLException
hiding in ResultSupport - and noted this in ResultSupport as per #46086" src

Sending        src/javax/servlet/jsp/jstl/sql/ResultSupport.java
Sending        src/org/apache/taglibs/standard/tag/common/sql/ResultImpl.java
Transmitting file data ..
Committed revision 711123 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=711123 ).


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.