From 36b7502bb586953f62f56a2c6edd3b22c184702d Mon Sep 17 00:00:00 2001
From: Simon Cozens <simon@simon-cozens.org>
Date: Sat, 31 Jan 2004 14:02:15 +0000
Subject: [PATCH] Make the view page a bit prettier, and link has-a and url
 links nicely.

git-svn-id: http://svn.maypole.perl.org/Maypole/trunk@23 48953598-375a-da11-a14b-00016c27c3ee
---
 templates/factory/edit        |  2 +-
 templates/factory/list        |  2 +-
 templates/factory/list_macros | 22 ----------------------
 templates/factory/macros      | 34 ++++++++++++++++++++++++++++++++++
 templates/factory/view        | 35 ++++++++++++++++++-----------------
 5 files changed, 54 insertions(+), 41 deletions(-)
 delete mode 100644 templates/factory/list_macros
 create mode 100644 templates/factory/macros

diff --git a/templates/factory/edit b/templates/factory/edit
index 11a2539..712348a 100644
--- a/templates/factory/edit
+++ b/templates/factory/edit
@@ -1,4 +1,4 @@
-[% PROCESS list_macros %]
+[% PROCESS macros %]
 [% INCLUDE header %]
 
 [% FOR item = objects; %]
diff --git a/templates/factory/list b/templates/factory/list
index 69bb868..5bd6a10 100644
--- a/templates/factory/list
+++ b/templates/factory/list
@@ -1,4 +1,4 @@
-[% PROCESS list_macros %]
+[% PROCESS macros %]
 
 [% INCLUDE header %]
 
diff --git a/templates/factory/list_macros b/templates/factory/list_macros
deleted file mode 100644
index 0a88996..0000000
--- a/templates/factory/list_macros
+++ /dev/null
@@ -1,22 +0,0 @@
-[% MACRO display_line(item) BLOCK;
-     FOR col = classmetadata.columns;
-        NEXT IF col == "id";
-        "<td>";
-        "<A HREF='" _ base _ item.$col.moniker _ "/view/" _ 
-            item.$col.id _"'>" IF item.$col.moniker; # it's an object
-        item.$col;
-        "</A>" IF item.$col.moniker; 
-        "</td>";
-     END;
-    button(item, "edit");
-    button(item, "delete");
-END %]
-[% MACRO button(obj, action) BLOCK; %]
-
-<TD>
-<FORM METHOD="post" ACTION="[%base%]/[%classmetadata.moniker%]/[%action%]/[%obj.id%]">
-    <INPUT TYPE="submit" NAME="[%action%]" VALUE="[%action%]">
-</FORM>
-</TD>
-[% END %]
-
diff --git a/templates/factory/macros b/templates/factory/macros
new file mode 100644
index 0000000..b9b8dee
--- /dev/null
+++ b/templates/factory/macros
@@ -0,0 +1,34 @@
+[% MACRO maybe_link_view(object) BLOCK;
+    IF object.moniker; # It's an object, i.e. a has-a
+        "<A HREF='" _ base _ object.moniker _ "/view/" _ object.id _"'>";
+        object;
+        "</A>";
+    ELSE;
+        object;
+    END;
+END;
+%]
+
+[% MACRO display_line(item) BLOCK;
+     FOR col = classmetadata.columns;
+        NEXT IF col == "id";
+        "<td>";
+        IF col == "url";  # Possibly too much magic.
+            "<A HREF="; item.url; "> "; item.url; "</A>";
+        ELSE;
+            maybe_link_view(item.$col);
+        END;
+        "</td>";
+     END;
+    button(item, "edit");
+    button(item, "delete");
+END %]
+
+[% MACRO button(obj, action) BLOCK; %]
+<TD>
+<FORM METHOD="post" ACTION="[%base%]/[%obj.moniker%]/[%action%]/[%obj.id%]">
+    <INPUT TYPE="submit" NAME="[%action%]" VALUE="[%action%]">
+</FORM>
+</TD>
+[% END %]
+
diff --git a/templates/factory/view b/templates/factory/view
index 48dec99..861e876 100644
--- a/templates/factory/view
+++ b/templates/factory/view
@@ -1,26 +1,27 @@
-[% MACRO button(obj, action) BLOCK; %]
-
-<TD>
-<FORM METHOD="post" ACTION="[%base%]/[%classmetadata.moniker%]/[%action%]/[%obj.id%]">
-    <INPUT TYPE="submit" NAME="[%action%]" VALUE="[%action%]">
-</FORM>
-</TD>
-[% END %]
-
+[% PROCESS macros %]
 [% INCLUDE header %]
 
 [% FOR item = objects %]
 <h2> [% item.name %]</h2>
 
-[% FOR col = classmetadata.columns.list;
-    NEXT IF col == "id";
-    "<P>";
-    classmetadata.colnames.$col; ":";
-    " "; item.$col;"</p>";
-     END;
+[% INCLUDE navbar %]
+
+<TABLE class="view">
+[% 
+    FOR col = classmetadata.columns.list;
+    NEXT IF col == "id" OR col == "name";
+%]
+
+<TR>
+    <TD class="field"> [% classmetadata.colnames.$col; %] </TD>
+    <TD> [% maybe_link_view(item.$col) %] </TD>
+</TR>
+[% END; %]
+</TABLE>
+[%
     button(item, "edit");
     button(item, "delete");
-    "</tr>";
-END;
 %]
+</TABLE>
+[% END; %]
 
-- 
2.39.5