From 6bc8a2e11fd0b528cc788722aeb0414143f9bf1d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Umberto=20Ciocca=20de=20Almeida?= Date: Wed, 8 Nov 2017 18:36:03 -0200 Subject: [PATCH] MDL-60654 calendar: Clean up lines 140-154 The if+switch code between lines 140 and 154, even though they selected 3 different choices, all of them gave the same result. Therefore, it'd be cleaner to replace those 16 lines for just two. --- calendar/view.php | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/calendar/view.php b/calendar/view.php index f6b0e6f817e..b7fba4d1858 100644 --- a/calendar/view.php +++ b/calendar/view.php @@ -123,21 +123,10 @@ echo $renderer->start_layout(); echo html_writer::start_tag('div', array('class'=>'heightcontainer')); echo $OUTPUT->heading(get_string('calendar', 'calendar')); -if ($view == 'day' || $view == 'upcoming') { - switch($view) { - case 'day': - list($data, $template) = calendar_get_view($calendar, $view); - echo $renderer->render_from_template($template, $data); - break; - case 'upcoming': - list($data, $template) = calendar_get_view($calendar, $view); - echo $renderer->render_from_template($template, $data); - break; - } -} else if ($view == 'month') { - list($data, $template) = calendar_get_view($calendar, $view); - echo $renderer->render_from_template($template, $data); -} + +list($data, $template) = calendar_get_view($calendar, $view); +echo $renderer->render_from_template($template, $data); + echo html_writer::end_tag('div'); list($data, $template) = calendar_get_footer_options($calendar);