Changeset 297

Timestamp:

05/18/06 17:11:36 (6 years ago)

Author:

nick

Message:

Newest, not oldest (doh!)

Location:

wiki-toolkit/trunk/lib/Wiki/Toolkit/Feed

Files:

• Atom.pm (modified) (2 diffs)
• Listing.pm (modified) (1 diff)
• RSS.pm (modified) (2 diffs)

wiki-toolkit/trunk/lib/Wiki/Toolkit/Feed/Atom.pm

@@ -56 +56 @@
     my @changes = $self->fetch_recently_changed_nodes(%args);
     my $atom_timestamp = $self->feed_timestamp(
-                              $self->fetch_oldest_for_recently_changed(%args)
+                              $self->fetch_newest_for_recently_changed(%args)
     );
 
@@ -151 +151 @@
 =item B<feed_timestamp>
 
-Generate the timestamp for the Atom, based on the oldest node (if available)
+Generate the timestamp for the Atom, based on the newest node (if available)
 
 =cut
 sub feed_timestamp
 {
-  my ($self, $oldest_node) = @_;
-  
-  if ($oldest_node->{last_modified})
-  {
-    my $time = Time::Piece->strptime( $oldest_node->{last_modified}, $self->{timestamp_fmt} );
+  my ($self, $newest_node) = @_;
+  
+  if ($newest_node->{last_modified})
+  {
+    my $time = Time::Piece->strptime( $newest_node->{last_modified}, $self->{timestamp_fmt} );
 
     my $utc_offset = $self->{utc_offset};

wiki-toolkit/trunk/lib/Wiki/Toolkit/Feed/Listing.pm

@@ -43 +43 @@
 
 Based on the supplied criteria (but not using all of those used by
-B<fetch_recently_changed_nodes>), find the oldest node from the recently
+B<fetch_recently_changed_nodes>), find the newest node from the recently
 changed nodes set. Normally used for dating the whole of a Feed.
 
 =cut
-sub fetch_oldest_for_recently_changed {
+sub fetch_newest_for_recently_changed {
     my ($self, %args) = @_;
 

wiki-toolkit/trunk/lib/Wiki/Toolkit/Feed/RSS.pm

@@ -63 +63 @@
   my @changes = $self->fetch_recently_changed_nodes(%args);
   my $rss_timestamp = $self->rss_timestamp(
-                              $self->fetch_oldest_for_recently_changed(%args)
+                              $self->fetch_newest_for_recently_changed(%args)
   );
 
@@ -216 +216 @@
 =item B<rss_timestamp>
 
-Generate the timestamp for the RSS, based on the oldest node (if available)
+Generate the timestamp for the RSS, based on the newest node (if available)
 
 =cut
 sub rss_timestamp
 {
-    my ($self, $oldest_node) = @_;
-
-    if ($oldest_node->{last_modified})
+    my ($self, $newest_node) = @_;
+
+    if ($newest_node->{last_modified})
     {
-        my $time = Time::Piece->strptime( $oldest_node->{last_modified}, $self->{timestamp_fmt} );
+        my $time = Time::Piece->strptime( $newest_node->{last_modified}, $self->{timestamp_fmt} );
 
         my $utc_offset = $self->{utc_offset};