From 1e6c1cf78a76370af278f8debb4c3a11cb6798da Mon Sep 17 00:00:00 2001 From: =?utf8?q?Sven=20Sch=C3=B6ling?= Date: Mon, 7 Aug 2017 16:31:42 +0200 Subject: [PATCH] GetModels Paginated: Historische Filterduplizierung deaktivieren MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Die war noch aus der Zeit vor GetModels, und war nötig damit Paginating die gleichen Filter hatte wie das Hauptquery. Mittlerweile ist das aber nicht mehr nötig, weil die GetModels-Architektur die einzelnen Plugins eh chained. Beschleunigt einige Queries, wo der Queryplaner in Bedingungen in Joins die Duplizierung nicht automatisch entfernen kann, und dann in nested loops nachprüfen muss. --- SL/Controller/Helper/GetModels/Paginated.pm | 2 -- 1 file changed, 2 deletions(-) diff --git a/SL/Controller/Helper/GetModels/Paginated.pm b/SL/Controller/Helper/GetModels/Paginated.pm index d900c18be..86b248c20 100644 --- a/SL/Controller/Helper/GetModels/Paginated.pm +++ b/SL/Controller/Helper/GetModels/Paginated.pm @@ -57,8 +57,6 @@ sub finalize { # try to use Filtered if available and nothing else is configured, but don't # blow up if the controller does not use Filtered my %paginate_args = ref($self->paginate_args) eq 'CODE' ? %{ $self->paginate_args->($self) } - : $self->paginate_args eq '__FILTER__' - && $self->get_models->filtered ? $self->get_models->filtered->read_params : $self->paginate_args ne '__FILTER__' ? do { my $sub = $self->paginate_args; %{ $self->get_models->controller->$sub() } } : (); -- 2.20.1