[Rt-commit] rt branch 5.0/asset-search-chart updated. rt-5.0.5-100-g68fa9b30f2

BPS Git Server git at git.bestpractical.com
Tue Jan 9 21:37:49 UTC 2024


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "rt".

The branch, 5.0/asset-search-chart has been updated
       via  68fa9b30f2863f6d628b063ff08ce2e75d4e64aa (commit)
      from  2765706a1baeafd2cf778a576acc037f79a18078 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 68fa9b30f2863f6d628b063ff08ce2e75d4e64aa
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Tue Jan 9 16:26:06 2024 -0500

    Log about the inaccurate chart data if UseSQLForACLChecks is disabled

diff --git a/lib/RT/Report.pm b/lib/RT/Report.pm
index 81f9b74c43..5e76fa29ed 100644
--- a/lib/RT/Report.pm
+++ b/lib/RT/Report.pm
@@ -606,6 +606,9 @@ sub SetupGroupings {
         }
         $self->{'_sql_current_user_can_see_applied'} = 1
     }
+    elsif ( !RT->Config->Get('UseSQLForACLChecks') ) {
+        RT->Logger->notice('UseSQLForACLChecks is disabled, results might be inaccurate');
+    }
 
     my %res = $self->_SetupGroupings(%args);
 

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

Summary of changes:
 lib/RT/Report.pm | 3 +++
 1 file changed, 3 insertions(+)


hooks/post-receive
-- 
rt


More information about the rt-commit mailing list