[Rt-commit] r5913 - in rt/branches/3.6-RELEASE: . lib

[jesse at bestpractical.com]

Author: jesse
Date: Mon Sep 11 19:55:41 2006
New Revision: 5913

Modified:
   rt/branches/3.6-RELEASE/   (props changed)
   rt/branches/3.6-RELEASE/etc/RT_Config.pm.in
   rt/branches/3.6-RELEASE/lib/RT.pm.in

Log:
 r27407 at pinglin:  jesse | 2006-09-12 00:55:29 +0100
 RT Essentials mistakenly suggests that users use '/' as an RT::WebPath.
 
 Do what they mean if they screw up and do that.
 


Modified: rt/branches/3.6-RELEASE/etc/RT_Config.pm.in
==============================================================================
--- rt/branches/3.6-RELEASE/etc/RT_Config.pm.in	(original)
+++ rt/branches/3.6-RELEASE/etc/RT_Config.pm.in	Mon Sep 11 19:55:41 2006
@@ -327,8 +327,11 @@
 # documents.
 
 # If you're putting the web ui somewhere other than at the root of
-# your server
-# $WebPath requires a leading / but no trailing /
+# your server, you should set $WebPath to the path you'll be 
+# serving RT at.
+# $WebPath requires a leading / but no trailing /.
+#
+# In most cases, you should leave $WebPath set to '' (an empty value).
 
 Set($WebPath , "");
 

Modified: rt/branches/3.6-RELEASE/lib/RT.pm.in
==============================================================================
--- rt/branches/3.6-RELEASE/lib/RT.pm.in	(original)
+++ rt/branches/3.6-RELEASE/lib/RT.pm.in	Mon Sep 11 19:55:41 2006
@@ -133,6 +133,12 @@
     }
     require $CORE_CONFIG_FILE
       || die ("Couldn't load RT config file '$CORE_CONFIG_FILE'\n$@");
+
+
+    # RT::Essentials mistakenly recommends that WebPath be set to '/'.
+    # If the user does that, do what they mean.
+    $RT::WebPath = '' if ($RT::WebPath eq '/');
+
     RT::I18N->Init;
 }