Change from use (compiletime) to require (runtime). Remove unnecessary
use line (not actually used in the module). Tweak comments.
Patch by kareila.
diffs (44 lines):
diff -r 6cc9672da71b -r cf5e3742f568 cgi-bin/DW/Controller.pm
--- a/cgi-bin/DW/Controller.pm Wed Nov 10 17:10:46 2010 +0800
+++ b/cgi-bin/DW/Controller.pm Wed Nov 10 17:26:39 2010 +0800
@@ -19,7 +19,6 @@ use strict;
use strict;
use warnings;
use Exporter;
-use DW::Routing;
use DW::Template;
use URI;
diff -r 6cc9672da71b -r cf5e3742f568 cgi-bin/DW/Controller/Admin.pm
--- a/cgi-bin/DW/Controller/Admin.pm Wed Nov 10 17:10:46 2010 +0800
+++ b/cgi-bin/DW/Controller/Admin.pm Wed Nov 10 17:26:39 2010 +0800
@@ -31,8 +31,10 @@ use strict;
use strict;
use warnings;
use DW::Controller;
-use DW::Routing;
use DW::Template;
+
+# this needs to be included at run time to avoid circular requirements
+require DW::Routing;
my $admin_pages = {};
@@ -179,7 +181,7 @@ sub admin_handler {
return DW::Template->render_template( 'admin/index.tt', $vars );
}
-=head2 C<< $class->register_regex( $scope, %opts ) >>
+=head2 C<< $class->register_admin_scope( $scope, %opts ) >>
Register an admin scope.
@@ -225,7 +227,7 @@ sub register_admin_scope {
=head2 C<< $class->register_admin_page( $scope, %opts ) >>
-Register an admin scope.
+Register an admin page.
Arguments:
_______________________________________________
dw-changelog mailing list
[email protected]
http://lists.dwscoalition.org/mailman/listinfo/dw-changelog
