Sebastian Zoglowek authored on13/01/2023 19:37:52
Showing5 changed files
... ...
@@ -29,7 +29,6 @@ use Contao\System;
29 29
 use Contao\Validator;
30 30
 use Psr\Log\LogLevel;
31 31
 
32
-
33 32
 /**
34 33
  * Class Member
35 34
  *
... ...
@@ -262,7 +261,7 @@ class Member extends Frontend
262 261
         $projectDir = System::getContainer()->getParameter('kernel.project_dir');
263 262
 
264 263
         // Check if member avatar exists
265
-        if(null === $objMember || null === $objMember->avatar || null === ($objFile = FilesModel::findByUuid($objMember->avatar)) || !\is_file($projectDir.'/'.$objFile->path))
264
+        if (null === $objMember || null === $objMember->avatar || null === ($objFile = FilesModel::findByUuid($objMember->avatar)) || !\is_file($projectDir.'/'.$objFile->path))
266 265
         {
267 266
             $objFile = !!($uuidDefault = Config::get('defaultAvatar')) ? FilesModel::findByUuid($uuidDefault) : null;
268 267
         }
... ...
@@ -294,7 +293,7 @@ class Member extends Frontend
294 293
         // ToDo: Merge logic with parseMemberAvatar
295 294
         $projectDir = System::getContainer()->getParameter('kernel.project_dir');
296 295
 
297
-        if(null === $objMember || null === $objMember->avatar || null === ($objFile = FilesModel::findByUuid($objMember->avatar)) || !\is_file($projectDir.'/'. $objFile->path))
296
+        if (null === $objMember || null === $objMember->avatar || null === ($objFile = FilesModel::findByUuid($objMember->avatar)) || !\is_file($projectDir.'/'. $objFile->path))
298 297
         {
299 298
             $objFile = !!($uuidDefault = Config::get('defaultAvatar')) ? FilesModel::findByUuid($uuidDefault) : null;
300 299
         }
... ...
@@ -313,7 +312,7 @@ class Member extends Frontend
313 312
      */
314 313
     public static function deleteAvatar(MemberModel $objMember): void
