[Catalyst-commits] r8977 - in Catalyst-Runtime/5.80/trunk: . lib/Catalyst

[t0m at dev.catalyst.perl.org]

Author: t0m
Date: 2008-12-29 20:13:23 +0000 (Mon, 29 Dec 2008)
New Revision: 8977

Modified:
   Catalyst-Runtime/5.80/trunk/Changes
   Catalyst-Runtime/5.80/trunk/TODO
   Catalyst-Runtime/5.80/trunk/lib/Catalyst/Request.pm
Log:
Make Catalyst::Engine::Apaches tests pass.

Modified: Catalyst-Runtime/5.80/trunk/Changes
===================================================================
--- Catalyst-Runtime/5.80/trunk/Changes	2008-12-29 19:54:00 UTC (rev 8976)
+++ Catalyst-Runtime/5.80/trunk/Changes	2008-12-29 20:13:23 UTC (rev 8977)
@@ -1,5 +1,7 @@
 # This file documents the revision history for Perl extension Catalyst.
 
+        - Make Catalyst::Request::uploads attribute non-lazy, to fix
+          test for Catalyst-Engine-Apache (t0m)
         - Bump version of MooseX::Emulate::Class::Accessor::Fast (t0m)
         - Stop using MooseX::Adopt::Class::Accessor::Fast by default, to stop
           breaking other packages which use Class::Accessor::Fast

Modified: Catalyst-Runtime/5.80/trunk/TODO
===================================================================
--- Catalyst-Runtime/5.80/trunk/TODO	2008-12-29 19:54:00 UTC (rev 8976)
+++ Catalyst-Runtime/5.80/trunk/TODO	2008-12-29 20:13:23 UTC (rev 8977)
@@ -10,7 +10,8 @@
     - Generally unhappy with the on_end_of_scope immutable approach,
       try removing new method from plugins (and emitting warning).
 
-  - Common engine test failures, look into and get tests into core.
+  - Engine test failures, look into and update tests / do fixes as
+    needed.
 
   - CatalystX-Imports, Class::MOP doesn't consider anon subs in the symbol 
     table as methods, tests + fix, or explanation and documentation?
@@ -30,10 +31,7 @@
      - Catalyst-Plugin-Cache dies due to mk_accessors('meta')
      
      - CatalystX-CRUD fails tests against 5.80 (karpet)
-    
-     - Catalyst-Engine-Apache - Deep recursion on subroutine 
-       "Catalyst::Action::execute", investigate once all other tests pass
-    
+
   - Issues with TWMC not being loaded when it used to be in 5.70 
     (Bill Moseley)
 

Modified: Catalyst-Runtime/5.80/trunk/lib/Catalyst/Request.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/lib/Catalyst/Request.pm	2008-12-29 19:54:00 UTC (rev 8976)
+++ Catalyst-Runtime/5.80/trunk/lib/Catalyst/Request.pm	2008-12-29 20:13:23 UTC (rev 8977)
@@ -64,7 +64,6 @@
 has uploads => (
   is => 'rw',
   required => 1,
-  lazy => 1,
   default => sub { {} },
 );