| Conditions | 1 |
| Paths | 1 |
| Total Lines | 110 |
| Code Lines | 78 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 1 | ||
| Bugs | 0 | Features | 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 |
||
| 126 | public function buildForm(FormBuilderInterface $builder) |
||
| 127 | { |
||
| 128 | $tools = $this->getProcessedToolChain(); |
||
| 129 | |||
| 130 | $builder |
||
| 131 | ->add( |
||
| 132 | 'homepage_view', |
||
| 133 | ChoiceType::class, |
||
| 134 | [ |
||
| 135 | 'choices' => [ |
||
| 136 | //'HomepageView2column' => '2column', |
||
| 137 | //'HomepageView3column' => '3column', |
||
| 138 | //'HomepageViewVerticalActivity' => 'vertical_activity', |
||
| 139 | //'HomepageViewActivity' => 'activity', |
||
| 140 | 'HomepageViewActivityBig' => 'activity_big', |
||
| 141 | ], |
||
| 142 | ] |
||
| 143 | ) |
||
| 144 | ->add('show_tool_shortcuts', YesNoType::class) |
||
| 145 | ->add( |
||
| 146 | 'active_tools_on_create', |
||
| 147 | ChoiceType::class, |
||
| 148 | [ |
||
| 149 | 'choices' => $tools, |
||
| 150 | 'multiple' => true, |
||
| 151 | 'expanded' => true, |
||
| 152 | ] |
||
| 153 | ) |
||
| 154 | ->add('display_coursecode_in_courselist', YesNoType::class) |
||
| 155 | ->add('display_teacher_in_courselist', YesNoType::class) |
||
| 156 | ->add('student_view_enabled', YesNoType::class) |
||
| 157 | ->add('go_to_course_after_login', YesNoType::class) |
||
| 158 | ->add( |
||
| 159 | 'show_navigation_menu', |
||
| 160 | ChoiceType::class, |
||
| 161 | [ |
||
| 162 | 'choices' => [ |
||
| 163 | 'No' => 'false', |
||
| 164 | 'IconsOnly' => 'icons', |
||
| 165 | 'TextOnly' => 'text', |
||
| 166 | 'IconsText' => 'iconstext', |
||
| 167 | ], |
||
| 168 | ] |
||
| 169 | ) |
||
| 170 | ->add('enable_tool_introduction', YesNoType::class) |
||
| 171 | ->add( |
||
| 172 | 'breadcrumbs_course_homepage', |
||
| 173 | ChoiceType::class, |
||
| 174 | [ |
||
| 175 | 'choices' => [ |
||
| 176 | 'CourseHomepage' => 'course_home', |
||
| 177 | 'CourseCode' => 'course_code', |
||
| 178 | 'CourseTitle' => 'course_title', |
||
| 179 | 'SessionNameAndCourseTitle' => 'session_name_and_course_title', |
||
| 180 | ], |
||
| 181 | ] |
||
| 182 | ) |
||
| 183 | ->add('example_material_course_creation', YesNoType::class) |
||
| 184 | ->add('allow_course_theme', YesNoType::class) |
||
| 185 | ->add('allow_users_to_create_courses', YesNoType::class) |
||
| 186 | ->add('show_courses_descriptions_in_catalog', YesNoType::class) |
||
| 187 | ->add('send_email_to_admin_when_create_course', YesNoType::class) |
||
| 188 | ->add('allow_user_course_subscription_by_course_admin', YesNoType::class) |
||
| 189 | ->add('course_validation', YesNoType::class) |
||
| 190 | ->add('course_validation_terms_and_conditions_url', UrlType::class) |
||
| 191 | ->add( |
||
| 192 | 'course_hide_tools', |
||
| 193 | ChoiceType::class, |
||
| 194 | [ |
||
| 195 | 'choices' => $tools, |
||
| 196 | 'multiple' => true, |
||
| 197 | 'expanded' => true, |
||
| 198 | ] |
||
| 199 | ) |
||
| 200 | ->add('scorm_cumulative_session_time', YesNoType::class) |
||
| 201 | ->add( |
||
| 202 | 'courses_default_creation_visibility', |
||
| 203 | ChoiceType::class, |
||
| 204 | [ |
||
| 205 | 'choices' => [ |
||
| 206 | 'Public' => '3', |
||
| 207 | 'Open' => '2', |
||
| 208 | 'Private' => '1', |
||
| 209 | 'Closed' => '0', |
||
| 210 | ], |
||
| 211 | ] |
||
| 212 | ) |
||
| 213 | ->add('allow_public_certificates', YesNoType::class) |
||
| 214 | ->add('allow_lp_return_link', YesNoType::class) |
||
| 215 | ->add( |
||
| 216 | 'course_creation_use_template', |
||
| 217 | EntityType::class, |
||
| 218 | [ |
||
| 219 | 'class' => Course::class, |
||
| 220 | 'placeholder' => 'Choose ...', |
||
| 221 | 'empty_data' => null, |
||
| 222 | 'data' => null, |
||
| 223 | ] |
||
| 224 | ) |
||
| 225 | ->add('hide_scorm_export_link', YesNoType::class) |
||
| 226 | ->add('hide_scorm_copy_link', YesNoType::class) |
||
| 227 | ->add('hide_scorm_pdf_link', YesNoType::class) |
||
| 228 | ->add('course_catalog_published', YesNoType::class) |
||
| 229 | ->add('course_images_in_courses_list', YesNoType::class) |
||
| 230 | ->add('teacher_can_select_course_template', YesNoType::class) |
||
| 231 | ->add('show_toolshortcuts', YesNoType::class) |
||
| 232 | ->add('enable_record_audio', YesNoType::class) |
||
| 233 | ->add('lp_show_reduced_report', YesNoType::class) |
||
| 234 | ->add('course_creation_splash_screen', YesNoType::class) |
||
| 235 | ->add('block_registered_users_access_to_open_course_contents', YesNoType::class) |
||
| 236 | ; |
||
| 239 |