fix DAILY frequency when only 1 BYHOUR is provided

fixes https://github.com/arolson101/rrule-alt/issues/6
minhtandnvn · Feb 14, 2018 · 6fd1e1e · 6fd1e1e
1 parent 1aa59b7
commit 6fd1e1e
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/lib/nlp.js b/lib/nlp.js
@@ -402,6 +402,9 @@
       },
 
       list: function (arr, callback, finalDelim, delim) {
+        if (!(arr instanceof Array)) {
+          arr = [arr]
+        }
         var delimJoin = function (array, delimiter, finalDelimiter) {
           var list = ''
 

diff --git a/test/rrule.js b/test/rrule.js
@@ -3524,4 +3524,16 @@ describe('RRule', function () {
       assert.equal(rrstr, newrr.toString())
     })
   })
+
+  it('testByHourValues', function () {
+    [
+      ['DTSTART=20171101T010000Z;UNTIL=20171214T013000Z;FREQ=DAILY;INTERVAL=2;WKST=MO;BYHOUR=11,12;BYMINUTE=30;BYSECOND=0', 'every 2 days at 11 and 12 until December 13, 2017'],
+      ['DTSTART=20171101T010000Z;UNTIL=20171214T013000Z;FREQ=DAILY;INTERVAL=2;WKST=MO;BYHOUR=11;BYMINUTE=30;BYSECOND=0', 'every 2 days at 11 until December 13, 2017']
+    ].forEach(function (pair) {
+      var rule = pair[0]
+      var str = pair[1]
+      var rr = RRule.fromString(rule)
+      assert.equal(rr.toText(), str)
+    })
+  })
 })