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

Timestamp:

05/25/06 17:53:56 (6 years ago)

Author:

nick

Message:

Refactoring - shift fetch methods into Listing.pm, to avoid yet more duplication

Files:

• wiki-toolkit/trunk/lib/Wiki/Toolkit/Feed/Listing.pm (modified) (1 diff)

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

@@ -101 +101 @@
 
 
+=item B<recent_changes>
+
+Build an Atom Feed of the recent changes to the Wiki::Toolkit instance,
+using any supplied parameters to narrow the results.
+
+If the argument "also_return_timestamp" is supplied, it will return an
+array of the feed, and the feed timestamp. Otherwise it just returns the feed.
+
+=cut
+sub recent_changes
+{
+    my ($self, %args) = @_;
+
+    my @changes = $self->fetch_recently_changed_nodes(%args);
+    my $feed_timestamp = $self->feed_timestamp(
+                              $self->fetch_newest_for_recently_changed(%args)
+    );
+
+    my $feed = $self->generate_node_list_feed($feed_timestamp, @changes);
+
+    if($args{'also_return_timestamp'}) {
+        return ($feed,$feed_timestamp);
+    } else {
+        return $feed;
+    }
+}
+
+
+=item B<node_all_versions>
+
+Build an Atom Feed of all the different versions of a given node.
+
+If the argument "also_return_timestamp" is supplied, it will return an
+array of the feed, and the feed timestamp. Otherwise it just returns the feed.
+
+=cut
+sub node_all_versions
+{
+    my ($self, %args) = @_;
+
+    my @all_versions = $self->fetch_node_all_versions(%args);
+    my $feed_timestamp = $self->feed_timestamp( $all_versions[0] );
+
+    my $feed = $self->generate_node_list_feed($feed_timestamp, @all_versions);
+
+    if($args{'also_return_timestamp'}) {
+        return ($feed,$feed_timestamp);
+    } else {
+        return $feed;
+    }
+} 
+
+
+
 # The following are methods that any feed renderer must provide
 
-=item B<recent_changes>
-All implementing feed renderers must implement a method to fetch a list
-of recent changes, and render them
-=cut
-sub recent_changes    { die("Not implemented by feed renderer!"); }
-=item B<node_all_versions>
-All implementing feed renderers must implement a method to fetch a list
-of the different versions of a node, and render them
-=cut
-sub node_all_versions { die("Not implemented by feed renderer!"); }
 =item B<feed_timestamp>
 All implementing feed renderers must implement a method to produce a
 feed specific timestamp, based on the supplied node
 =cut
-sub feed_timestamp    { die("Not implemented by feed renderer!"); }
+sub feed_timestamp          { die("Not implemented by feed renderer!"); }
+=item B<generate_node_list_feed>
+All implementing feed renderers must implement a method to produce a
+feed from the supplied list of nodes
+=cut
+sub generate_node_list_feed { die("Not implemented by feed renderer!"); }
 
 1;