[Rt-commit] rt branch, master, updated. rt-4.0.0rc7-246-g3c61239

[Alex Vandiver]

The branch, master has been updated
       via  3c61239a784c7c506b27a0fadbafa4a16324e51d (commit)
      from  de1482c9c02dc7cb0c4582ba9db24e31629b04c6 (commit)

Summary of changes:
 etc/RT_Config.pm.in |   11 ++++-------
 1 files changed, 4 insertions(+), 7 deletions(-)

- Log -----------------------------------------------------------------
commit 3c61239a784c7c506b27a0fadbafa4a16324e51d
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Mon Mar 28 12:45:41 2011 -0400

    Flip UseSQLForACLChecks to on by default
    
    It has been "experimental" for long enough, and fixes a common class of
    reported bugs.

diff --git a/etc/RT_Config.pm.in b/etc/RT_Config.pm.in
index 5a1c261..2fa417d 100755
--- a/etc/RT_Config.pm.in
+++ b/etc/RT_Config.pm.in
@@ -1095,16 +1095,13 @@ Set($HomepageComponents, [qw(QuickCreate Quicksearch MyAdminQueues MySupportQueu
 =item C<$UseSQLForACLChecks>
 
 Historically, ACLs were checked on display, which could lead to empty
-search pages and wrong ticket counts.  Set C<$UseSQLForACLChecks> to 1
-to limit search results in SQL instead, which eliminates these
-problems.
-
-This option is still relatively new; it may result in performance
-problems in some cases, or significant speedups in others.
+search pages and wrong ticket counts.  Set C<$UseSQLForACLChecks> to 0
+to go back to this method; this will reduce the complexity of the
+generated SQL statements, at the cost of the aforementioned bugs.
 
 =cut
 
-Set($UseSQLForACLChecks, undef);
+Set($UseSQLForACLChecks, 1);
 
 =item C<$TicketsItemMapSize>
 

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