Merge pull request #161 from Sterc/fix-143

gpsietzema · web-flow · commit 41e1802cefc1 · 2018-09-04T13:10:31.000+02:00
Fixing not empty check 143
diff --git a/core/components/stercseo/model/stercseo/stercseo.class.php b/core/components/stercseo/model/stercseo/stercseo.class.php
@@ -586,7 +586,7 @@ public function checkUserAccess($user = false)
         if (!$user) {
             $user = $this->modx->getUser();
         }
-        $exclUsergroups = explode(',', $this->modx->getOption('stercseo.hide_from_usergroups'));
+        $exclUsergroups = array_filter(explode(',', $this->modx->getOption('stercseo.hide_from_usergroups')));
         if (!empty($exclUsergroups)) {
             foreach ($exclUsergroups as $exclUserGroup) {
                 if ($user->isMember($exclUserGroup)) {

Original file line number	Diff line number	Diff line change
`@@ -586,7 +586,7 @@ public function checkUserAccess($user = false)`
`586`	`586`	`if (!$user) {`
`587`	`587`	`$user = $this->modx->getUser();`
`588`	`588`	`}`
`589`		`- $exclUsergroups = explode(',', $this->modx->getOption('stercseo.hide_from_usergroups'));`
	`589`	`+ $exclUsergroups = array_filter(explode(',', $this->modx->getOption('stercseo.hide_from_usergroups')));`
`590`	`590`	`if (!empty($exclUsergroups)) {`
`591`	`591`	`foreach ($exclUsergroups as $exclUserGroup) {`
`592`	`592`	`if ($user->isMember($exclUserGroup)) {`