Use PHP CS Fixer (#397)

* Use PHP CS Fixer * Autocommit PHP CS Fixer
humhub · Jun 27, 2024 · 0f85030 · 0f85030
1 parent d7b96ca
commit 0f85030
Show file tree

Hide file tree

Showing 91 changed files with 248 additions and 304 deletions.
diff --git a/.github/workflows/php-cs-fixer.yml b/.github/workflows/php-cs-fixer.yml
@@ -0,0 +1,7 @@
+name: PHP CS Fixer
+
+on: push
+
+jobs:
+  tests:
+    uses: humhub/actions/.github/workflows/module-php-cs-fixer.yml@main
diff --git a/Module.php b/Module.php
@@ -20,7 +20,6 @@
  */
 class Module extends \humhub\components\Module
 {
-
     /**
      * @inheritdoc
      */
@@ -82,11 +81,11 @@ public function getPermissions($contentContainer = null)
     {
         if (!$contentContainer) {
             return [
-                new StartConversation()
+                new StartConversation(),
             ];
-        } else if ($contentContainer instanceof User) {
+        } elseif ($contentContainer instanceof User) {
             return [
-                new SendMail()
+                new SendMail(),
             ];
         }
 
@@ -97,14 +96,14 @@ public function getNotifications()
     {
         return [
             MailNotification::class,
-            ConversationNotification::class
+            ConversationNotification::class,
         ];
     }
 
     /**
      * Determines showInTopNav is enabled or not
      *
-     * @return boolean is showInTopNav enabled
+     * @return bool is showInTopNav enabled
      */
     public function hideInTopNav()
     {

diff --git a/assets/MailMessengerAsset.php b/assets/MailMessengerAsset.php
@@ -17,6 +17,6 @@ class MailMessengerAsset extends AssetBundle
     ];
 
     public $depends = [
-        MailNotificationAsset::class
+        MailNotificationAsset::class,
     ];
 }
diff --git a/assets/MailNotificationAsset.php b/assets/MailNotificationAsset.php
@@ -24,7 +24,7 @@ class MailNotificationAsset extends AssetBundle
     ];
 
     public $depends = [
-        MailStyleAsset::class
+        MailStyleAsset::class,
     ];
 
     public static function register($view)
@@ -34,10 +34,10 @@ public static function register($view)
                 'url' => [
                     'count' => Url::toMessageCountUpdate(),
                     'list' => Url::toNotificationList(),
-                ]
-            ]
+                ],
+            ],
         ]);
 
         return parent::register($view);
     }
-}
+}
diff --git a/assets/MailStyleAsset.php b/assets/MailStyleAsset.php
@@ -19,6 +19,6 @@ class MailStyleAsset extends AssetBundle
     ];
 
     public $css = [
-        'humhub.mail.min.css'
+        'humhub.mail.min.css',
     ];
-}
+}
diff --git a/controllers/ConfigController.php b/controllers/ConfigController.php
@@ -20,7 +20,6 @@
  */
 class ConfigController extends \humhub\modules\admin\components\Controller
 {
-
     /**
      * Configuration action for super admins.
      */
@@ -35,5 +34,3 @@ public function actionIndex()
         return $this->render('index', ['model' => $form]);
     }
 }
-
-?>
diff --git a/controllers/InboxController.php b/controllers/InboxController.php
@@ -17,14 +17,13 @@
  */
 class InboxController extends Controller
 {
-
     /**
      * @inheritdoc
      */
     protected function getAccessRules()
     {
         return [
-            [ControllerAccess::RULE_LOGGED_IN_ONLY]
+            [ControllerAccess::RULE_LOGGED_IN_ONLY],
         ];
     }
 
@@ -37,7 +36,7 @@ protected function getAccessRules()
     public function actionIndex()
     {
         return ConversationInbox::widget([
-            'filter' => new InboxFilterForm()
+            'filter' => new InboxFilterForm(),
         ]);
     }
 
@@ -57,7 +56,7 @@ public function actionLoadMore()
 
         return $this->asJson([
             'result' => $result,
-            'isLast' => $filter->wasLastPage()
+            'isLast' => $filter->wasLastPage(),
         ]);
 
     }

