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

[rafl at dev.catalyst.perl.org]

Author: rafl
Date: 2009-04-29 11:45:34 +0100 (Wed, 29 Apr 2009)
New Revision: 9937

Modified:
   Catalyst-Runtime/5.80/trunk/Changes
   Catalyst-Runtime/5.80/trunk/lib/Catalyst/Log.pm
   Catalyst-Runtime/5.80/trunk/lib/Catalyst/Upgrading.pod
   Catalyst-Runtime/5.80/trunk/t/unit_core_setup_log.t
Log:
Changelogging.

Modified: Catalyst-Runtime/5.80/trunk/Changes
===================================================================
--- Catalyst-Runtime/5.80/trunk/Changes	2009-04-29 10:45:25 UTC (rev 9936)
+++ Catalyst-Runtime/5.80/trunk/Changes	2009-04-29 10:45:34 UTC (rev 9937)
@@ -1,5 +1,10 @@
 # This file documents the revision history for Perl extension Catalyst.
 
+        - Various POD tweaks. (hdp, dandv)
+        - Fix formatting error in the regex fallback warning. (rafl)
+        - Convert the dispatcher's BUILD method to attribute builders to not
+          override the BUILD method from
+          MooseX::Emulate::Class::Accessor::Fast. (rafl)
         - Fix classes without metaclasses restarting, when not using
           B::Hooks::OP::Check::StashChange (t0m)
         - Fix the unattached chain debug table for endpoints with no
@@ -7,7 +12,6 @@
         - Turn of test aggregation by default. Only aggregate if the
           AGGREGATE_TESTS environment variable is set and a recent
           Test::Aggregate is available. (rafl)
-        - POD formatting fix for Catalyst::Test (Dan Dascalescu)
         - Bump to MooseX::MethodAttributes 0.09, to gain the
           get_nearest_methods_with_attributes method allowing methods without
           attributes in a subclass to override those with attributes in a

Modified: Catalyst-Runtime/5.80/trunk/lib/Catalyst/Log.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/lib/Catalyst/Log.pm	2009-04-29 10:45:25 UTC (rev 9936)
+++ Catalyst-Runtime/5.80/trunk/lib/Catalyst/Log.pm	2009-04-29 10:45:34 UTC (rev 9937)
@@ -21,10 +21,10 @@
     for ( my $i = $#levels ; $i >= 0 ; $i-- ) {
 
         my $name  = $levels[$i];
-        
+
         my $level = 1 << $i;
         $summed_level |= $level;
-        
+
         $LEVELS{$name} = $level;
         $LEVEL_MATCH{$name} = $summed_level;
 
@@ -112,7 +112,7 @@
 # Alias _body to body, add a before modifier to warn..
 my $meta = __PACKAGE__->meta; # Calling meta method here fine as we happen at compile time.
 $meta->add_method('body', $meta->get_method('_body'));
-my %package_hash; # Only warn once per method, per package. 
+my %package_hash; # Only warn once per method, per package.
                   # I haven't provided a way to disable them, patches welcome.
 $meta->add_before_method_modifier('body', sub {
     my $class = blessed(shift);
@@ -242,8 +242,8 @@
 
 =head2 abort
 
-Should Catalyst emit logs for this request? Will be reset at the end of 
-each request. 
+Should Catalyst emit logs for this request? Will be reset at the end of
+each request.
 
 *NOTE* This method is not compatible with other log apis, so if you plan
 to use Log4Perl or another logger, you should call it like this:

Modified: Catalyst-Runtime/5.80/trunk/lib/Catalyst/Upgrading.pod
===================================================================
--- Catalyst-Runtime/5.80/trunk/lib/Catalyst/Upgrading.pod	2009-04-29 10:45:25 UTC (rev 9936)
+++ Catalyst-Runtime/5.80/trunk/lib/Catalyst/Upgrading.pod	2009-04-29 10:45:34 UTC (rev 9937)
@@ -34,7 +34,7 @@
 L<Catalyst::Plugin::Session::Store::FastMmap> - if you are using this plugin
 and see issues, then please upgrade - and please try upgrading your plugins
 if you have this issue, as it has been fixed. Note that Makefile.PL in the
-distribution will warn about known incompatible components. 
+distribution will warn about known incompatible components.
 
 This issue can, however, be found in your own application - the only solution is
 to go through each base class of the class the error was reported against, until
@@ -99,14 +99,14 @@
 Note that the C< extends > declaration needs to occur in a begin block for
 L<attributes> to operate correctly.
 
-You also don't get the L<Moose::Object> constructor, and therefore attribute 
-initialization will not work as normally expected. If you want to use Moose 
+You also don't get the L<Moose::Object> constructor, and therefore attribute
+initialization will not work as normally expected. If you want to use Moose
 attributes, then they need to be made lazy to correctly initialize.
 
 Note that this only applies if your component needs to maintain component
 backwards compatibility for Catalyst versions before 5.71001 - in 5.71001
 attributes work as expected, and the BUILD method is called normally
-(although BUILDARGS is not). 
+(although BUILDARGS is not).
 
 If you depend on Catalyst 5.8, then B<all> Moose features work as expected.
 
@@ -226,7 +226,7 @@
 
     BaseClass->foo('base class');
     Child->foo('sub class');
-    
+
     use Test::More;
     isnt(BaseClass->can('foo'), Child->can('foo'));
 

Modified: Catalyst-Runtime/5.80/trunk/t/unit_core_setup_log.t
===================================================================
--- Catalyst-Runtime/5.80/trunk/t/unit_core_setup_log.t	2009-04-29 10:45:25 UTC (rev 9936)
+++ Catalyst-Runtime/5.80/trunk/t/unit_core_setup_log.t	2009-04-29 10:45:34 UTC (rev 9937)
@@ -34,7 +34,7 @@
     $app->setup_log('error,warn');
     ok !$app->debug, 'Not in debug mode';
     test_log_object($app->log,
-        fatal => 1, 
+        fatal => 1,
         error => 1,
         warn => 1,
         info => 0,