Title: [opsview] [9608] Don't extend from Catalyst::Controller when the use base'd chain already does this.
Revision
9608
Author
pknight
Date
2012-07-23 16:37:42 +0100 (Mon, 23 Jul 2012)

Log Message

Don't extend from Catalyst::Controller when the use base'd chain already does this. Avoids a C3 MRO clash.

Modified Paths

Modified: trunk/opsview-web/lib/Opsview/Web/Controller/State/Service.pm
===================================================================
--- trunk/opsview-web/lib/Opsview/Web/Controller/State/Service.pm	2012-07-23 14:16:47 UTC (rev 9607)
+++ trunk/opsview-web/lib/Opsview/Web/Controller/State/Service.pm	2012-07-23 15:37:42 UTC (rev 9608)
@@ -21,20 +21,16 @@
 package Opsview::Web::Controller::State::Service;
 use strict;
 use warnings;
-use Moose;
-use namespace::autoclean;
-BEGIN { extends 'Catalyst::Controller' }
-
 use MRO::Compat;
 
-# use base qw(
-#   Opsview::Web::ControllerBase::Menus
-#   Opsview::Web::ControllerBase::Chained
-#   Opsview::Web::ControllerBase::State
-#   Opsview::Web::ControllerBase::Detail
-#   Opsview::Web::ControllerBase::Downtime
-#   Opsview::Web::ControllerBase::Service
-# );
+use base qw(
+  Opsview::Web::ControllerBase::Menus
+  Opsview::Web::ControllerBase::Chained
+  Opsview::Web::ControllerBase::State
+  Opsview::Web::ControllerBase::Detail
+  Opsview::Web::ControllerBase::Downtime
+  Opsview::Web::ControllerBase::Service
+);
 
 __PACKAGE__->config( model_name => "RuntimeDB::Service", );
 

_______________________________________________
Opsview-checkins mailing list
[email protected]
http://lists.opsview.org/lists/listinfo/opsview-checkins

Reply via email to