Skip to content

Commit 4cfe191

Browse files
committed
ModifierChain: switch parameter order
1 parent a8ea5be commit 4cfe191

File tree

3 files changed

+9
-11
lines changed

3 files changed

+9
-11
lines changed

application/controllers/ConfigurationController.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -689,8 +689,8 @@ protected function showRules(
689689
$modifiers,
690690
$url,
691691
$this->getServerRequest(),
692+
$modifierRuleStore,
692693
$sampleObject,
693-
$modifierRuleStore
694694
);
695695
if ($info->hasModifications()) {
696696
$this->redirectNow($this->url());

library/Eventtracker/Modifier/ModifierChain.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,11 @@
22

33
namespace Icinga\Module\Eventtracker\Modifier;
44

5-
use gipfl\Json\JsonString;
5+
use Countable;
66
use JsonSerializable;
77
use stdClass;
88

9-
class ModifierChain implements JsonSerializable, \Countable
9+
class ModifierChain implements JsonSerializable, Countable
1010
{
1111
protected array $modifiers = [];
1212

@@ -125,7 +125,7 @@ public function jsonSerialize()
125125
return $result;
126126
}
127127

128-
public function count()
128+
public function count(): int
129129
{
130130
return count($this->modifiers);
131131
}

library/Eventtracker/Web/Table/ChannelRulesTable.php

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,11 @@
1414
use gipfl\Web\Widget\Hint;
1515
use Icinga\Module\Eventtracker\Data\PlainObjectRenderer;
1616
use Icinga\Module\Eventtracker\Modifier\ModifierChain;
17-
use Icinga\Module\Eventtracker\Modifier\ModifierRegistry;
1817
use Icinga\Module\Eventtracker\Modifier\ModifierRuleStore;
1918
use Icinga\Module\Eventtracker\Web\Form\ChannelConfigForm;
2019
use Icinga\Module\Eventtracker\Web\Form\InstanceInlineForm;
2120
use ipl\Html\Html;
2221
use ipl\Html\Table;
23-
use mysql_xdevapi\CrudOperationBindable;
2422
use Psr\Http\Message\ServerRequestInterface;
2523

2624
class ChannelRulesTable extends Table
@@ -37,7 +35,7 @@ class ChannelRulesTable extends Table
3735
protected ModifierChain $modifierChain;
3836
protected Url $url;
3937
protected ServerRequestInterface $request;
40-
protected ModifierRuleStore $ruleStore;
38+
protected ModifierRuleStore $modifierRuleStore;
4139
private ?object $sampleObject;
4240

4341
public function getModifierChain(): ModifierChain
@@ -48,11 +46,11 @@ public function getModifierChain(): ModifierChain
4846
private bool $hasBeenModified = false;
4947

5048
public function __construct(
51-
ModifierChain $modifierChain,
52-
Url $url,
49+
ModifierChain $modifierChain,
50+
Url $url,
5351
ServerRequestInterface $request,
54-
?object $sampleObject = null,
55-
ModifierRuleStore $modifierRuleStore
52+
ModifierRuleStore $modifierRuleStore,
53+
?object $sampleObject = null
5654
) {
5755
$this->modifierChain = $modifierChain;
5856
$this->url = $url;

0 commit comments

Comments
 (0)