Apply suggestions from code review

glpi-project · Apr 26, 2024 · 1d23d5d · 1d23d5d
1 parent 79d4121
commit 1d23d5d
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 12 deletions.
diff --git a/ajax/form/answer.php b/ajax/form/answer.php
@@ -36,7 +36,6 @@
 use Glpi\Form\AnswersHandler\AnswersHandler;
 use Glpi\Form\EndUserInputNameProvider;
 use Glpi\Form\Form;
-use Glpi\Form\Question;
 use Glpi\Http\Response;
 
 include('../../inc/includes.php');
@@ -60,7 +59,7 @@
 }
 
 // Validate the 'answers' parameter by filtering and reindexing the $_POST array.
-$answers = EndUserInputNameProvider::getAnswers();
+$answers = (new EndUserInputNameProvider())->getAnswers($_POST);
 if (empty($answers)) {
     Response::sendError(400, __('Invalid answers'));
 }

diff --git a/src/Form/EndUserInputNameProvider.php b/src/Form/EndUserInputNameProvider.php
@@ -36,7 +36,7 @@
 namespace Glpi\Form;
 
 /**
- * Helpdesk form
+ * Utility class to provide the end user input name
  */
 final class EndUserInputNameProvider
 {
@@ -49,20 +49,21 @@ final class EndUserInputNameProvider
      * @param Question $question
      * @return string
      */
-    public static function getEndUserInputName(Question $question): string
+    public function getEndUserInputName(Question $question): string
     {
-        return sprintf(static::END_USER_INPUT_NAME, $question->getID());
+        return sprintf(self::END_USER_INPUT_NAME, $question->getID());
     }
 
     /**
      * Get the answers submitted by the end user
      * The answers are indexed by question ID
      *
+     * @param array $inputs The inputs submitted by the end user
      * @return array
      */
-    public static function getAnswers(): array
+    public function getAnswers(array $inputs): array
     {
-        $filteredAnswers = self::filterAnswers($_POST);
+        $filteredAnswers = self::filterAnswers($inputs);
         $reindexedAnswers = self::reindexAnswers($filteredAnswers);
 
         return $reindexedAnswers;
@@ -75,7 +76,7 @@ public static function getAnswers(): array
      * @param array $answers
      * @return array
      */
-    private static function filterAnswers(array $answers): array
+    private function filterAnswers(array $answers): array
     {
         return array_filter(
             $answers,
@@ -93,7 +94,7 @@ function ($key) {
      * @param array $answers
      * @return array
      */
-    private static function reindexAnswers(array $answers): array
+    private function reindexAnswers(array $answers): array
     {
         return array_reduce(
             array_keys($answers),

diff --git a/src/Form/Question.php b/src/Form/Question.php
@@ -123,7 +123,7 @@ protected function getForm(): Form
 
     public function getEndUserInputName(): string
     {
-        return EndUserInputNameProvider::getEndUserInputName($this);
+        return (new EndUserInputNameProvider())->getEndUserInputName($this);
     }
 
     public function prepareInputForAdd($input)

diff --git a/tests/functional/Glpi/Form/EndUserInputNameProvider.php b/tests/functional/Glpi/Form/EndUserInputNameProvider.php
@@ -71,14 +71,14 @@ public function testGetAnswers()
         );
 
         // Generate the answers
-        $_POST = [
+        $inputs = [
             $form->getQuestions()[array_keys($form->getQuestions())[0]]->getEndUserInputName() => 'John Doe',
             $form->getQuestions()[array_keys($form->getQuestions())[1]]->getEndUserInputName() => 'john.doe@mail.mail',
             'invalid_input' => 'invalid_value',
         ];
 
         // Check that the answers are correctly indexed by question ID
-        $this->array(\Glpi\Form\EndUserInputNameProvider::getAnswers())
+        $this->array((new \Glpi\Form\EndUserInputNameProvider())->getAnswers($inputs))
             ->hasSize(2)
             ->isEqualTo([
                 $form->getQuestions()[array_keys($form->getQuestions())[0]]->getID() => 'John Doe',