[Bps-public-commit] rt-extension-rightsdebugger branch, master, updated. ff8a000eb0047f47bbca086a20ce7f301a3eaf5f

[Shawn Moore]

The branch, master has been updated
       via  ff8a000eb0047f47bbca086a20ce7f301a3eaf5f (commit)
      from  d96cab0df9040db7f9e676f4cca9ddcb6b619cc2 (commit)

Summary of changes:
 lib/RT/Extension/RightsDebugger.pm | 4 ----
 1 file changed, 4 deletions(-)

- Log -----------------------------------------------------------------
commit ff8a000eb0047f47bbca086a20ce7f301a3eaf5f
Author: Shawn M Moore <shawn at bestpractical.com>
Date:   Thu Mar 16 18:49:57 2017 +0000

    Remove unnecessary performance optimization
    
    This can trigger too early - and thus produce a truncated set of results - if
    for some reason AddRecord doesn't include 100% of the records.

diff --git a/lib/RT/Extension/RightsDebugger.pm b/lib/RT/Extension/RightsDebugger.pm
index 3090a0c..8f75289 100644
--- a/lib/RT/Extension/RightsDebugger.pm
+++ b/lib/RT/Extension/RightsDebugger.pm
@@ -527,10 +527,6 @@ sub Search {
         push @results, $serialized;
     }
 
-    # if we didn't fill the whole page, then we know there are
-    # no more rows to consider
-    undef $continueAfter if $ACL->Count < $ACL->RowsPerPage;
-
     return {
         results => \@results,
         continueAfter => $continueAfter,

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