diff --git a/controllers/MailController.php b/controllers/MailController.php
@@ -36,7 +36,6 @@
  */
 class MailController extends Controller
 {
-
     /**
      * @inheritdoc
      */
@@ -51,7 +50,7 @@ protected function getAccessRules()
     {
         return [
             [ControllerAccess::RULE_LOGGED_IN_ONLY],
-            [ControllerAccess::RULE_PERMISSION => StartConversation::class, 'actions' => ['create', 'add-user']]
+            [ControllerAccess::RULE_PERMISSION => StartConversation::class, 'actions' => ['create', 'add-user']],
         ];
     }
 
@@ -98,7 +97,7 @@ public function actionSeen()
         }
 
         return $this->asJson([
-            'messageCount' => UserMessage::getNewMessageCount()
+            'messageCount' => UserMessage::getNewMessageCount(),
         ]);
     }
 
@@ -111,7 +110,7 @@ public function actionUpdate($id, $from = null)
         return $this->renderAjaxContent(Messages::widget([
             'message' => $message,
             'entries' => $message->getEntryUpdates($from)->all(),
-            'showDateBadge' => false
+            'showDateBadge' => false,
         ]));
     }
 
@@ -127,7 +126,7 @@ public function actionLoadMore($id, $from)
 
         return $this->asJson([
             'result' => $result,
-            'isLast' => (count($entries) < Module::getModuleInstance()->conversationUpdatePageSize)
+            'isLast' => (count($entries) < Module::getModuleInstance()->conversationUpdatePageSize),
         ]);
     }
 
@@ -142,15 +141,15 @@ public function actionReply($id)
         if ($replyForm->load(Yii::$app->request->post()) && $replyForm->save()) {
             return $this->asJson([
                 'success' => true,
-                'content' => ConversationEntry::widget(['entry' => $replyForm->reply, 'showDateBadge' => $replyForm->reply->isFirstToday()])
+                'content' => ConversationEntry::widget(['entry' => $replyForm->reply, 'showDateBadge' => $replyForm->reply->isFirstToday()]),
             ]);
         }
 
         return $this->asJson([
             'success' => false,
             'error' => [
-                'message' => $replyForm->getFirstError('message')
-            ]
+                'message' => $replyForm->getFirstError('message'),
+            ],
         ]);
     }
 
@@ -163,7 +162,7 @@ public function actionUserList($id)
     {
         return $this->renderAjaxContent(UserListBox::widget([
             'query' => $this->getMessage($id, true)->getUsers(),
-            'title' => '<strong>' . Yii::t('MailModule.base', 'Participants') . '</strong>'
+            'title' => '<strong>' . Yii::t('MailModule.base', 'Participants') . '</strong>',
         ]));
     }
 
@@ -184,15 +183,15 @@ public function actionAddUser($id)
         if ($inviteForm->load(Yii::$app->request->post())) {
             if ($inviteForm->save()) {
                 return $this->asJson([
-                    'result' => ConversationHeader::widget(['message' => $message])
+                    'result' => ConversationHeader::widget(['message' => $message]),
                 ]);
             }
 
             return $this->asJson([
                 'success' => false,
                 'error' => [
-                    'message' => $inviteForm->getFirstError('recipients')
-                ]
+                    'message' => $inviteForm->getFirstError('recipients'),
+                ],
             ]);
         }
 
@@ -232,7 +231,7 @@ public function actionSearchUser($keyword, $id = null)
             'keyword' => $keyword,
             'permission' => (!Yii::$app->user->isAdmin()) ? new SendMail() : null,
             'disableFillUser' => true,
-            'disabledText' => Yii::t('MailModule.base', 'You are not allowed to start a conversation with this user.')
+            'disabledText' => Yii::t('MailModule.base', 'You are not allowed to start a conversation with this user.'),
         ]);
 
         // Disable already participating users
@@ -266,7 +265,7 @@ private function checkMessagePermissions($message)
      *
      * @param type $message
      * @param type $user
-     * @return boolean
+     * @return bool
      */
     private function isParticipant($message, $user)
     {
@@ -355,7 +354,7 @@ public function actionMarkUnread($id)
 
         return $this->asJson([
             'success' => true,
-            'redirect' => $nextReadMessage ? Url::toMessenger($nextReadMessage) : Url::to(['/dashboard'])
+            'redirect' => $nextReadMessage ? Url::toMessenger($nextReadMessage) : Url::to(['/dashboard']),
         ]);
     }
 
