Fix styling

Author: sten

src/Commands/SurveyheroResponseImportCommand.php

@@ -34,7 +34,7 @@ public function handle(): int
             $this->deleteResponses();
         }
 
-        $importInfo = new ResponseImportInfo();
+        $importInfo = new ResponseImportInfo;
 
         $surveyId = trim($this->option('survey'));
 

src/Services/AbstractSurveyheroAPIService.php

@@ -10,7 +10,7 @@ class AbstractSurveyheroAPIService
 
     public function __construct()
     {
-        $this->client = new SurveyheroClient();
+        $this->client = new SurveyheroClient;
     }
 
     public function getApiClient(): SurveyheroClient

src/Services/Factories/QuestionAndAnswerCreator/AbstractQuestionAndAnswerCreator.php

@@ -27,14 +27,14 @@ public function updateOrCreateQuestion(SurveyContract $survey, string $lang, str
                     $lang => $label ?? '',
                 ],
                 'surveyhero_element_id' => $questionId,
-                'field' => (new SurveyMappingService())->findQuestionField($survey, $questionId, $subquestionId),
+                'field' => (new SurveyMappingService)->findQuestionField($survey, $questionId, $subquestionId),
             ]);
     }
 
     protected function getChoiceMapping(string|int $choiceId, string|int $questionId, array $questionMapping): int|string|null
     {
         $answerMapping = $questionMapping['answer_mapping'];
-        $mappingService = new SurveyMappingService();
+        $mappingService = new SurveyMappingService;
 
         if ($questionMapping['question_id'] !== $questionId && isset($questionMapping['subquestion_mapping'])) {
             $subquestionMapping = $mappingService->getSubquestionMapping($questionId, $questionMapping);

src/Services/Factories/QuestionAndAnswerCreator/ChoiceListQuestionAndAnswerCreator.php

@@ -29,7 +29,7 @@ public function updateOrCreateQuestionAndAnswer(\stdClass $question, SurveyContr
                 ],
             ];
 
-            $questionMapping = (new SurveyMappingService())->getQuestionMappingForSurvey($survey, $question->element_id);
+            $questionMapping = (new SurveyMappingService)->getQuestionMappingForSurvey($survey, $question->element_id);
             $mappedChoice = $this->getChoiceMapping($choice->choice_id, $question->element_id, $questionMapping);
 
             $this->setChoiceAndConvertToDataType($mappedChoice, $questionMapping['mapped_data_type'], $responseData, $choice);

src/Services/Factories/QuestionAndAnswerCreator/ChoiceTableQuestionAndAnswerCreator.php

@@ -26,7 +26,7 @@ public function updateOrCreateQuestionAndAnswer(\stdClass $question, SurveyContr
                     ],
                 ];
 
-                $questionMapping = (new SurveyMappingService())->getQuestionMappingForSurvey($survey, $question->element_id);
+                $questionMapping = (new SurveyMappingService)->getQuestionMappingForSurvey($survey, $question->element_id);
                 $mappedChoice = $this->getChoiceMapping($choice->choice_id, $rowQuestion->row_id, $questionMapping);
 
                 $this->setChoiceAndConvertToDataType($mappedChoice, $questionMapping['mapped_data_type'], $responseData, $choice);

src/Services/Factories/QuestionAndAnswerCreator/RatingScaleQuestionAndAnswerCreator.php

