LearningPathSettingsSchema::buildForm() - Code Metrics - Inspection of "Admin: Improve settings categorization - refs #681..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#6824)

unknown

created 2025-09-27 07:29 UTC

LearningPathSettingsSchema::buildForm() B

↳ Parent: LearningPathSettingsSchema

Complexity

Conditions	1
Paths	1

Size

Total Lines	61
Code Lines	55

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	55
nc	1
nop	1
dl	0
loc	61
rs	8.9818
c	0
b	0
f	0

How to fix Long Method

<?php

declare(strict_types=1);

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

namespace Chamilo\CoreBundle\Settings;

use Chamilo\CoreBundle\Form\Type\YesNoType;
use Sylius\Bundle\SettingsBundle\Schema\AbstractSettingsBuilder;
use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
use Symfony\Component\Form\Extension\Core\Type\TextareaType;
use Symfony\Component\Form\Extension\Core\Type\TextType;
use Symfony\Component\Form\FormBuilderInterface;

class LearningPathSettingsSchema extends AbstractSettingsSchema
{
    public function buildSettings(AbstractSettingsBuilder $builder): void
    {
        $builder->setDefaults([
            'show_invisible_exercise_in_lp_toc' => 'false',
            'add_all_files_in_lp_export' => 'false',
            'show_prerequisite_as_blocked' => 'false',
            'hide_lp_time' => 'false',
            'lp_view_accordion' => 'false',
            'disable_js_in_lp_view' => 'false',
            'allow_teachers_to_access_blocked_lp_by_prerequisite' => 'false',
            'allow_lp_chamilo_export' => 'false',
            'hide_accessibility_label_on_lp_item' => 'true',
            'lp_minimum_time' => 'false',
            'validate_lp_prerequisite_from_other_session' => 'false',
            'show_hidden_exercise_added_to_lp' => 'true',
            'lp_menu_location' => 'left',
            'lp_score_as_progress_enable' => 'false',
            'lp_prevents_beforeunload' => 'false',
            'disable_my_lps_page' => 'false',
            'scorm_api_username_as_student_id' => 'false',
            'scorm_api_extrafield_to_use_as_student_id' => '',
            'allow_import_scorm_package_in_course_builder' => 'false',
            'allow_htaccess_import_from_scorm' => 'false',
            'allow_session_lp_category' => 'false',
            'ticket_lp_quiz_info_add' => 'false',
            'lp_subscription_settings' => '',
            'lp_view_settings' => '',
            'download_files_after_all_lp_finished' => '',
            'allow_lp_subscription_to_usergroups' => 'false',
            'lp_fixed_encoding' => 'false',
            'lp_prerequisite_use_last_attempt_only' => 'false',
            'show_invisible_exercise_in_lp_list' => 'false',
            'force_edit_exercise_in_lp' => 'false',
            'student_follow_page_add_LP_subscription_info' => 'false',
            'lp_show_max_progress_instead_of_average' => 'false',
            'lp_show_max_progress_or_average_enable_course_level_redefinition' => 'false',
            'lp_allow_export_to_students' => 'false',
            'show_invisible_lp_in_course_home' => 'false',
            'lp_start_and_end_date_visible_in_student_view' => 'false',
            'scorm_lms_update_sco_status_all_time' => 'false',
            'scorm_upload_from_cache' => 'false',
            'lp_prerequisite_on_quiz_unblock_if_max_attempt_reached' => 'false',
            'student_follow_page_hide_lp_tests_average' => 'false',
            'student_follow_page_add_lp_acquisition_info' => 'false',
            'student_follow_page_add_lp_invisible_checkbox' => 'false',
            'student_follow_page_include_not_subscribed_lp_students' => 'false',
            'lp_enable_flow' => 'false',
            'lp_item_prerequisite_dates' => 'false',
            'allow_lp_return_link' => 'true',
            'hide_scorm_export_link' => 'false',
            'hide_scorm_copy_link' => 'false',
            'hide_scorm_pdf_link' => 'true',
            'lp_show_reduced_report' => 'false',
        ]);

        $allowedTypes = [];

        $this->setMultipleAllowedTypes($allowedTypes, $builder);
    }

