Use PHP CS Fixer

.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

Events.php

@@ -35,13 +35,13 @@ public static function onWallEntryControlsInit($event)
 
         if($object->content->canEdit()) {
             $event->sender->addWidget(CloseButton::class, [
-                'poll' => $object
+                'poll' => $object,
             ]);
         }
 
         if($object->isResetAllowed()) {
             $event->sender->addWidget(ResetButton::class, [
-                'poll' => $object
+                'poll' => $object,
             ]);
         }
     }
@@ -121,7 +121,7 @@ public static function onSampleDataInstall($event)
         $poll->newAnswers = [
             Yii::t('PollsModule.events', "To Daniel"),
             Yii::t('PollsModule.events', "Club A Steakhouse"),
-            Yii::t('PollsModule.events', "Pisillo Italian Panini")
+            Yii::t('PollsModule.events', "Pisillo Italian Panini"),
         ];
         $poll->content->container = $space;
         $poll->allow_multiple = Yii::$app->request->post('allowMultiple', 0);

Module.php

@@ -21,7 +21,6 @@
  */
 class Module extends ContentContainerModule
 {
-
     /**
      * @inheritdoc
      */
@@ -42,7 +41,7 @@ public function getContentContainerTypes()
     {
         return [
             Space::class,
-            User::class
+            User::class,
         ];
     }
 
@@ -77,7 +76,7 @@ public function getPermissions($contentContainer = null)
     {
         if ($contentContainer) {
             return [
-                new permissions\CreatePoll()
+                new permissions\CreatePoll(),
             ];
         }
 

activities/NewVote.php

@@ -14,12 +14,11 @@
 
 class NewVote extends BaseActivity implements ConfigurableActivityInterface
 {
-
     /**
      * @inheritdoc
      */
     public $moduleId = 'polls';
-    
+
     /**
      * @inheritdoc
      */

assets/PollsAsset.php

@@ -14,12 +14,11 @@
 
 class PollsAsset extends AssetBundle
 {
-
     public $jsOptions = ['position' => \yii\web\View::POS_END];
     public $sourcePath = '@polls/resources';
     public $css = [];
     public $js = [
-        'js/humhub.polls.js'
+        'js/humhub.polls.js',
     ];
 
     /**
@@ -30,8 +29,8 @@ public static function register($view)
     {
         $view->registerJsConfig('polls', [
             'text' => [
-                'warn.answer_required' => Yii::t('PollsModule.base', 'At least one answer is required')
-            ]
+                'warn.answer_required' => Yii::t('PollsModule.base', 'At least one answer is required'),
+            ],
         ]);
         return parent::register($view);
     }

config.php

@@ -15,5 +15,5 @@
         ['class' => IntegrityController::class, 'event' => IntegrityController::EVENT_ON_RUN, 'callback' => ['humhub\modules\polls\Events', 'onIntegrityCheck']],
         ['class' => 'humhub\modules\installer\controllers\ConfigController', 'event' => 'install_sample_data', 'callback' => ['humhub\modules\polls\Events', 'onSampleDataInstall']],
         ['class' => 'humhub\modules\rest\Module', 'event' => 'restApiAddRules', 'callback' => ['humhub\modules\polls\Events', 'onRestApiAddRules']],
-    ]
+    ],
 ];

controllers/PollController.php

@@ -24,7 +24,6 @@
  */
 class PollController extends ContentContainerController
 {
-
     public function actionCreateForm()
     {
         if (!(new Poll($this->contentContainer))->content->canEdit()) {
@@ -47,7 +46,7 @@ public function actionCreate()
         if (!$this->contentContainer->permissionManager->can(new CreatePoll())) {
             throw new HttpException(400, 'Access denied!');
         }
-        
+
         $poll = new Poll(['scenario' => Poll::SCENARIO_CREATE]);
         $poll->load(Yii::$app->request->post());
         return WallCreateForm::create($poll, $this->contentContainer);
@@ -108,12 +107,12 @@ public function actionEdit()
 
     public function actionOpen()
     {
-        return $this->asJson($this->setClosed(Yii::$app->request->get('id'), false)); 
+        return $this->asJson($this->setClosed(Yii::$app->request->get('id'), false));
     }
 
     public function actionClose()
     {
-        return  $this->asJson($this->setClosed(Yii::$app->request->get('id'), true)); 
+        return  $this->asJson($this->setClosed(Yii::$app->request->get('id'), true));
     }
 
     public function setClosed($id, $closed)
@@ -131,7 +130,7 @@ public function setClosed($id, $closed)
         $model->save();
         // Refresh updated_at
         $model->content->refresh();
-        
+
         return Stream::getContentResultEntry($model->content);
     }
 

controllers/rest/PollsController.php

@@ -249,4 +249,4 @@ public function actionVotes(int $id): array
             ->andWhere(['poll_id' => $id])
             ->column();
     }
-}
+}

docs/CHANGELOG.md

@@ -1,6 +1,10 @@
 Changelog
 =========
 
+1.3.6 (Unreleased)
+----------------------
+- Enh #137: Use PHP CS Fixer
+
 1.3.5 (June 18, 2024)
 ----------------------
 - Fix #129: Fix memory usage on integrity check

helpers/RestDefinitions.php

@@ -64,4 +64,4 @@ public static function getAnswer(PollAnswer $pollAnswer): array
             'updated_by' => UserDefinitions::getUserShort($pollAnswer->updatedBy),
         ];
     }
-}
+}

