Skip to content

Commit e2f0438

Browse files
hjuarez20enzolutions
authored andcommitted
[statistics] Add extra validation (#357)
1 parent 563c08b commit e2f0438

File tree

2 files changed

+14
-4
lines changed

2 files changed

+14
-4
lines changed

src/EventSubscriber/SaveStatisticsListener.php

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,8 +74,13 @@ public function saveStatistics(ConsoleTerminateEvent $event)
7474

7575
$configGlobalAsArray = $this->configurationManager->getConfigGlobalAsArray();
7676

77-
//Validate if the config is enable.
78-
if (is_null($configGlobalAsArray) || !$configGlobalAsArray['application']['statistics']['enabled']) {
77+
//Validate if the config is defined.
78+
if (is_null($configGlobalAsArray) || !isset($configGlobalAsArray['application']['statistics'])) {
79+
return;
80+
}
81+
82+
//Validate if the statistics is enabled.
83+
if (!isset($configGlobalAsArray['application']['statistics']['enabled']) || !$configGlobalAsArray['application']['statistics']['enabled']) {
7984
return;
8085
}
8186

src/EventSubscriber/SendStatisticsListener.php

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,8 +67,13 @@ public function calculateStatistics(ConsoleTerminateEvent $event)
6767
$date = date('Y-m-d');
6868
$configGlobalAsArray = $this->configurationManager->getConfigGlobalAsArray();
6969

70-
//Validate if the config is enable.
71-
if (is_null($configGlobalAsArray) || !$configGlobalAsArray['application']['statistics']['enabled']) {
70+
//Validate if the config is defined.
71+
if (is_null($configGlobalAsArray) || !isset($configGlobalAsArray['application']['statistics'])) {
72+
return;
73+
}
74+
75+
//Validate if the statistics is enabled.
76+
if (!isset($configGlobalAsArray['application']['statistics']['enabled']) || !$configGlobalAsArray['application']['statistics']['enabled']) {
7277
return;
7378
}
7479

0 commit comments

Comments
 (0)