| Conditions | 1 |
| Paths | 1 |
| Total Lines | 103 |
| Code Lines | 82 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php |
||
| 155 | public function buildForm(FormBuilderInterface $builder): void |
||
| 156 | { |
||
| 157 | $toolChoices = $this->getFilteredToolChoices(); |
||
| 158 | |||
| 159 | $builder |
||
| 160 | ->add('active_tools_on_create', ChoiceType::class, [ |
||
| 161 | 'choices' => $toolChoices, |
||
| 162 | 'multiple' => true, |
||
| 163 | 'expanded' => true, |
||
| 164 | ]) |
||
| 165 | ->add('display_coursecode_in_courselist', YesNoType::class) |
||
| 166 | ->add('display_teacher_in_courselist', YesNoType::class) |
||
| 167 | ->add('student_view_enabled', YesNoType::class) |
||
| 168 | ->add('show_navigation_menu', ChoiceType::class, [ |
||
| 169 | 'choices' => [ |
||
| 170 | 'No' => 'false', |
||
| 171 | 'Icons only' => 'icons', |
||
| 172 | 'Text only' => 'text', |
||
| 173 | 'Icons text' => 'iconstext', |
||
| 174 | ], |
||
| 175 | ]) |
||
| 176 | ->add('enable_tool_introduction', YesNoType::class) |
||
| 177 | ->add('breadcrumbs_course_homepage', ChoiceType::class, [ |
||
| 178 | 'choices' => [ |
||
| 179 | 'Course homepage' => 'course_home', |
||
| 180 | 'Course code' => 'course_code', |
||
| 181 | 'Course title' => 'course_title', |
||
| 182 | 'Session name and course title' => 'session_name_and_course_title', |
||
| 183 | ], |
||
| 184 | ]) |
||
| 185 | ->add('example_material_course_creation', YesNoType::class) |
||
| 186 | ->add('allow_course_theme', YesNoType::class) |
||
| 187 | ->add('send_email_to_admin_when_create_course', YesNoType::class) |
||
| 188 | ->add('course_validation', YesNoType::class) |
||
| 189 | ->add('course_validation_terms_and_conditions_url', UrlType::class) |
||
| 190 | ->add('course_hide_tools', ChoiceType::class, [ |
||
| 191 | 'choices' => $toolChoices, |
||
| 192 | 'multiple' => true, |
||
| 193 | 'expanded' => true, |
||
| 194 | ]) |
||
| 195 | ->add('scorm_cumulative_session_time', YesNoType::class) |
||
| 196 | ->add('courses_default_creation_visibility', ChoiceType::class, [ |
||
| 197 | 'choices' => [ |
||
| 198 | 'Public' => '3', |
||
| 199 | 'Open' => '2', |
||
| 200 | 'Private' => '1', |
||
| 201 | 'Closed' => '0', |
||
| 202 | ], |
||
| 203 | ]) |
||
| 204 | ->add('course_creation_use_template', EntityType::class, [ |
||
| 205 | 'class' => Course::class, |
||
| 206 | 'placeholder' => 'Choose ...', |
||
| 207 | 'empty_data' => null, |
||
| 208 | 'choice_label' => 'title', |
||
| 209 | 'choice_value' => 'id', |
||
| 210 | 'required' => false, |
||
| 211 | ]) |
||
| 212 | ->add('course_images_in_courses_list', YesNoType::class) |
||
| 213 | ->add('show_toolshortcuts', YesNoType::class) |
||
| 214 | ->add('course_creation_splash_screen', YesNoType::class) |
||
| 215 | ->add('block_registered_users_access_to_open_course_contents', YesNoType::class) |
||
| 216 | ->add('view_grid_courses', YesNoType::class) |
||
| 217 | ->add('my_courses_show_courses_in_user_language_only', YesNoType::class) |
||
| 218 | ->add('allow_public_course_with_no_terms_conditions', YesNoType::class) |
||
| 219 | ->add('allow_base_course_category', YesNoType::class) |
||
| 220 | ->add('hide_course_sidebar', YesNoType::class) |
||
| 221 | ->add('multiple_access_url_show_shared_course_marker', YesNoType::class) |
||
| 222 | ->add('course_category_code_to_use_as_model', TextType::class) |
||
| 223 | ->add('enable_unsubscribe_button_on_my_course_page', YesNoType::class) |
||
| 224 | ->add('course_creation_donate_message_show', YesNoType::class) |
||
| 225 | ->add('course_creation_donate_link', TextType::class) |
||
| 226 | ->add('hide_course_rating', YesNoType::class) |
||
| 227 | ->add('course_log_hide_columns', TextareaType::class, [ |
||
| 228 | 'attr' => ['rows' => 5, 'style' => 'font-family: monospace;'], |
||
| 229 | ]) |
||
| 230 | ->add('course_student_info', TextareaType::class, [ |
||
| 231 | 'attr' => ['rows' => 5, 'style' => 'font-family: monospace;'], |
||
| 232 | ]) |
||
| 233 | ->add('resource_sequence_show_dependency_in_course_intro', YesNoType::class) |
||
| 234 | ->add('course_sequence_valid_only_in_same_session', YesNoType::class) |
||
| 235 | ->add('course_creation_form_set_course_category_mandatory', YesNoType::class) |
||
| 236 | ->add('course_creation_form_hide_course_code', YesNoType::class) |
||
| 237 | ->add('course_about_teacher_name_hide', YesNoType::class) |
||
| 238 | ->add('course_log_default_extra_fields', TextareaType::class, [ |
||
| 239 | 'attr' => ['rows' => 5, 'style' => 'font-family: monospace;'], |
||
| 240 | ]) |
||
| 241 | ->add('course_creation_by_teacher_extra_fields_to_show', TextareaType::class, [ |
||
| 242 | 'attr' => ['rows' => 3, 'style' => 'font-family: monospace;'], |
||
| 243 | ]) |
||
| 244 | ->add('course_creation_form_set_extra_fields_mandatory', TextareaType::class, [ |
||
| 245 | 'attr' => ['rows' => 3, 'style' => 'font-family: monospace;'], |
||
| 246 | ]) |
||
| 247 | ->add('course_configuration_tool_extra_fields_to_show_and_edit', TextareaType::class, [ |
||
| 248 | 'attr' => ['rows' => 3, 'style' => 'font-family: monospace;'], |
||
| 249 | ]) |
||
| 250 | ->add('course_creation_user_course_extra_field_relation_to_prefill', TextareaType::class, [ |
||
| 251 | 'attr' => ['rows' => 5, 'style' => 'font-family: monospace;'], |
||
| 252 | ]) |
||
| 253 | ->add('show_course_duration', YesNoType::class) |
||
| 254 | ->add('profiling_filter_adding_users', YesNoType::class) |
||
| 255 | ; |
||
| 256 | |||
| 257 | $this->updateFormFieldsFromSettingsInfo($builder); |
||
| 258 | } |
||
| 260 |