Changeset 602

Timestamp:

12/23/04 09:37:18 (4 years ago)

Author:

kake

Message:

Fixed a couple of bugs to do with nodes with only one version - list_all_versions, diff link.

Location:

trunk

Files:

• Changes (modified) (1 diff)
• MANIFEST (modified) (1 diff)
• lib/OpenGuides.pm (modified) (1 diff)
• templates/node.tt (modified) (1 diff)

trunk/Changes

@@ +1 @@
+        Fixed bug with list_all_versions for nodes with only one version.
+
 0.46    21 December 2004
         Minor bug fixes: remove bogus edit link on index listings,

trunk/MANIFEST

@@ -86 +86 @@
 t/65_bug_metadata_escape.t
 t/66_bug_latlong_representation.t
+t/67_bug_first_version.t
 t/templates/15_test.tt
 wiki.cgi

trunk/lib/OpenGuides.pm

@@ -528 +528 @@
     my %curr_data = $self->wiki->retrieve_node($node);
     my $curr_version = $curr_data{version};
-    croak "This is the first version" unless $curr_version > 1;
     my @history;
     for my $version ( 1 .. $curr_version ) {

trunk/templates/node.tt

@@ -69 +69 @@
 <div id="version">
 [% IF current %]
-Last edited [% last_modified %] (version [% version %]; <a href="[% cgi_url %]?id=[% node_param %];version=[% version - 1 %];diffversion=[% version %]">diff</a>).
+  Last edited [% last_modified %]
+  [% IF version == 1 %]
+    (version [% version %])
+  [% ELSE %]
+    (version [% version %];
+    <a href="[% cgi_url %]?id=[% node_param %];version=[% version - 1 %];diffversion=[% version %]">diff</a>).
+  [% END %]
 [% ELSE %]
-This is version [% version %] (as of [% last_modified %]). 
-<a href="[% cgi_url %]?[% node_param %]">View current version.</a> 
+  This is version [% version %] (as of [% last_modified %]). 
+  <a href="[% cgi_url %]?[% node_param %]">View current version.</a> 
 [% END %]
 [% IF version AND version > 1 %]