needed a quick solution to target tag issues

src/com/silverwrist/venice/ui/jsp/UtilTargetTag.java

@@ -0,0 +1,45 @@
+/*
+ * The contents of this file are subject to the Mozilla Public License Version 1.1
+ * (the "License"); you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at <http://www.mozilla.org/MPL/>.
+ * 
+ * Software distributed under the License is distributed on an "AS IS" basis, WITHOUT
+ * WARRANTY OF ANY KIND, either express or implied. See the License for the specific
+ * language governing rights and limitations under the License.
+ * 
+ * The Original Code is the Venice Web Communities System.
+ * 
+ * The Initial Developer of the Original Code is Eric J. Bowersox <erbo@silcom.com>,
+ * for Silverwrist Design Studios.  Portions created by Eric J. Bowersox are
+ * Copyright (C) 2001 Eric J. Bowersox/Silverwrist Design Studios.  All Rights Reserved.
+ * 
+ * Contributor(s): 
+ */
+package com.silverwrist.venice.ui.jsp;
+
+import javax.servlet.jsp.*;
+import javax.servlet.jsp.tagext.*;
+import com.silverwrist.venice.ui.*;
+
+public class UtilTargetTag extends VeniceBodyTagSupport
+{
+  /*--------------------------------------------------------------------------------
+   * Overrides from class BodyTagSupport
+   *--------------------------------------------------------------------------------
+   */
+
+  public int doAfterBody() throws JspException
+  {
+    UtilXLinkTag xlink = (UtilXLinkTag)findAncestorWithClass(this,UtilXLinkTag.class);
+    if (xlink!=null)
+    { // set the link data
+      xlink.setTarget(getBodyContent().getString().trim());
+      return SKIP_BODY;
+
+    } // end if
+
+    throw new JspTagException("<util:target/> tag not nested inside a compatible tag!");
+
+  } // end doAfterBody
+
+} // end class UtilHrefTag

tlds/utils-taglib.tld

@@ -358,4 +358,11 @@
     <bodycontent>JSP</bodycontent>
   </tag>
 
+  <tag>
+    <name>target</name>
+    <tagclass>com.silverwrist.venice.ui.jsp.UtilTargetTag</tagclass>
+    <info>Supplies a target for the xlink tag.</info>
+    <bodycontent>JSP</bodycontent>
+  </tag>
+
 </taglib>

web/format/conf/posts.jsp

@@ -272,27 +272,11 @@ conf/posts.js.vs?<%= topic_locator %>&p1=<%= my_last + 1 %>&p2=<%= Math.min(my_l
         </util:xlink>,
         <%= view.formatDate(msg.getPostDate()) %>
       </EM>)
-<%
+      <% if (msg.hasAttachment()) { %>
-      if (msg.hasAttachment()) {
-        if (view.displayAttachmentInNewWindow(msg)) {
-%>
         <util:xlink target="_blank">
-            <util:href type="servlet">
+          <% if (view.displayAttachmentInNewWindow(msg)) { %>
-              conf/view_attachment.js.vs?<%= base_locator %>&msg=<%= msg.getPostID() %>
+            <util:target>_blank</util:target>
-            </util:href>
+          <% } // end if %>
-            <util:text>
-              <util:image src="attachment.gif" fixup="true" width="16" height="16">
-                <util:alt>
-                  (Attachment <%= msg.getAttachmentFilename() %> - <%= msg.getAttachmentLength() %> bytes)
-                </util:alt>
-              </util:image>
-            </util:text>
-            <util:linktitle>
-              (Attachment <%= msg.getAttachmentFilename() %> - <%= msg.getAttachmentLength() %> bytes)
-            </util:linktitle>
-          </util:xlink>
-        <% } else { %>
-          <util:xlink>
           <util:href type="servlet">
             conf/view_attachment.js.vs?<%= base_locator %>&msg=<%= msg.getPostID() %>
           </util:href>
@@ -310,7 +294,6 @@ conf/posts.js.vs?<%= topic_locator %>&p1=<%= my_last + 1 %>&p2=<%= Math.min(my_l
 <%
       } // end if
     } // end if
-    } // end if
 %>
   </util:font>
   <P>