[Jifty-commit] r6801 - jifty/trunk/lib/Jifty

Jifty commits jifty-commit at lists.jifty.org
Tue Apr 21 01:35:47 EDT 2009


Author: sunnavy
Date: Tue Apr 21 01:35:46 2009
New Revision: 6801

Modified:
   jifty/trunk/lib/Jifty/Config.pm

Log:
comment the warning when ApplicationClass::Config is not a sub-class of Jifty::Config

Modified: jifty/trunk/lib/Jifty/Config.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Config.pm	(original)
+++ jifty/trunk/lib/Jifty/Config.pm	Tue Apr 21 01:35:46 2009
@@ -259,8 +259,11 @@
     if ( $found && $app_class->isa('Jifty::Config') ) {
         bless $self, $app_class;
     } elsif ( $found ) {
-        warn "You have $app_class, however it's not an sub-class of Jifty::Config."
-            ." Read `perldoc Jifty::Config` about subclassing. Skipping.";
+# XXX this warning is not always useful, sometimes annoying, 
+# e.g. RT has its own config mechanism, we don't want to sub-class
+# Jifty::Config at all.
+#        warn "You have $app_class, however it's not an sub-class of Jifty::Config."
+#            ." Read `perldoc Jifty::Config` about subclassing. Skipping.";
     }
 
     # post load hook for sub-classes


More information about the Jifty-commit mailing list