315 314
     {
316
-        if(!!$objMember->avatar)
315
+        if (!!$objMember->avatar)
317 316
         {
318 317
             $objFile = FilesModel::findByUuid($objMember->avatar) ?: '';
319 318
             $projectDir = System::getContainer()->getParameter('kernel.project_dir');
... ...
@@ -76,7 +76,7 @@ class ModuleDeleteAvatar extends Module
76 76
         $this->import(FrontendUser::class, 'User');
77 77
         $objMember = MemberModel::findByPk($this->User->id);
78 78
 
79
-        if(null === $objMember)
79
+        if (null === $objMember)
80 80
         {
81 81
             return '';
82 82
         }
... ...
@@ -110,7 +110,7 @@ class ModuleDeleteAvatar extends Module
110 110
             $objMember = MemberModel::findByPk($this->User->id);
111 111
 
112 112
             // Delete avatar if it exists
113
-            if(!!$objMember->avatar)
113
+            if (!!$objMember->avatar)
114 114
             {
115 115
                 Member::deleteAvatar($objMember);
116 116
                 // Unset avatar
... ...
@@ -124,7 +124,7 @@ class ModuleDeleteAvatar extends Module
124 124
         }
125 125
 
126 126
         // Confirmation message
127
-        if($session->isStarted() && $flashBag->has('mod_avatar_deleted')) {
127
+        if ($session->isStarted() && $flashBag->has('mod_avatar_deleted')) {
128 128
             $arrMessages = $flashBag->get('mod_avatar_deleted');
129 129
             $this->Template->message = $arrMessages[0];
130 130
         }
... ...
@@ -52,7 +52,7 @@ abstract class ModuleMemberExtension extends Module
52 52
 
53 53
         foreach ($arrMemberFields as $field)
54 54
         {
55
-            switch($field)
55
+            switch ($field)
56 56
             {
57 57
                 /*case 'homeDir':
58 58
                 case 'assignDir':
... ...
@@ -63,7 +63,7 @@ abstract class ModuleMemberExtension extends Module
63 63
                     break;
64 64
 
65 65
                 default:
66
-                    if($varValue = $objMember->{$field})
66
+                    if ($varValue = $objMember->{$field})
67 67
                     {
68 68
                         if (\is_array(($arrValue = StringUtil::deserialize($varValue))))
69 69
                         {
... ...
@@ -80,7 +80,7 @@ abstract class ModuleMemberExtension extends Module
80 80
 
81 81
         $objTemplate->fields = $arrFields;
82 82
 
83
-        if($this->jumpTo)
83
+        if ($this->jumpTo)
84 84
         {
85 85
             $objTemplate->link = $this->generateMemberUrl($objMember);
86 86
         }
... ...
@@ -85,7 +85,7 @@ class ModuleMemberList extends ModuleMemberExtension
85 85
 
86 86
         $arrGroups = StringUtil::deserialize($this->ext_groups);
87 87
 
88
-        if(empty($arrGroups) || !\is_array($arrGroups))
88
+        if (empty($arrGroups) || !\is_array($arrGroups))
89 89
         {
90 90
             $this->Template->empty = $GLOBALS['TL_LANG']['MSC']['emptyMemberList'];
91 91
             return;
... ...
@@ -99,13 +99,13 @@ class ModuleMemberList extends ModuleMemberExtension
99 99
 
100 100
         $arrMembers = [];
101 101
 
102
-        if(null !== $objMembers)
102
+        if (null !== $objMembers)
103 103
         {
104 104
             while($objMembers->next())
105 105
             {
106 106
                 $objMember = $objMembers->current();
107 107
 
108
-                if(!$this->checkMemberGroups($arrGroups, $objMember))
108
+                if (!$this->checkMemberGroups($arrGroups, $objMember))
109 109
                 {
110 110
                     continue;
111 111
                 }
... ...
@@ -156,7 +156,7 @@ class ModuleMemberList extends ModuleMemberExtension
156 156
             $this->Template->pagination = $objPagination->generate("\n  ");
157 157
         }
158 158
 
159
-        if(empty($arrMembers))
159
+        if (empty($arrMembers))
160 160
         {
161 161
             $this->Template->empty = $GLOBALS['TL_LANG']['MSC']['emptyMemberList'];
162 162
         }
... ...
@@ -173,14 +173,14 @@ class ModuleMemberList extends ModuleMemberExtension
173 173
      */
174 174
     private function checkMemberGroups(array $arrGroups, MemberModel $objMember): bool
175 175
     {
176
-        if(empty($arrGroups))
176
+        if (empty($arrGroups))
177 177
         {
178 178
             return false;
179 179
         }
180 180
 
181 181
         $arrMemberGroups = StringUtil::deserialize($objMember->groups);
182 182
 
183
-        if(!\is_array($arrMemberGroups) || !\count(array_intersect($arrGroups, $arrMemberGroups)))
183
+        if (!\is_array($arrMemberGroups) || !\count(array_intersect($arrGroups, $arrMemberGroups)))
184 184
         {
185 185
             return false;
186 186
         }
... ...
@@ -60,15 +60,16 @@ class InsertTagsListener
60 60
         $this->framework->initialize();
61 61
         $tokenChecker = System::getContainer()->get('contao.security.token_checker');
62 62
 
63
-        if($elements[1] !== 'member')
63
+        if ($elements[1] !== 'member')
64 64
         {
65 65
             return '';
66 66
         }
67 67
 
68
-        switch ($elements[2]) {
68
+        switch ($elements[2])
69
+        {
69 70
 
70 71
             case 'current':
71
-                if(!$tokenChecker->hasFrontendUser())
72
+                if (!$tokenChecker->hasFrontendUser())
72 73
                 {
73 74
                     return '';
74 75
                 }
... ...
@@ -76,7 +77,7 @@ class InsertTagsListener
76 77
                 break;
77 78
 
78 79
             default:
79
-                if(!\is_numeric($elements[2]))
80
+                if (!\is_numeric($elements[2]))
80 81
                 {
81 82
                     return '';
82 83
                 }
... ...
@@ -128,7 +129,7 @@ class InsertTagsListener
128 129
             ResizeConfiguration::MODE_CROP,
129 130
         ];
130 131
 
131
-        if(!!$mode && in_array($mode, $arrValidModes, true))
132
+        if (!!$mode && in_array($mode, $arrValidModes, true))
132 133
         {
133 134
             $arrSizes[] = $mode;
134 135
         }