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 |