Repeat day options are checked off for unselected occurrence filters and saved to the RRULE data.

Still looking for a way to fix this - but it can be worked around for now if you filter duplicate values out of the array of weekday items:

 _getRRule: function () {
      //modified from rrule/tests/demo/demo.js
      //ignore 'end', because it's part of the ui but not the spec
      var values = this._getFormValues($(this.element).find('select, input[class!="end-radio"]'));
      var options = {};

      var weekdays = values.byweekday;

      values.byweekday = weekdays.filter(function(item, pos, self) {
        return self.indexOf(item) == pos;
      });

This turns the byweekday item from [0, 2, 4, 0, 2, 4] to just [0, 2, 4] which fixes the UI output in the widget and on the frontend (where the RRULE is displayed in human readable form) and also corrects the RRULE value stored in the database. I believe the bug is in how the values are initially fetched - only the inputs from the currently selected occurrence value should be fetched.

Yeah, now I see where that happens. So while this seemed to work in one case, it fails when you have a date field that can have multiple delta values. (different day/time schedules within the same event). With the duplicates filtered out, it works in some cases, but not all cases.

Is there any reason this widget has to be like 800 lines of jQuery? It makes it really hard to debug and process.

This patch seems to work - I only tested it with weekly so far (my main use case). However I noticed one bug.

If you save a node and edit it, then uncheck "Repeat" but then check it back on - the widget UI has your last saved values pre-selected. However, it does not save these values back at all. If you were to recheck the options that are already selected, then it would work.

Heres another bug... the monthly widget suffers from the same issues, you can see duplicate values in the screenshot.

First, checking off every week option causes the widget to display:

=> TypeError: Cannot read property 'n' of undefined

in the summary area. I am not sure what that means, but this option in the widget is pretty confusing. I want to show this event and it happens monthly, what are these week options? Every other week style option? There is a lot to unpack there, I just wanted to set an event to run every month for six months, two days every week. Perhaps "All month" and "every other week" could be broken out into two new options, but that can be done for a UX feature issue later.

At the same time, selecting week options doubles up on the day values, and vice versa. You can see that in the screenshot.