@@ -25,7 +25,7 @@ public function updateOrCreateQuestionAndAnswer(\stdClass $question, SurveyContr
         $maxValue = $ratingScale->right->value;
         $stepSize = $ratingScale->step_size;
 
-        $questionMapping = (new SurveyMappingService())->getQuestionMappingForSurvey($survey, $question->element_id);
+        $questionMapping = (new SurveyMappingService)->getQuestionMappingForSurvey($survey, $question->element_id);
         $mappedDataType = $questionMapping['mapped_data_type'] ?? SurveyAnswerContract::CONVERTED_TYPE_INT;
 
         for ($i = $minValue; $i <= $maxValue; $i += $stepSize) {

src/Services/Factories/ResponseCreator/ChoiceTableResponseCreator.php

@@ -78,7 +78,7 @@ public function updateOrCreateQuestionResponse(\stdClass $surveyheroQuestionResp
          */
 
         $responseList = [];
-        $mappingService = new SurveyMappingService();
+        $mappingService = new SurveyMappingService;
 
         foreach ($surveyheroQuestionResponse->choice_table as $surveyheroChoiceQuestion) {
             $subquestionMapping = $mappingService->getSubquestionMapping($surveyheroChoiceQuestion->row_id, $questionMapping);

src/Services/SurveyMappingService.php

@@ -201,10 +201,10 @@ public function findQuestionField(SurveyContract $survey, string $questionId, ?s
     private function getQuestionMapper(string $surveyheroFieldType): ?QuestionMapper
     {
         return match ($surveyheroFieldType) {
-            InputQuestionMapper::TYPE => new InputQuestionMapper(),
-            RatingScaleQuestionMapper::TYPE => new RatingScaleQuestionMapper(),
-            ChoiceListQuestionMapper::TYPE => new ChoiceListQuestionMapper(),
-            ChoiceTableQuestionMapper::TYPE => new ChoiceTableQuestionMapper(),
+            InputQuestionMapper::TYPE => new InputQuestionMapper,
+            RatingScaleQuestionMapper::TYPE => new RatingScaleQuestionMapper,
+            ChoiceListQuestionMapper::TYPE => new ChoiceListQuestionMapper,
+            ChoiceTableQuestionMapper::TYPE => new ChoiceTableQuestionMapper,
             default => null,
         };
     }

src/Services/SurveyQuestionsAndAnswersImportService.php

@@ -50,10 +50,10 @@ public function importSurveyQuestionsAndAnswers(SurveyContract $survey): array
     private function getQuestionAndAnswerCreator(string $surveyheroFieldType): ?QuestionAndAnswerCreator
     {
         return match ($surveyheroFieldType) {
-            ChoiceListQuestionAndAnswerCreator::TYPE => new ChoiceListQuestionAndAnswerCreator(),
-            ChoiceTableQuestionAndAnswerCreator::TYPE => new ChoiceTableQuestionAndAnswerCreator(),
-            RatingScaleQuestionAndAnswerCreator::TYPE => new RatingScaleQuestionAndAnswerCreator(),
-            InputQuestionAndAnswerCreator::TYPE => new InputQuestionAndAnswerCreator(),
+            ChoiceListQuestionAndAnswerCreator::TYPE => new ChoiceListQuestionAndAnswerCreator,
+            ChoiceTableQuestionAndAnswerCreator::TYPE => new ChoiceTableQuestionAndAnswerCreator,
+            RatingScaleQuestionAndAnswerCreator::TYPE => new RatingScaleQuestionAndAnswerCreator,
+            InputQuestionAndAnswerCreator::TYPE => new InputQuestionAndAnswerCreator,
             default => null,
         };
     }

src/Services/SurveyResponseImportService.php

@@ -40,7 +40,7 @@ public function importSurveyResponses(SurveyContract $survey): ResponseImportInf
     {
         $surveyCollectorIds = $this->surveyMappingService->getSurveyCollectors($survey);
         $surveyQuestionMapping = $this->surveyMappingService->getSurveyQuestionMapping($survey);
-        $responseImportInfo = new ResponseImportInfo();
+        $responseImportInfo = new ResponseImportInfo;
 
         try {
             DB::beginTransaction();
@@ -78,7 +78,7 @@ public function importSurveyResponses(SurveyContract $survey): ResponseImportInf
      */
     public function importSurveyResponse($responseId, SurveyContract $survey, ?array $surveyQuestionMapping = null): ?ResponseImportInfo
     {
-        $importInfo = new ResponseImportInfo();
+        $importInfo = new ResponseImportInfo;
 
         if (! $surveyQuestionMapping) {
             $surveyQuestionMapping = $this->surveyMappingService->getSurveyQuestionMapping($survey);
@@ -187,10 +187,10 @@ public function deleteSurveyResponse($surveyId, $responseId)
     private function getQuestionResponseCreator(string $surveyheroFieldType): ?QuestionResponseCreator
     {
         return match ($surveyheroFieldType) {
-            TextResponseCreator::TYPE => new TextResponseCreator(),
-            NumberResponseCreator::TYPE => new NumberResponseCreator(),
-            ChoicesResponseCreator::TYPE => new ChoicesResponseCreator(),
-            ChoiceTableResponseCreator::TYPE => new ChoiceTableResponseCreator(),
+            TextResponseCreator::TYPE => new TextResponseCreator,
+            NumberResponseCreator::TYPE => new NumberResponseCreator,
+            ChoicesResponseCreator::TYPE => new ChoicesResponseCreator,
+            ChoiceTableResponseCreator::TYPE => new ChoiceTableResponseCreator,
             default => null,
         };
     }

src/SurveyheroServiceProvider.php

@@ -45,7 +45,7 @@ public function packageBooted()
         parent::packageBooted();
 
         $this->app->singleton(SurveyheroRegistrar::class, function ($app) {
-            return new SurveyheroRegistrar();
+            return new SurveyheroRegistrar;
         });
     }
 }

src/Traits/HasCollectors.php

@@ -11,7 +11,7 @@ trait HasCollectors
     public function getCollectors(): array
     {
         if (! $this->collectors) {
-            $surveyMappingService = new SurveyMappingService();
+            $surveyMappingService = new SurveyMappingService;
             $surveyMappingConfig = $surveyMappingService->getSurveyMappingFromConfig($this);
             $surveyCollectors = $this->collector_ids;
 

src/Traits/HasQuestionMapping.php

@@ -11,7 +11,7 @@ trait HasQuestionMapping
     public function getQuestionMapping(): array
     {
         if (! $this->mergedQuestionMapping) {
-            $surveyMappingService = new SurveyMappingService();
+            $surveyMappingService = new SurveyMappingService;
             $surveyMappingConfig = $surveyMappingService->getSurveyMappingFromConfig($this);
             $surveyMappingDatabase = $this->question_mapping;