Ims2Question::createAnswersForm() - Code Metrics - Inspection of "Merge pull request #7336 from christianbeeznest/GH..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ca27a7...8ba864 )

unknown

created 2026-01-10 03:31 UTC

Ims2Question::createAnswersForm() A

↳ Parent: Ims2Question

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	1
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php

/* For licensing terms, see /license.txt */

/**
 * @author Claro Team <[email protected]>
 * @author Yannick Warnier <[email protected]> -
 * updated ImsAnswerHotspot to match QTI norms
 */
class Ims2Question extends Question
{
    /**
     * Create the proper Answer handler.
     * Messages/strings in code must be English.
     */
    public function setAnswer()
    {
        $questionId = 0;

        if (method_exists($this, 'getIid')) {
            $questionId = (int) $this->getIid();
        } elseif (property_exists($this, 'iid')) {
            $questionId = (int) $this->iid;
        } elseif (property_exists($this, 'id')) {
            $questionId = (int) $this->id;
        }

        // Avoid null IDs: use 0 as safe fallback (some flows set the ID later).
        if ($questionId < 0) {
            $questionId = 0;
        }

        switch ($this->type) {
            case TF:
            case MCUA:
            case MCMA:
                $answer = new ImsAnswerMultipleChoice($questionId);
                break;

            case FIB:
                $answer = new ImsAnswerFillInBlanks($questionId);
                break;

            case MATCHING:
            case MATCHING_DRAGGABLE:
                $answer = new ImsAnswerMatching($questionId);
                break;

            case FREE_ANSWER:
                $answer = new ImsAnswerFree($questionId);
                break;

            case HOT_SPOT:
                $answer = new ImsAnswerHotspot($questionId);
                break;

            default:
                $answer = null;
        }

        // If the parent Question class expects an internal property, keep it.
        if (property_exists($this, 'answer')) {
            $this->answer = $answer;
        }

        return $answer;
    }

    public function createAnswersForm($form)
    {
        return true;
    }

    public function processAnswersCreation($form, $exercise)
    {
        return true;
    }
}


1			<?php
2
3			/* For licensing terms, see /license.txt */
4
5			/**
6			* @author Claro Team <[email protected]>
7			* @author Yannick Warnier <[email protected]> -
8			* updated ImsAnswerHotspot to match QTI norms
9			*/
10			class Ims2Question extends Question
11			{
12			/**
13			* Create the proper Answer handler.
14			* Messages/strings in code must be English.
15			*/
16			public function setAnswer()
17			{
18			$questionId = 0;
19
20			if (method_exists($this, 'getIid')) {
21			$questionId = (int) $this->getIid();
22			} elseif (property_exists($this, 'iid')) {
23			$questionId = (int) $this->iid;
24			} elseif (property_exists($this, 'id')) {
25			$questionId = (int) $this->id;
26			}
27
28			// Avoid null IDs: use 0 as safe fallback (some flows set the ID later).
29			if ($questionId < 0) {
30			$questionId = 0;
31			}
32
33			switch ($this->type) {
34			case TF:
35			case MCUA:
36			case MCMA:
37			$answer = new ImsAnswerMultipleChoice($questionId);
38			break;
39
40			case FIB:
41			$answer = new ImsAnswerFillInBlanks($questionId);
42			break;
43
44			case MATCHING:
45			case MATCHING_DRAGGABLE:
46			$answer = new ImsAnswerMatching($questionId);
47			break;
48
49			case FREE_ANSWER:
50			$answer = new ImsAnswerFree($questionId);
51			break;
52
53			case HOT_SPOT:
54			$answer = new ImsAnswerHotspot($questionId);
55			break;
56
57			default:
58			$answer = null;
59			}
60
61			// If the parent Question class expects an internal property, keep it.
62			if (property_exists($this, 'answer')) {
63			$this->answer = $answer;
64			}
65
66			return $answer;
67			}
68
69			public function createAnswersForm($form)
70			{
71			return true;
72			}
73
74			public function processAnswersCreation($form, $exercise)
75			{
76			return true;
77			}
78			}
79

chamilo / chamilo-lms

Push — master ( ca27a7...8ba864 )

Ims2Question::createAnswersForm() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like