Take setting default value out of loop

diff --git a/apps/calendar/templates/part.eventform.php b/apps/calendar/templates/part.eventform.php
index 4c34b3e1fcc02cc316ee1c0912f8dff3f7f527be..8588b9168f7bb03bee5e60c1cda4e831b4661662 100644 (file)
--- a/apps/calendar/templates/part.eventform.php
+++ b/apps/calendar/templates/part.eventform.php
@@ -12,8 +12,8 @@
                        <td>
                        <select id="category" name="categories[]" multiple="multiple" title="<?php echo $l->t("Select category") ?>">
                                <?php
+                               if (!isset($_['categories'])) {$_['categories'] = array();}
                                foreach($_['category_options'] as $category){
-                                       if (!isset($_['categories'])) {$_['categories'] = array();}
                                        echo '<option value="' . $category . '"' . (in_array($category, $_['categories']) ? ' selected="selected"' : '') . '>' . $category . '</option>';
                                }
                                ?>
@@ -22,8 +22,8 @@
                        <td>
                        <select style="width:140px;" name="calendar">
                                <?php
+                               if (!isset($_['calendar'])) {$_['calendar'] = false;}
                                foreach($_['calendar_options'] as $calendar){
-                                       if (!isset($_['calendar'])) {$_['calendar'] = false;}
                                        echo '<option value="' . $calendar['id'] . '"' . ($_['calendar'] == $calendar['id'] ? ' selected="selected"' : '') . '>' . $calendar['displayname'] . '</option>';
                                }
                                ?>