[Rt-commit] rt branch, 3.9-trunk, updated. rt-3.9.7-993-g41ed12e

Shawn Moore sartak at bestpractical.com
Tue Dec 21 18:20:24 EST 2010


The branch, 3.9-trunk has been updated
       via  41ed12ed639dabd9ac396cef52c9bb528f8a0d28 (commit)
      from  438a9f30d49ff6f56abb013b228f9e5dc77c6989 (commit)

Summary of changes:
 lib/RT/Interface/Web.pm |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

- Log -----------------------------------------------------------------
commit 41ed12ed639dabd9ac396cef52c9bb528f8a0d28
Author: Shawn M Moore <sartak at bestpractical.com>
Date:   Tue Dec 21 18:18:45 2010 -0500

    Try to load Module::Refresh before calling ->refresh
    
        Otherwise, in some circumstances, we won't have Module::Refresh
        loaded and it'll break. This happened in t/web/squish.t which
        enables DevelMode and not having Module::Refresh loaded broke a
        test. Tests being otherwise warnings-clean caught this. :)

diff --git a/lib/RT/Interface/Web.pm b/lib/RT/Interface/Web.pm
index 7292780..3311021 100755
--- a/lib/RT/Interface/Web.pm
+++ b/lib/RT/Interface/Web.pm
@@ -191,7 +191,10 @@ sub WebExternalAutoInfo {
 sub HandleRequest {
     my $ARGS = shift;
 
-    Module::Refresh->refresh if RT->Config->Get('DevelMode');
+    if (RT->Config->Get('DevelMode')) {
+        require Module::Refresh;
+        Module::Refresh->refresh;
+    }
 
     $HTML::Mason::Commands::r->content_type("text/html; charset=utf-8");
 

-----------------------------------------------------------------------


More information about the Rt-commit mailing list