From 36f00f456bbd78276a91ff7372a086cc8d343920 Mon Sep 17 00:00:00 2001
From: Troels Ugilt Jensen <6103205+tuj@users.noreply.github.com>
Date: Wed, 14 Aug 2024 10:38:20 +0200
Subject: [PATCH] 2000: Minor fixes

---
 src/components/util/schedule/schedule.jsx | 30 +++++++++++++++--------
 src/translations/da/common.json           | 13 +++++-----
 2 files changed, 27 insertions(+), 16 deletions(-)

diff --git a/src/components/util/schedule/schedule.jsx b/src/components/util/schedule/schedule.jsx
index 22748bcd..7e931d4e 100644
--- a/src/components/util/schedule/schedule.jsx
+++ b/src/components/util/schedule/schedule.jsx
@@ -39,6 +39,7 @@ function Schedule({ schedules, onChange }) {
   const byMonthOptions = getByMonthOptions(t);
   const [localSchedules, setLocalSchedules] = useState([]);
   const [durationError, setDurationError] = useState(false);
+  const [activatedRepeatIds, setActivatedRepeatIds] = useState({});
 
   useEffect(() => {
     const newSchedules = schedules.map((schedule) =>
@@ -209,7 +210,7 @@ function Schedule({ schedules, onChange }) {
   }
 
   const isRepeatingEvent = (schedule) => {
-    return !(schedule.duration === 60 * 60 &&
+    return !(
       schedule.freq === RRule.DAILY &&
       schedule.count === 1 &&
       schedule.until === undefined &&
@@ -221,11 +222,19 @@ function Schedule({ schedules, onChange }) {
       schedule.byweekno === undefined);
   }
 
+  const displayRepeat = (schedule) => {
+    return isRepeatingEvent(schedule) || activatedRepeatIds[schedule.id] === true;
+  }
+
+  const updateActivatedRepeats = (id, value) => {
+    const updated = {...activatedRepeatIds};
+    updated[id] = value;
+    setActivatedRepeatIds(updated);
+  }
+
   const toggleRepeat = (schedule) => {
-    if ((isRepeatingEvent(schedule) || schedule.repeatActivated)) {
+    if (displayRepeat(schedule)) {
       // Disable repeating schedule.
-      changeSchedule(schedule.id, 'repeatActivated', false);
-      changeSchedule(schedule.id, 'duration', 60 * 60);
       changeSchedule(schedule.id, 'freq', RRule.DAILY);
       changeSchedule(schedule.id, 'count', 1);
       changeSchedule(schedule.id, 'interval', null);
@@ -235,10 +244,11 @@ function Schedule({ schedules, onChange }) {
       changeSchedule(schedule.id, 'byweekday', null);
       changeSchedule(schedule.id, 'bymonth', null);
       changeSchedule(schedule.id, 'byweekno', null);
+      updateActivatedRepeats(schedule.id, false);
     } else {
       // Activate repeating schedule.
-      changeSchedule(schedule.id, 'repeatActivated', true);
       changeSchedule(schedule.id, 'count', null);
+      updateActivatedRepeats(schedule.id, true);
     }
   }
 
@@ -307,13 +317,13 @@ function Schedule({ schedules, onChange }) {
                     onChange={() => toggleRepeat(schedule)}
                     name="enableRepeat"
                     type="checkbox"
-                    value={(isRepeatingEvent(schedule) || schedule.repeatActivated)}
+                    value={displayRepeat(schedule)}
                     label={t('schedule.enable-repeat')}
                     formGroupClasses="d-inline-block mb-2"
                   />
                   <Tooltip id={'tooltip-enable-repeat'} content={t('schedule.enable-repeat-tooltip')}></Tooltip>
 
-                  {(isRepeatingEvent(schedule) || schedule.repeatActivated) && (
+                  {displayRepeat(schedule) && (
                     <>
                       <div className="row mt-2">
                         <div className={`col col-md-${!isNaN(schedule.byhour) ? "3" : "6"}`}>
@@ -367,7 +377,7 @@ function Schedule({ schedules, onChange }) {
                         <div className="col">
                           <FormGroup>
                             <label htmlFor="byweekday">{t("schedule.byweekday")}</label>
-                            <Tooltip id={'tooltip-byweekday'} content={t('schedule.tooltip-byweekday')}></Tooltip>
+                            <Tooltip id={'tooltip-byweekday'} content={t('schedule.byweekday-tooltip')}></Tooltip>
                             <MultiSelect
                               className="mt-2"
                               options={byWeekdayOptions}
@@ -404,7 +414,7 @@ function Schedule({ schedules, onChange }) {
                             <label htmlFor="bymonth" className="mb-2">
                               {t("schedule.bymonth")}
                             </label>
-                            <Tooltip id={'tooltip-bymonth'} content={t('schedule.tooltip-bymonth')}></Tooltip>
+                            <Tooltip id={'tooltip-bymonth'} content={t('schedule.bymonth-tooltip')}></Tooltip>
                             <MultiSelect
                               options={byMonthOptions}
                               value={getByMonthValue(schedule.bymonth)}
@@ -460,7 +470,7 @@ function Schedule({ schedules, onChange }) {
                     </>
                   )}
                 </div>
-                {(isRepeatingEvent(schedule) || schedule.repeatActivated) && (
+                {displayRepeat(schedule) && (
                   <div className="card-footer">
                     <div id="schedule_details" className="row">
                       <div className="mb-2">
diff --git a/src/translations/da/common.json b/src/translations/da/common.json
index a2ca48d0..406f1720 100644
--- a/src/translations/da/common.json
+++ b/src/translations/da/common.json
@@ -789,17 +789,18 @@
     "count": "Antal gentagelser",
     "count-placeholder": "Ubegrænset",
     "interval": "Interval",
-    "dtstart-tooltip": "Start definerer tidspunkt på dagen og første mulige dato for gentagelserne, hvis dette aktiveres.",
-    "end-tooltip": "Forskellen imellem start og slut definerer længden på en gentagelse.",
     "repetition": "RRule",
     "enable-repeat": "Gentag planlægning",
     "enable-repeat-tooltip": "Når gentagelse aktiveres bestemmer reglerne man vælger, hvornår spillelisten bliver vist.",
-    "frequency-tooptip": "Hvordan planlægningen skal gentages?",
+    "end-tooltip": "Forskellen imellem start og slut definerer længden på en gentagelse.",
+    "dtstart-tooltip": "Start definerer tidspunkt på dagen og første mulige dato for gentagelserne, hvis dette aktiveres.",
+    "frequency-tooltip": "Hvordan planlægningen skal gentages?",
     "interval-tooltip": "Skal der være spring imellem gentagelse? Afhænger af frekvensen. F.eks. hver 3 dag.",
     "count-tooltip": "Hvor mange gange skal planlægningen gentages? Tom værdi betyder ubegrænset.",
-    "tooltip-byweekday": "På hvilke dage skal reglen gælde?",
-    "tooltip-bymonth": "I hvilke månender skal reglen gælde?",
-    "tooltip-byweekno": "Hvilket ugenummer skal reglen gælde for?",
+    "bytime-tooltip": "På hvilke dage skal reglen gælde?",
+    "byweekday-tooltip": "På hvilke dage skal reglen gælde?",
+    "bymonth-tooltip": "I hvilke månender skal reglen gælde?",
+    "byweekno-tooltip": "Hvilket ugenummer skal reglen gælde for?",
     "until-tooltip": "Hvornår skal gentagelserne afsluttes? Dette tidspunkt påvirker starttidspunktet på en gentagelse, ikke sluttidspunktet.",
     "helptext": "Når minimum én planlægning er tilføjet, bliver spillelisten kun vist i planlægningsintervallet."
   },