From: Moritz Bunkus <m.bunkus@linet-services.de>
Date: Fri, 3 Jul 2015 08:23:14 +0000 (+0200)
Subject: GetModels: Disablen von Plugins auch für undef
X-Git-Tag: release-3.3.0beta~32^2~15
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=96f8eef08701d2296bebee75db99557bffb92278;p=kivitendo-erp.git

GetModels: Disablen von Plugins auch für undef

Die Dokumentation sagt Folgendes zum Deaktivieren von Plugins:

> Configuration for plugins. If the option for any plugin is omitted,
> it defaults to enabled and is configured by default. Giving a
> falsish value as first argument will disable the plugin.

Das bisherige Verhalten war aber, dass bei Übergabe von
undef (z.B. »paginated => undef«) die Standardeinstellungen gegriffen
haben.
---

diff --git a/SL/Controller/Helper/GetModels.pm b/SL/Controller/Helper/GetModels.pm
index 861834b83..cb73ab354 100644
--- a/SL/Controller/Helper/GetModels.pm
+++ b/SL/Controller/Helper/GetModels.pm
@@ -95,7 +95,9 @@ sub init {
 
   my @plugins;
   for my $plugin (qw(filtered sorted paginated)) {
-    next unless my $spec = delete $params{$plugin} // {};
+    next if exists($params{$plugin}) && !$params{$plugin};
+
+    my $spec         = delete $params{$plugin} // {};
     my $plugin_class = "SL::Controller::Helper::GetModels::" . ucfirst $plugin;
     push @plugins, $self->$plugin($plugin_class->new(%$spec, get_models => $self));
   }