@@ -372,7 +371,7 @@ public function actionPin($id)
 
         return $this->asJson([
             'success' => true,
-            'redirect' => Url::toMessenger($message)
+            'redirect' => Url::toMessenger($message),
         ]);
     }
 
@@ -389,7 +388,7 @@ public function actionUnpin($id)
 
         return $this->asJson([
             'success' => true,
-            'redirect' => Url::toMessenger($message)
+            'redirect' => Url::toMessenger($message),
         ]);
     }
 
@@ -411,7 +410,7 @@ public function actionLeave($id)
 
         return $this->asJson([
             'success' => true,
-            'redirect' => Url::toMessenger()
+            'redirect' => Url::toMessenger(),
         ]);
     }
 
@@ -439,8 +438,8 @@ public function actionEditEntry($id)
                 'success' => true,
                 'content' => ConversationEntry::widget([
                     'entry' => $entry,
-                    'showDateBadge' => false
-                ])
+                    'showDateBadge' => false,
+                ]),
             ]);
         }
 
@@ -472,7 +471,7 @@ public function actionDeleteEntry($id)
         $entry->message->deleteEntry($entry);
 
         return $this->asJson([
-            'success' => true
+            'success' => true,
         ]);
     }
 
@@ -520,7 +519,7 @@ private function getNextReadMessage($id): ?Message
             ->andWhere('user_message.last_viewed >= message.updated_at')
             ->orderBy([
                 'user_message.pinned' => SORT_DESC,
-                'message.updated_at' => SORT_DESC
+                'message.updated_at' => SORT_DESC,
             ])
             ->one();
     }

diff --git a/controllers/TagController.php b/controllers/TagController.php
@@ -30,7 +30,7 @@ class TagController extends Controller
     protected function getAccessRules()
     {
         return [
-            [ControllerAccess::RULE_LOGGED_IN_ONLY]
+            [ControllerAccess::RULE_LOGGED_IN_ONLY],
         ];
     }
 
@@ -117,7 +117,7 @@ public function actionEditConversation($messageId)
 
         if ($model->load(Yii::$app->request->post()) && $model->save()) {
             return ModalClose::widget([
-                'script' => '$("#' . ConversationTags::ID . '").replaceWith(\'' . ConversationTags::widget(['message' => $message]) . '\');'
+                'script' => '$("#' . ConversationTags::ID . '").replaceWith(\'' . ConversationTags::widget(['message' => $message]) . '\');',
             ]);
         }
 

diff --git a/controllers/rest/EntryController.php b/controllers/rest/EntryController.php
@@ -14,13 +14,11 @@
 use Yii;
 use yii\web\HttpException;
 
-
 /**
  * Class EntryController
  */
 class EntryController extends BaseController
 {
-
     /**
      * Get all entries of the conversation
      *
@@ -151,7 +149,7 @@ protected function getMessageEntry($messageId, $entryId)
 
         $entry = MessageEntry::findOne([
             'id' => $entryId,
-            'message_id' => $message->id
+            'message_id' => $message->id,
         ]);
 
         if (!$entry) {
@@ -160,4 +158,4 @@ protected function getMessageEntry($messageId, $entryId)
 
         return $entry;
     }
-}
+}
diff --git a/controllers/rest/MessageController.php b/controllers/rest/MessageController.php
@@ -16,13 +16,11 @@
 use yii\web\ForbiddenHttpException;
 use yii\web\HttpException;
 
-
 /**
  * Class MessageController
  */
 class MessageController extends BaseController
 {
-
     /**
      * Get list of mail conversations
      *
@@ -102,4 +100,4 @@ public static function getMessage($id)
 
         return $message;
     }
-}
+}
-Original file line number
+Diff line change
@@ Expand Up / @@ -20,7 +20,6 @@ @@
      */
     class ConfigController extends \humhub\modules\admin\components\Controller
     {
         /**
          * Configuration action for super admins.
          */
@@ Expand All / @@ -35,5 +34,3 @@ public function actionIndex() @@
             return $this->render('index', ['model' => $form]);
         }
     }
-    ?>