Make response body able to be undef to allow RenderView to see 'defined but empty...

diff --git a/lib/Catalyst/Response.pm b/lib/Catalyst/Response.pm
index f268aef..9c8a4b2 100644 (file)
--- a/lib/Catalyst/Response.pm
+++ b/lib/Catalyst/Response.pm
@@ -6,14 +6,8 @@ use HTTP::Headers;
 with 'MooseX::Emulate::Class::Accessor::Fast';
 
 has cookies   => (is => 'rw', default => sub { {} });
-has body      => (is => 'rw', default => '', lazy => 1, predicate => 'has_body',
-    clearer => '_clear_body'
-);
-after 'body' => sub { # If someone assigned undef, clear the body so we get ''
-    if (scalar(@_) == 2 && !defined($_[1])) {
-         $_[0]->_clear_body;
-    }
-};
+has body      => (is => 'rw', default => undef, lazy => 1, predicate => 'has_body');
+
 has location  => (is => 'rw');
 has status    => (is => 'rw', default => 200);
 has finalized_headers => (is => 'rw', default => 0);

diff --git a/t/aggregate/unit_response.t b/t/aggregate/unit_response.t
index 31e397a..2380439 100644 (file)
--- a/t/aggregate/unit_response.t
+++ b/t/aggregate/unit_response.t
@@ -1,8 +1,9 @@
 use strict;
 use warnings;
-use Test::More tests => 6;
+use Test::More;
 
 use_ok('Catalyst::Response');
+use_ok('Catalyst::Engine');
 
 my $res = Catalyst::Response->new;
 
@@ -12,7 +13,5 @@ is($res->code, 500, 'code sets itself');
 is($res->status, 500, 'code sets status');
 $res->status(501);
 is($res->code, 501, 'status sets code');
-is($res->body, '', "default response body ''");
-$res->body(undef);
-is($res->body, '', "response body '' after assigned undef");
 
+done_testing;