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

[rafl at dev.catalyst.perl.org]

Author: rafl
Date: 2009-06-29 22:05:02 +0000 (Mon, 29 Jun 2009)
New Revision: 10730

Modified:
   Catalyst-Runtime/5.80/trunk/lib/Catalyst/Action.pm
   Catalyst-Runtime/5.80/trunk/lib/Catalyst/DispatchType/Path.pm
Log:
Remove <=> overloading for actions.

Modified: Catalyst-Runtime/5.80/trunk/lib/Catalyst/Action.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/lib/Catalyst/Action.pm	2009-06-29 19:57:17 UTC (rev 10729)
+++ Catalyst-Runtime/5.80/trunk/lib/Catalyst/Action.pm	2009-06-29 22:05:02 UTC (rev 10730)
@@ -37,9 +37,6 @@
     # Codulate to execute to invoke the encapsulated action coderef
     '&{}' => sub { my $self = shift; sub { $self->execute(@_); }; },
 
-    # Which action takes precedence
-    '<=>' => 'compare',
-
     # Make general $stuff still work
     fallback => 1,
 
@@ -124,8 +121,6 @@
 Compares 2 actions based on the value of the C<Args> attribute, with no C<Args>
 having the highest precedence.
 
-C<< <=> >> is overloaded to use this method.
-
 =head2 namespace
 
 Returns the private namespace this action lives in.

Modified: Catalyst-Runtime/5.80/trunk/lib/Catalyst/DispatchType/Path.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/lib/Catalyst/DispatchType/Path.pm	2009-06-29 19:57:17 UTC (rev 10729)
+++ Catalyst-Runtime/5.80/trunk/lib/Catalyst/DispatchType/Path.pm	2009-06-29 22:05:02 UTC (rev 10730)
@@ -119,7 +119,7 @@
     $path =~ s{(?<=[^/])/+\z}{};
 
     $self->_paths->{$path} = [
-        sort { $a <=> $b } ($action, @{ $self->_paths->{$path} || [] })
+        sort { $a->compare($b) } ($action, @{ $self->_paths->{$path} || [] })
     ];
 
     return 1;