link() htmltemplate function now has a "showid" option for links & multilinks.

When true, it only displays the linked node id as the anchor text. The
link value is displayed as a tooltip using the title anchor attribute.

Author: Richard Jones

CHANGES.txt

@@ -1,7 +1,22 @@
 This file contains the changes to the Roundup system over time. The entries
 are given with the most recent entry first.
 
-2002-03-?? - 0.4.1
+0.4.2
+Feature:
+ . link() htmltemplate function now has a "showid" option for links and
+   multilinks. When true, it only displays the linked node id as the anchor
+   text. The link value is displayed as a tooltip using the title anchor
+   attribute.
+   To use in eg. the superseder field, have something like this:
+   <td>
+    <display call="field('superseder', showid=1)">
+    <display call="classhelp('issue', 'id,title', label='list', width=500)">
+    <property name="superseder">
+     <br>View: <display call="link('superseder', showid=1)">
+    </property>
+   </td>
+
+2002-03-25 - 0.4.1
 Feature:
  . use blobfiles in back_anydbm which is used in back_bsddb.
    change test_db as dirlist does not work for subdirectories.

doc/customizing.txt

@@ -321,6 +321,8 @@ highly customized.
 |         |text.                                                              |
 |         |Options:                                                           |
 |         |property (property name) - the property to use in the second case. |
+|         |showid - use the linked node id as the link text (linked node      |
+|         |         "value" will be set as a tooltip)                         |
 +---------+-------------------------------------------------------------------+
 |count    |For a Multilink property, display a count of the number of links in|
 |         |the list.                                                          |
@@ -515,6 +517,21 @@ As shown in the example, the editor template can also request the display of a
 "note" field, which is a text area for entering a note to go along with a
 change.
 
+The <property> tag used in the index may also be used here - it checks to see
+if the nominated Multilink property has any entries. This can be used to
+eliminate sections of the editor section if the property has no entries::
+
+  <td class="form-text">
+    <display call="field('superseder', size=40, showid=1)">
+    <display call="classhelp('issue', 'id,title', label='list', width=500)">
+    <property name="superseder">
+      <br>View: <display call="link('superseder', showid=1)">
+    </property>
+  </td>
+
+The "View: " part with the links will only display if the superseder property
+has values.
+
 When a change is submitted, the system automatically generates a message
 describing the changed properties.
 
@@ -543,7 +560,7 @@ message nodes, and selecting a message takes you to its content.
 
 The <property> tag used in the index may also be used here - it checks to see
 if the nominated Multilink property has any entries. This can be used to
-eliminate sections of the spool section if the property has no entries.::
+eliminate sections of the spool section if the property has no entries::
 
      <property name="files">
       <tr class="strong-header">

roundup/htmltemplate.py

@@ -15,13 +15,13 @@
 # BASIS, AND THERE IS NO OBLIGATION WHATSOEVER TO PROVIDE MAINTENANCE,
 # SUPPORT, UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
 # 
-# $Id: htmltemplate.py,v 1.84 2002-03-29 19:41:48 rochecompaan Exp $
+# $Id: htmltemplate.py,v 1.85 2002-04-02 01:40:58 richard Exp $
 
 __doc__ = """
 Template engine.
 """
 
-import os, re, StringIO, urllib, cgi, errno, types
+import os, re, StringIO, urllib, cgi, errno, types, urllib
 
 import hyperdb, date
 from i18n import _
@@ -331,7 +331,7 @@ def do_menu(self, property, size=None, height=None, showid=0):
         return _('[Menu: not a link]')
 
     #XXX deviates from spec