    public function buildForm(FormBuilderInterface $builder): void
    {
        $builder
            ->add('show_invisible_exercise_in_lp_toc', YesNoType::class)
            ->add('add_all_files_in_lp_export', YesNoType::class)
            ->add('show_prerequisite_as_blocked', YesNoType::class)
            ->add('hide_lp_time', YesNoType::class)
            ->add('lp_view_accordion', YesNoType::class)
            ->add('disable_js_in_lp_view', YesNoType::class)
            ->add('allow_teachers_to_access_blocked_lp_by_prerequisite', YesNoType::class)
            ->add('allow_lp_chamilo_export', YesNoType::class)
            ->add('hide_accessibility_label_on_lp_item', YesNoType::class)
            ->add('lp_minimum_time', YesNoType::class)
            ->add('validate_lp_prerequisite_from_other_session', YesNoType::class)
            ->add('show_hidden_exercise_added_to_lp', YesNoType::class)
            ->add('lp_menu_location', ChoiceType::class, [
                'choices' => [
                    'Left' => 'left',
                    'Right' => 'right',
                ],
            ])
            ->add('lp_score_as_progress_enable', YesNoType::class)
            ->add('lp_prevents_beforeunload', YesNoType::class)
            ->add('disable_my_lps_page', YesNoType::class)
            ->add('scorm_api_username_as_student_id', YesNoType::class)
            ->add('scorm_api_extrafield_to_use_as_student_id', TextType::class)
            ->add('allow_import_scorm_package_in_course_builder', YesNoType::class)
            ->add('allow_htaccess_import_from_scorm', YesNoType::class)
            ->add('allow_session_lp_category', YesNoType::class)
            ->add('ticket_lp_quiz_info_add', YesNoType::class)
            ->add('lp_subscription_settings', TextareaType::class)
            ->add('lp_view_settings', TextareaType::class)
            ->add('download_files_after_all_lp_finished', TextareaType::class)
            ->add('allow_lp_subscription_to_usergroups', YesNoType::class)
            ->add('lp_fixed_encoding', YesNoType::class)
            ->add('lp_prerequisite_use_last_attempt_only', YesNoType::class)
            ->add('show_invisible_exercise_in_lp_list', YesNoType::class)
            ->add('force_edit_exercise_in_lp', YesNoType::class)
            ->add('student_follow_page_add_LP_subscription_info', YesNoType::class)
            ->add('lp_show_max_progress_instead_of_average', YesNoType::class)
            ->add('lp_show_max_progress_or_average_enable_course_level_redefinition', YesNoType::class)
            ->add('lp_allow_export_to_students', YesNoType::class)
            ->add('show_invisible_lp_in_course_home', YesNoType::class)
            ->add('lp_start_and_end_date_visible_in_student_view', YesNoType::class)
            ->add('scorm_lms_update_sco_status_all_time', YesNoType::class)
            ->add('scorm_upload_from_cache', YesNoType::class)
            ->add('lp_prerequisite_on_quiz_unblock_if_max_attempt_reached', YesNoType::class)
            ->add('student_follow_page_hide_lp_tests_average', YesNoType::class)
            ->add('student_follow_page_add_lp_acquisition_info', YesNoType::class)
            ->add('student_follow_page_add_lp_invisible_checkbox', YesNoType::class)
            ->add('student_follow_page_include_not_subscribed_lp_students', YesNoType::class)
            ->add('lp_enable_flow', YesNoType::class)
            ->add('lp_item_prerequisite_dates', YesNoType::class)
            ->add('allow_lp_return_link', YesNoType::class)
            ->add('hide_scorm_export_link', YesNoType::class)
            ->add('hide_scorm_copy_link', YesNoType::class)
            ->add('hide_scorm_pdf_link', YesNoType::class)
            ->add('lp_show_reduced_report', YesNoType::class)
        ;

        $this->updateFormFieldsFromSettingsInfo($builder);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/* For licensing terms, see /license.txt */
6
7			namespace Chamilo\CoreBundle\Settings;
8
9			use Chamilo\CoreBundle\Form\Type\YesNoType;
10			use Sylius\Bundle\SettingsBundle\Schema\AbstractSettingsBuilder;
11			use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
12			use Symfony\Component\Form\Extension\Core\Type\TextareaType;
13			use Symfony\Component\Form\Extension\Core\Type\TextType;
14			use Symfony\Component\Form\FormBuilderInterface;
15
16			class LearningPathSettingsSchema extends AbstractSettingsSchema
17			{
18			public function buildSettings(AbstractSettingsBuilder $builder): void
19			{
20			$builder->setDefaults([
21			'show_invisible_exercise_in_lp_toc' => 'false',
22			'add_all_files_in_lp_export' => 'false',
23			'show_prerequisite_as_blocked' => 'false',
24			'hide_lp_time' => 'false',
25			'lp_view_accordion' => 'false',
26			'disable_js_in_lp_view' => 'false',
27			'allow_teachers_to_access_blocked_lp_by_prerequisite' => 'false',
28			'allow_lp_chamilo_export' => 'false',
29			'hide_accessibility_label_on_lp_item' => 'true',
30			'lp_minimum_time' => 'false',
31			'validate_lp_prerequisite_from_other_session' => 'false',
32			'show_hidden_exercise_added_to_lp' => 'true',
33			'lp_menu_location' => 'left',
34			'lp_score_as_progress_enable' => 'false',
35			'lp_prevents_beforeunload' => 'false',
36			'disable_my_lps_page' => 'false',
37			'scorm_api_username_as_student_id' => 'false',
38			'scorm_api_extrafield_to_use_as_student_id' => '',
39			'allow_import_scorm_package_in_course_builder' => 'false',
40			'allow_htaccess_import_from_scorm' => 'false',
41			'allow_session_lp_category' => 'false',
42			'ticket_lp_quiz_info_add' => 'false',
43			'lp_subscription_settings' => '',
44			'lp_view_settings' => '',
45			'download_files_after_all_lp_finished' => '',
46			'allow_lp_subscription_to_usergroups' => 'false',
47			'lp_fixed_encoding' => 'false',
48			'lp_prerequisite_use_last_attempt_only' => 'false',
49			'show_invisible_exercise_in_lp_list' => 'false',
50			'force_edit_exercise_in_lp' => 'false',
51			'student_follow_page_add_LP_subscription_info' => 'false',
52			'lp_show_max_progress_instead_of_average' => 'false',
53			'lp_show_max_progress_or_average_enable_course_level_redefinition' => 'false',
54			'lp_allow_export_to_students' => 'false',
55			'show_invisible_lp_in_course_home' => 'false',
56			'lp_start_and_end_date_visible_in_student_view' => 'false',
57			'scorm_lms_update_sco_status_all_time' => 'false',
58			'scorm_upload_from_cache' => 'false',
59			'lp_prerequisite_on_quiz_unblock_if_max_attempt_reached' => 'false',
60			'student_follow_page_hide_lp_tests_average' => 'false',
61			'student_follow_page_add_lp_acquisition_info' => 'false',
62			'student_follow_page_add_lp_invisible_checkbox' => 'false',
63			'student_follow_page_include_not_subscribed_lp_students' => 'false',
64			'lp_enable_flow' => 'false',
65			'lp_item_prerequisite_dates' => 'false',
66			'allow_lp_return_link' => 'true',
67			'hide_scorm_export_link' => 'false',
68			'hide_scorm_copy_link' => 'false',
69			'hide_scorm_pdf_link' => 'true',
70			'lp_show_reduced_report' => 'false',
71			]);
72
73			$allowedTypes = [];
74
75			$this->setMultipleAllowedTypes($allowedTypes, $builder);
76			}
77
78			public function buildForm(FormBuilderInterface $builder): void
79			{
80			$builder
81			->add('show_invisible_exercise_in_lp_toc', YesNoType::class)
82			->add('add_all_files_in_lp_export', YesNoType::class)
83			->add('show_prerequisite_as_blocked', YesNoType::class)
84			->add('hide_lp_time', YesNoType::class)
85			->add('lp_view_accordion', YesNoType::class)
86			->add('disable_js_in_lp_view', YesNoType::class)
87			->add('allow_teachers_to_access_blocked_lp_by_prerequisite', YesNoType::class)
88			->add('allow_lp_chamilo_export', YesNoType::class)
89			->add('hide_accessibility_label_on_lp_item', YesNoType::class)
90			->add('lp_minimum_time', YesNoType::class)
91			->add('validate_lp_prerequisite_from_other_session', YesNoType::class)
92			->add('show_hidden_exercise_added_to_lp', YesNoType::class)
93			->add('lp_menu_location', ChoiceType::class, [
94			'choices' => [
95			'Left' => 'left',
96			'Right' => 'right',
97			],
98			])
99			->add('lp_score_as_progress_enable', YesNoType::class)
100			->add('lp_prevents_beforeunload', YesNoType::class)
101			->add('disable_my_lps_page', YesNoType::class)
102			->add('scorm_api_username_as_student_id', YesNoType::class)
103			->add('scorm_api_extrafield_to_use_as_student_id', TextType::class)
104			->add('allow_import_scorm_package_in_course_builder', YesNoType::class)
105			->add('allow_htaccess_import_from_scorm', YesNoType::class)
106			->add('allow_session_lp_category', YesNoType::class)
107			->add('ticket_lp_quiz_info_add', YesNoType::class)
108			->add('lp_subscription_settings', TextareaType::class)
109			->add('lp_view_settings', TextareaType::class)
110			->add('download_files_after_all_lp_finished', TextareaType::class)
111			->add('allow_lp_subscription_to_usergroups', YesNoType::class)
112			->add('lp_fixed_encoding', YesNoType::class)
113			->add('lp_prerequisite_use_last_attempt_only', YesNoType::class)
114			->add('show_invisible_exercise_in_lp_list', YesNoType::class)
115			->add('force_edit_exercise_in_lp', YesNoType::class)
116			->add('student_follow_page_add_LP_subscription_info', YesNoType::class)
117			->add('lp_show_max_progress_instead_of_average', YesNoType::class)
118			->add('lp_show_max_progress_or_average_enable_course_level_redefinition', YesNoType::class)
119			->add('lp_allow_export_to_students', YesNoType::class)
120			->add('show_invisible_lp_in_course_home', YesNoType::class)
121			->add('lp_start_and_end_date_visible_in_student_view', YesNoType::class)
122			->add('scorm_lms_update_sco_status_all_time', YesNoType::class)
123			->add('scorm_upload_from_cache', YesNoType::class)
124			->add('lp_prerequisite_on_quiz_unblock_if_max_attempt_reached', YesNoType::class)
125			->add('student_follow_page_hide_lp_tests_average', YesNoType::class)
126			->add('student_follow_page_add_lp_acquisition_info', YesNoType::class)
127			->add('student_follow_page_add_lp_invisible_checkbox', YesNoType::class)
128			->add('student_follow_page_include_not_subscribed_lp_students', YesNoType::class)
129			->add('lp_enable_flow', YesNoType::class)
130			->add('lp_item_prerequisite_dates', YesNoType::class)
131			->add('allow_lp_return_link', YesNoType::class)
132			->add('hide_scorm_export_link', YesNoType::class)
133			->add('hide_scorm_copy_link', YesNoType::class)
134			->add('hide_scorm_pdf_link', YesNoType::class)
135			->add('lp_show_reduced_report', YesNoType::class)
136			;
137
138			$this->updateFormFieldsFromSettingsInfo($builder);
139			}
140			}
141

chamilo / chamilo-lms

Pull Request — master (#6824)

LearningPathSettingsSchema::buildForm() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like