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

[t0m at dev.catalyst.perl.org]

Author: t0m
Date: 2009-01-04 21:59:02 +0000 (Sun, 04 Jan 2009)
New Revision: 9009

Modified:
   Catalyst-Runtime/5.80/trunk/Makefile.PL
   Catalyst-Runtime/5.80/trunk/TODO
   Catalyst-Runtime/5.80/trunk/lib/Catalyst.pm
   Catalyst-Runtime/5.80/trunk/t/unit_core_plugin.t
Log:
Warnings about deprecated runtime plugins

Modified: Catalyst-Runtime/5.80/trunk/Makefile.PL
===================================================================
--- Catalyst-Runtime/5.80/trunk/Makefile.PL	2009-01-04 21:43:22 UTC (rev 9008)
+++ Catalyst-Runtime/5.80/trunk/Makefile.PL	2009-01-04 21:59:02 UTC (rev 9009)
@@ -155,7 +155,7 @@
     return unless $found;
 
     # More or less copied from Module::Build
-    return if  $ENV{PERL_MM_USE_DEFAULT};
+    return if $ENV{PERL_MM_USE_DEFAULT};
     return unless -t STDIN && (-t STDOUT || !(-f STDOUT || -c STDOUT));
 
     sleep 4;

Modified: Catalyst-Runtime/5.80/trunk/TODO
===================================================================
--- Catalyst-Runtime/5.80/trunk/TODO	2009-01-04 21:43:22 UTC (rev 9008)
+++ Catalyst-Runtime/5.80/trunk/TODO	2009-01-04 21:59:02 UTC (rev 9009)
@@ -1,6 +1,3 @@
-Pending patches:
-  - meta test for MX::Emulate::CAF needed by Catalyst::Plugin::Cache::Curried
-    
 Back-compat investigation / known issues:
 
   - Get engines tested:
@@ -39,9 +36,6 @@
        an exception. Needs a test case (Caelum)
      
      - Catalyst::Plugin::Authentication new release.
-     
-     - Deprecation/Upgrade notice in Catalyst Makefile.PL for known broken
-       versions of plugins.
 
   - Issues with TWMC not being loaded when it used to be in 5.70 
     (Bill Moseley)
@@ -54,7 +48,10 @@
      This obviously needs better tests :/
 
 Cleanups:
-    
+
+  - Allow turning deprecated warnings off with use Catalyst 
+    qw/-NoDeprecationWarnings/
+
   - Update Test suite to not assume MyApp ISA Controller
     - After that set up attr handlers that will output helpful error messages 
       when you do it as well as how to fix it.
@@ -66,8 +63,6 @@
   - Find whatever in the test suite craps a file called GLOB(0xfoo) onto my 
     disk and fix it. (Believed to be in the optional TEST_HTTP tests?) (t0m)
 
-  - Deprecate runtime plugins with warning.
-
 Documentation:
 
    - extends in components with attributes must be inside a BEGIN block.
@@ -91,3 +86,6 @@
   - Moosified test application?
 
   - Test warning from back-compat methods in Catalyst::Dispatcher
+  
+  - Test warnings from plugin method on Catalyst
+

Modified: Catalyst-Runtime/5.80/trunk/lib/Catalyst.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/lib/Catalyst.pm	2009-01-04 21:43:22 UTC (rev 9008)
+++ Catalyst-Runtime/5.80/trunk/lib/Catalyst.pm	2009-01-04 21:59:02 UTC (rev 9009)
@@ -862,11 +862,19 @@
     MyApp->plugin( 'prototype', 'HTML::Prototype' );
 
     $c->prototype->define_javascript_functions;
+    
+B<Note:> This method of adding plugins is deprecated. The ability
+to add plugins like this B<will be removed> in a Catalyst 5.9.
+Please do not use this functionality in new code.
 
 =cut
 
 sub plugin {
     my ( $class, $name, $plugin, @args ) = @_;
+
+    # See block comment in t/unit_core_plugin.t    
+    $class->log->debug(qq/Adding plugin using the ->plugin method is deprecated, and will be removed in Catalyst 5.9/);
+    
     $class->_register_plugin( $plugin, 1 );
 
     eval { $plugin->import };

Modified: Catalyst-Runtime/5.80/trunk/t/unit_core_plugin.t
===================================================================
--- Catalyst-Runtime/5.80/trunk/t/unit_core_plugin.t	2009-01-04 21:43:22 UTC (rev 9008)
+++ Catalyst-Runtime/5.80/trunk/t/unit_core_plugin.t	2009-01-04 21:59:02 UTC (rev 9009)
@@ -19,6 +19,8 @@
 use Catalyst::Test qw/PluginTestApp/;
 
 ok( get("/compile_time_plugins"), "get ok" );
+# FIXME - Run time plugin support is insane, and should be removed
+#         for Catalyst 5.9
 ok( get("/run_time_plugins"),     "get ok" );
 
 use_ok 'TestApp';