Coercion throws UncheckedIOException instead of IOException

for methods that are not specifically I/O-related.
ao-apps · Oct 6, 2023 · 3c2b5c8 · 3c2b5c8
1 parent fdbfd50
commit 3c2b5c8
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 10 deletions.
diff --git a/book/pom.xml b/book/pom.xml
@@ -29,7 +29,7 @@ along with pragmatickm-password-taglib.  If not, see <https://www.gnu.org/licens
     <relativePath>../../../parent/pom.xml</relativePath>
   </parent>
 
-  <groupId>com.pragmatickm</groupId><artifactId>pragmatickm-password-taglib-book</artifactId><version>1.8.1-POST-SNAPSHOT</version>
+  <groupId>com.pragmatickm</groupId><artifactId>pragmatickm-password-taglib-book</artifactId><version>1.9.0-SNAPSHOT</version>
   <packaging>jar</packaging>
 
   <properties>
@@ -145,7 +145,7 @@ along with pragmatickm-password-taglib.  If not, see <https://www.gnu.org/licens
             <configuration>
               <artifactItems>
                 <artifactItem>
-                  <groupId>com.pragmatickm</groupId><artifactId>pragmatickm-password-taglib</artifactId><version>1.8.1${POST-SNAPSHOT}</version>
+                  <groupId>com.pragmatickm</groupId><artifactId>pragmatickm-password-taglib</artifactId><version>1.9.0-SNAPSHOT<!-- ${POST-SNAPSHOT} --></version>
                   <type>jar</type>
                   <includes>META-INF/pragmatickm-password.tld</includes>
                 </artifactItem>
@@ -155,7 +155,7 @@ along with pragmatickm-password-taglib.  If not, see <https://www.gnu.org/licens
         </executions>
         <dependencies>
           <dependency>
-            <groupId>com.pragmatickm</groupId><artifactId>pragmatickm-password-taglib</artifactId><version>1.8.1${POST-SNAPSHOT}</version>
+            <groupId>com.pragmatickm</groupId><artifactId>pragmatickm-password-taglib</artifactId><version>1.9.0-SNAPSHOT<!-- ${POST-SNAPSHOT} --></version>
             <type>jar</type>
             <exclusions><exclusion><groupId>*</groupId><artifactId>*</artifactId></exclusion></exclusions>
           </dependency>
@@ -242,7 +242,7 @@ along with pragmatickm-password-taglib.  If not, see <https://www.gnu.org/licens
         <groupId>com.semanticcms</groupId><artifactId>semanticcms-core-taglib</artifactId><version>1.16.0-SNAPSHOT<!-- ${POST-SNAPSHOT} --></version>
       </dependency>
       <dependency>
-        <groupId>com.semanticcms</groupId><artifactId>semanticcms-section-taglib</artifactId><version>1.9.1${POST-SNAPSHOT}</version>
+        <groupId>com.semanticcms</groupId><artifactId>semanticcms-section-taglib</artifactId><version>1.10.0-SNAPSHOT<!-- ${POST-SNAPSHOT} --></version>
       </dependency>
       <dependency>
         <groupId>com.semanticcms</groupId><artifactId>semanticcms-tag-reference</artifactId><version>1.10.1${POST-SNAPSHOT}</version>

diff --git a/book/src/main/resources-filtered/META-INF/resources/password/taglib/changelog.jspx b/book/src/main/resources-filtered/META-INF/resources/password/taglib/changelog.jspx
@@ -37,7 +37,7 @@ along with pragmatickm-password-taglib.  If not, see <https://www.gnu.org/licens
     shortTitle="Changelog"
     tocLevels="1"
     datePublished="2017-07-25T20:56:44-05:00"
-    dateModified="2022-05-15T19:16:45-05:00"
+    dateModified="2023-10-06T05:56:05Z"
   >
     <c:set var="latestRelease" value="1.8.1" />
     <c:if test="${
@@ -52,7 +52,11 @@ along with pragmatickm-password-taglib.  If not, see <https://www.gnu.org/licens
         artifactId="@{documented.artifactId}"
         repository="@{nexusUrl}content/repositories/snapshots/"
         scmUrl="@{project.scm.url}"
-      />
+      >
+        <ul>
+          <li><code>ElementTag.evaluateAttributes(…)</code> no longer throws <code>IOException</code>.</li>
+        </ul>
+      </changelog:release>
     </c:if>
 
     <changelog:release

diff --git a/pom.xml b/pom.xml
@@ -29,7 +29,7 @@ along with pragmatickm-password-taglib.  If not, see <https://www.gnu.org/licens
     <relativePath>../../parent/pom.xml</relativePath>
   </parent>
 
-  <groupId>com.pragmatickm</groupId><artifactId>pragmatickm-password-taglib</artifactId><version>1.8.1-POST-SNAPSHOT</version>
+  <groupId>com.pragmatickm</groupId><artifactId>pragmatickm-password-taglib</artifactId><version>1.9.0-SNAPSHOT</version>
   <packaging>jar</packaging>
 
   <properties>

diff --git a/src/main/java/com/pragmatickm/password/taglib/PasswordTableTag.java b/src/main/java/com/pragmatickm/password/taglib/PasswordTableTag.java
@@ -78,7 +78,7 @@ protected PasswordTable createElement() {
   }
 
   @Override
-  protected void evaluateAttributes(PasswordTable element, ELContext elContext) throws JspTagException, IOException {
+  protected void evaluateAttributes(PasswordTable element, ELContext elContext) throws JspTagException {
     super.evaluateAttributes(element, elContext);
     element.setHeader(resolveValue(header, String.class, elContext));
   }
@@ -97,7 +97,7 @@ protected void doBody(PasswordTable passwordTable, CaptureLevel captureLevel) th
         // Evaluate expressins
         @SuppressWarnings("unchecked")
         Iterable<? extends Password> passwordIter = resolveValue(passwords, Iterable.class, elContext);
-        Object styleObj = Coercion.nullIfEmpty(resolveValue(style, Object.class, elContext));
+        Object styleObj = Coercion.nullIfEmpty(resolveValue(style, Object.class, elContext)); // TODO: ao-fluent-html-any normalize here and similar
 
         BufferWriter capturedOut = EncodingBufferedTag.newBufferWriter(request);
         try {

diff --git a/src/main/java/com/pragmatickm/password/taglib/PasswordTag.java b/src/main/java/com/pragmatickm/password/taglib/PasswordTag.java
@@ -80,7 +80,7 @@ protected Password createElement() {
   }
 
   @Override
-  protected void evaluateAttributes(Password pssword, ELContext elContext) throws JspTagException, IOException {
+  protected void evaluateAttributes(Password pssword, ELContext elContext) throws JspTagException {
     super.evaluateAttributes(pssword, elContext);
     pssword.setHref(resolveValue(href, String.class, elContext));
     pssword.setUsername(resolveValue(username, String.class, elContext));