From: Yanick Champoux Date: Tue, 30 Nov 2010 01:53:14 +0000 (-0500) Subject: Merge remote branch 'ajgb/master' into www-sitemap-xml X-Git-Url: https://git.dogcows.com/gitweb?p=chaz%2Fp5-Catalyst-Plugin-Sitemap;a=commitdiff_plain;h=d558736b2efb240edede4074d6c8b9cdf012b13e;hp=b0594142cd44bf205ff2623161b27b504e624786 Merge remote branch 'ajgb/master' into www-sitemap-xml --- diff --git a/lib/Catalyst/Plugin/Sitemap.pm b/lib/Catalyst/Plugin/Sitemap.pm index 7f1aee2..cb57982 100644 --- a/lib/Catalyst/Plugin/Sitemap.pm +++ b/lib/Catalyst/Plugin/Sitemap.pm @@ -35,7 +35,7 @@ package Catalyst::Plugin::Sitemap; sub sitemap : Path('/sitemap') { my ( $self, $c ) = @_; - $c->res->body( $c->sitemap->xml ); + $c->res->body( $c->sitemap->as_xml->sprint ); } =head1 DESCRIPTION @@ -47,7 +47,7 @@ of the sitemap of a Catalyst application. =head2 sitemap() -Returns a L object. The sitemap object is populated by +Returns a L object. The sitemap object is populated by inspecting the controllers of the application for actions with the sub attribute C<:Sitemap>. @@ -89,7 +89,8 @@ resolves in a single url, this will results in an error. ... } -Adds the url with the given entry attributes (as defined by C). +Adds the url with the given entry attributes (as defined by +L). If the action does not resolves in a single url, this will results in an error. @@ -116,7 +117,7 @@ resolving to many urls. =head1 BUGS AND LIMITATIONS Currently, each invocation of the method C will -re-generate the C object. A future version +re-generate the C object. A future version of this module might offer a way to only compute the sitemap once. @@ -125,7 +126,7 @@ once. =over -=item L +=item L Module that C uses under the hood. @@ -149,14 +150,13 @@ use Moose::Role; no warnings qw/uninitialized/; -use Search::Sitemap; +use WWW::Sitemap::XML; use List::Util qw/ first /; sub sitemap { my $self = shift; - my $sitemap = Search::Sitemap->new; - $sitemap->pretty(1); + my $sitemap = WWW::Sitemap::XML->new; for my $controller ( $self->controller(qr//) ) { ACTION: diff --git a/t/lib/TestApp/Controller/Root.pm b/t/lib/TestApp/Controller/Root.pm index 0eb3c6e..4852ee1 100644 --- a/t/lib/TestApp/Controller/Root.pm +++ b/t/lib/TestApp/Controller/Root.pm @@ -8,7 +8,7 @@ use parent 'Catalyst::Controller'; sub sitemap : Path('/sitemap') { my ( $self, $c ) = @_; - $c->res->body( $c->sitemap->xml ); + $c->res->body( $c->sitemap->as_xml->sprint ); } sub alone :Local :Sitemap { }