X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FCatalyst%2FAction%2FSerialize.pm;h=2c231515f06a9e46fda19b6ed57e988c5489384a;hb=878b2b54fb06f18f019df1f380a12ebec9585195;hp=0ccc077e7f7f9598db8b69df8dc4b7c233f41f39;hpb=51cc8fe9230a36356a847ee2e443f47f3e390873;p=catagits%2FCatalyst-Action-REST.git diff --git a/lib/Catalyst/Action/Serialize.pm b/lib/Catalyst/Action/Serialize.pm index 0ccc077..2c23151 100644 --- a/lib/Catalyst/Action/Serialize.pm +++ b/lib/Catalyst/Action/Serialize.pm @@ -1,47 +1,51 @@ -# -# Catlyst::Action::Serialize.pm -# Created by: Adam Jacob, Marchex, -# -# $Id$ - package Catalyst::Action::Serialize; -use strict; -use warnings; +use Moose; +use namespace::autoclean; -use base 'Catalyst::Action::SerializeBase'; +extends 'Catalyst::Action::SerializeBase'; use Module::Pluggable::Object; -use Data::Dump qw(dump); +use MRO::Compat; sub execute { my $self = shift; my ( $controller, $c ) = @_; - $self->NEXT::execute( @_ ); + $self->maybe::next::method(@_); return 1 if $c->req->method eq 'HEAD'; return 1 if length( $c->response->body ); return 1 if scalar @{ $c->error }; return 1 if $c->response->status =~ /^(?:204|3\d\d)$/; - my ($sclass, $sarg, $content_type) = $self->_load_content_plugins("Catalyst::Action::Serialize", $controller, $c); + my ( $sclass, $sarg, $content_type ) = + $self->_load_content_plugins( "Catalyst::Action::Serialize", + $controller, $c ); unless ( defined($sclass) ) { - $c->log->debug("Could not find a serializer for $content_type"); + if ( defined($content_type) ) { + $c->log->info("Could not find a serializer for $content_type"); + } else { + $c->log->info( + "Could not find a serializer for an empty content-type"); + } return 1; } - $c->log->debug("Serializing with $sclass" . ($sarg ? " [$sarg]" : '')); + $c->log->debug( + "Serializing with $sclass" . ( $sarg ? " [$sarg]" : '' ) ) if $c->debug; my $rc; - if ( defined($sarg) ) { - $rc = $sclass->execute( $controller, $c, $sarg ); - } else { - $rc = $sclass->execute( $controller, $c ); + eval { + if ( defined($sarg) ) { + $rc = $sclass->execute( $controller, $c, $sarg ); + } else { + $rc = $sclass->execute( $controller, $c ); + } + }; + if ($@) { + return $self->_serialize_bad_request( $c, $content_type, $@ ); + } elsif (!$rc) { + return $self->_unsupported_media_type( $c, $content_type ); } - if ($rc eq 0) { - return $self->_unsupported_media_type($c, $content_type); - } elsif ($rc ne 1) { - return $self->_serialize_bad_request($c, $content_type, $rc); - } return 1; } @@ -57,13 +61,12 @@ Catalyst::Action::Serialize - Serialize Data in a Response package Foo::Controller::Bar; __PACKAGE__->config( - serialize => { - 'default' => 'YAML', - 'stash_key' => 'rest', - 'map' => { - 'text/x-yaml' => 'YAML', - 'text/x-data-dumper' => [ 'Data::Serializer', 'Data::Dumper' ], - }, + 'default' => 'text/x-yaml', + 'stash_key' => 'rest', + 'map' => { + 'text/html' => [ 'View', 'TT', ], + 'text/x-yaml' => 'YAML', + 'text/x-data-dumper' => [ 'Data::Serializer', 'Data::Dumper' ], } ); @@ -74,12 +77,11 @@ Catalyst::Action::Serialize - Serialize Data in a Response This action will serialize the body of an HTTP Response. The serializer is selected by introspecting the HTTP Requests content-type header. -It requires that your Catalyst controller have a "serialize" entry -in it's configuration, which sets up the mapping between Content Type's -and Serialization classes. +It requires that your Catalyst controller is properly configured to set up the +mapping between Content Type's and Serialization classes. -The specifics of serializing each content-type is implemented as -a plugin to L. +The specifics of serializing each content-type is implemented as a plugin to +L. Typically, you would use this ActionClass on your C method. However, nothing is stopping you from choosing specific methods to Serialize: @@ -88,28 +90,35 @@ nothing is stopping you from choosing specific methods to Serialize: .. populate stash with data .. } +When you use this module, the request class will be changed to +L. + =head1 CONFIGURATION -=over 4 +=head2 map + +Takes a hashref, mapping Content-Types to a given serializer plugin. -=item default +=head2 default -The default Serialization format. See the next section for -available options. This is used if a requested content-type -is not recognized. +This is the 'fall-back' Content-Type if none of the requested or acceptable +types is found in the L. It must be an entry in the L. -=item stash_key +=head2 stash_key -We will serialize the data that lives in this location in the stash. So -if the value is "rest", we will serialize the data under: +Specifies the key of the stash entry holding the data that is to be serialized. +So if the value is "rest", we will serialize the data under: $c->stash->{'rest'} -=item map +=head2 content_type_stash_key -Takes a hashref, mapping Content-Types to a given plugin. +Specifies the key of the stash entry that optionally holds an overriding +Content-Type. If set, and if the specified stash entry has a valid value, +then it takes priority over the requested content types. -=back +This can be useful if you want to dynamically force a particular content type, +perhaps for debugging. =head1 HELPFUL PEOPLE @@ -124,11 +133,9 @@ a sensible set of defaults for doing a REST controller. L, L -=head1 AUTHOR - -Adam Jacob , with lots of help from mst and jrockway +=head1 AUTHORS -Marchex, Inc. paid me while I developed this module. (http://www.marchex.com) +See L for authors. =head1 LICENSE