Fixed duration issue when having multiple time units

Author: wouterlemcke

ical.js

@@ -470,15 +470,19 @@ module.exports = {
               S: 'seconds'
             };
             // Get the list of duration elements
-            const r = curr.duration.match(/-?\d{1,10}[YMWDHS]/g);
+            const duration = curr.duration.match(/-?\d{1,10}[YMWDHS]/g);
 
             // Use the duration to create the end value, from the start
-            let newend = moment.utc(curr.start);
+            let newEnd = moment.utc(curr.start);
             // Is the 1st character a negative sign?
             const indicator = curr.duration.startsWith('-') ? -1 : 1;
-            newend = newend.add(Number.parseInt(r, 10) * indicator, durationUnits[r.toString().slice(-1)]);
+
+            for (const r of duration) {
+              newEnd = newEnd.add(Number.parseInt(r, 10) * indicator, durationUnits[r.toString().slice(-1)]);
+            }
+
             // End is a Date type, not moment
-            curr.end = newend.toDate();
+            curr.end = newEnd.toDate();
           }
         }
 

test/test-async.js

@@ -851,7 +851,7 @@ vows
         },
         'it uses the start/end of the event'(event) {
           assert.equal(event.start.toJSON(), '2024-02-15T09:00:00.000Z');
-          assert.equal(event.end.toJSON(), '2024-02-15T09:15:00.000Z');
+          assert.equal(event.end.toJSON(), '2024-02-15T10:15:00.000Z');
         }
       }
     },

test/test.js

@@ -1041,7 +1041,7 @@ vows
         },
         'it uses the start/end of the event'(event) {
           assert.equal(event.start.toJSON(), '2024-02-15T09:00:00.000Z');
-          assert.equal(event.end.toJSON(), '2024-02-15T09:15:00.000Z');
+          assert.equal(event.end.toISOString(), '2024-02-15T10:15:00.000Z');
         }
       }
     },

test/test_date_time_duration.ics

@@ -13,6 +13,6 @@ DESCRIPTION:
 LOCATION:Kriftel
 X-APPLE-STRUCTURED-LOCATION;VALUE=URI;X-TITLE=Kriftel: geo:50.083558\,8.4693855
 DTSTART:20240215T090000Z
-DURATION:PT15M
+DURATION:PT1H15M
 END:VEVENT
 END:VCALENDAR