-    def do_link(self, property=None, is_download=0):
+    def do_link(self, property=None, is_download=0, showid=0):
         '''For a Link or Multilink property, display the names of the linked
            nodes, hyperlinked to the item views on those nodes.
            For other properties, link to this node with the property as the
@@ -355,24 +355,36 @@ def do_link(self, property=None, is_download=0):
             linkcl = self.db.classes[linkname]
             k = linkcl.labelprop()
             linkvalue = cgi.escape(linkcl.get(value, k))
+            if showid:
+                label = value
+		title = ' title="%s"'%linkvalue
+		# note ... this should be urllib.quote(linkcl.get(value, k))
+            else:
+                label = linkvalue
             if is_download:
-                return '<a href="%s%s/%s">%s</a>'%(linkname, value,
-                    linkvalue, linkvalue)
+                return '<a href="%s%s/%s"%s>%s</a>'%(linkname, value,
+                    linkvalue, title, label)
             else:
-                return '<a href="%s%s">%s</a>'%(linkname, value, linkvalue)
+                return '<a href="%s%s"%s>%s</a>'%(linkname, value, title, label)
         if isinstance(propclass, hyperdb.Multilink):
             linkname = propclass.classname
             linkcl = self.db.classes[linkname]
             k = linkcl.labelprop()
             l = []
             for value in value:
                 linkvalue = cgi.escape(linkcl.get(value, k))
+                if showid:
+                    label = value
+                    title = ' title="%s"'%linkvalue
+		    # note ... this should be urllib.quote(linkcl.get(value, k))
+                else:
+                    label = linkvalue
                 if is_download:
-                    l.append('<a href="%s%s/%s">%s</a>'%(linkname, value,
-                        linkvalue, linkvalue))
+                    l.append('<a href="%s%s/%s"%s>%s</a>'%(linkname, value,
+                        linkvalue, title, label))
                 else:
-                    l.append('<a href="%s%s">%s</a>'%(linkname, value,
-                        linkvalue))
+                    l.append('<a href="%s%s"%s>%s</a>'%(linkname, value,
+                        title, label))
             return ', '.join(l)
         if is_download:
             return '<a href="%s%s/%s">%s</a>'%(self.classname, self.nodeid,
@@ -1114,6 +1126,10 @@ def render(self, form):
 
 #
 # $Log: not supported by cvs2svn $
+# Revision 1.84  2002/03/29 19:41:48  rochecompaan
+#  . Fixed display of mutlilink properties when using the template
+#    functions, menu and plain.
+#
 # Revision 1.83  2002/02/27 04:14:31  richard
 # Ran it through pychecker, made fixes
 #

roundup/templates/classic/html/issue.item

@@ -23,8 +23,13 @@
 
 <tr bgcolor="ffffea">
     <td width=1% nowrap align=right><span class="form-label">Superseder</span></td>
-    <td class="form-text"><display call="field('superseder', size=40, showid=1)">
-     <display call="classhelp('issue', 'id,title', label='list', width=500)"></td>
+    <td class="form-text">
+     <display call="field('superseder', size=40, showid=1)">
+     <display call="classhelp('issue', 'id,title', label='list', width=500)">
+     <property name="superseder">
+      <br>View: <display call="link('superseder', showid=1)">
+     </property>
+    </td>
     <td width=1% nowrap align=right><span class="form-label">Nosy List</span></td>
     <td class="form-text"><display call="field('nosy')">
      <display call="classhelp('user', 'username,realname,address,phone', label='list', width=500)"></td>

roundup/templates/extended/html/issue.item

@@ -1,4 +1,4 @@
-<!-- $Id: issue.item,v 1.10 2002-02-22 00:06:58 richard Exp $-->
+<!-- $Id: issue.item,v 1.11 2002-04-02 01:40:59 richard Exp $-->
 <table border=0 cellspacing=0 cellpadding=2>
 
 <tr class="strong-header">
@@ -51,11 +51,21 @@
 
 <tr bgcolor="ffffea">
     <td width=1% nowrap align=right><span class="form-label">Superseder</span></td>
-    <td class="form-text"><display call="field('superseder', size=40, showid=1)">
-     <display call="classhelp('issue', 'id,title', label='list', width=500)"></td>
+    <td class="form-text">
+     <display call="field('superseder', size=40, showid=1)">
+     <display call="classhelp('issue', 'id,title', label='list', width=500)">
+     <property name="superseder">
+      <br>View: <display call="link('superseder', showid=1)">
+     </property>
+    </td>
     <td width=1% nowrap align=right><span class="form-label">Support call</span></td>
-    <td class="form-text"><display call="field('supportcall', size=40, showid=1)">
-     <display call="classhelp('support', 'id,title', label='list', width=500)"></td>
+    <td class="form-text">
+     <display call="field('supportcall', size=40, showid=1)">
+     <display call="classhelp('support', 'id,title', label='list', width=500)">
+     <property name="supportcall">
+      <br>View: <display call="link('supportcall', showid=1)">
+     </property>
+    </td>
 </tr>
 
 <tr bgcolor="ffffea">

roundup/templates/extended/html/support.item

@@ -1,4 +1,4 @@
-<!-- $Id: support.item,v 1.4 2002-01-22 00:21:32 richard Exp $-->
+<!-- $Id: support.item,v 1.5 2002-04-02 01:40:59 richard Exp $-->
 <table border=0 cellspacing=0 cellpadding=2>
 
 <tr class="strong-header">
@@ -49,7 +49,13 @@
 
 <tr bgcolor="ffffea">
     <td width=1% nowrap align=right><span class="form-label">Superseder</span></td>
-    <td class="form-text"><display call="field('superseder', size=40, showid=1)"></td>
+    <td class="form-text">
+     <display call="field('superseder', size=40, showid=1)">
+     <display call="classhelp('issue', 'id,title', label='list', width=500)">
+     <property name="superseder">
+      <br>View: <display call="link('superseder', showid=1)">
+     </property>
+    </td>
     <td width=1% nowrap align=right><span class="form-label">Nosy List</span></td>
     <td class="form-text"><display call="field('nosy')"></td>
 </tr>