migrations/m131023_165921_initial.php

@@ -4,7 +4,6 @@
 
 class m131023_165921_initial extends Migration
 {
-
     public function up()
     {
 
@@ -16,7 +15,7 @@ public function up()
             'created_by' => 'int(11) NOT NULL',
             'updated_at' => 'datetime NOT NULL',
             'updated_by' => 'int(11) NOT NULL',
-                ), '');
+        ), '');
 
         $this->createTable('poll_answer', array(
             'id' => 'pk',
@@ -26,7 +25,7 @@ public function up()
             'created_by' => 'int(11) NOT NULL',
             'updated_at' => 'datetime NOT NULL',
             'updated_by' => 'int(11) NOT NULL',
-                ), '');
+        ), '');
 
         $this->createTable('poll_answer_user', array(
             'id' => 'pk',
@@ -36,7 +35,7 @@ public function up()
             'created_by' => 'int(11) NOT NULL',
             'updated_at' => 'datetime NOT NULL',
             'updated_by' => 'int(11) NOT NULL',
-                ), '');
+        ), '');
     }
 
     public function down()

migrations/m131030_122743_longer_questions.php

@@ -4,7 +4,6 @@
 
 class m131030_122743_longer_questions extends Migration
 {
-
     public function up()
     {
         $this->alterColumn('poll', 'question', 'TEXT NOT NULL');

migrations/m141002_093710_fix_default.php

@@ -4,7 +4,6 @@
 
 class m141002_093710_fix_default extends Migration
 {
-
     public function up()
     {
         $this->alterColumn('poll', 'allow_multiple', "smallint(6) NOT NULL DEFAULT 0");

migrations/m150709_151858_namespace.php

@@ -5,7 +5,6 @@
 
 class m150709_151858_namespace extends Migration
 {
-
     public function up()
     {
         $this->renameClass('Poll', humhub\modules\polls\models\Poll::className());

migrations/m160218_132531_close_and_anonymous_poll.php

@@ -7,9 +7,9 @@ class m160218_132531_close_and_anonymous_poll extends Migration
 {
     public function up()
     {
-        $this->addColumn('poll', 'is_random', Schema::TYPE_BOOLEAN. ' DEFAULT 0');
-        $this->addColumn('poll', 'closed', Schema::TYPE_BOOLEAN. ' DEFAULT 0');
-        $this->addColumn('poll', 'anonymous', Schema::TYPE_BOOLEAN. ' DEFAULT 0');
+        $this->addColumn('poll', 'is_random', Schema::TYPE_BOOLEAN . ' DEFAULT 0');
+        $this->addColumn('poll', 'closed', Schema::TYPE_BOOLEAN . ' DEFAULT 0');
+        $this->addColumn('poll', 'anonymous', Schema::TYPE_BOOLEAN . ' DEFAULT 0');
     }
 
     public function down()

migrations/m180327_125017_show_result_after_close.php

@@ -13,7 +13,7 @@ class m180327_125017_show_result_after_close extends Migration
 {
     public function safeUp()
     {
-        $this->addColumn('poll', 'show_result_after_close', Schema::TYPE_BOOLEAN. ' DEFAULT 0');
+        $this->addColumn('poll', 'show_result_after_close', Schema::TYPE_BOOLEAN . ' DEFAULT 0');
     }
 
     public function safeDown()

migrations/m201020_125649_question_description.php

@@ -23,6 +23,6 @@ public function safeUp()
     public function safeDown()
     {
         $this->dropColumn('poll', 'question');
-        $this->renameColumn('poll', 'description','question');
+        $this->renameColumn('poll', 'description', 'question');
     }
 }

migrations/uninstall.php

@@ -4,7 +4,6 @@
 
 class uninstall extends Migration
 {
-
     public function up()
     {