Forgive me if I've missed something obvious, but it appears that with the new Date Repeat form in D7 one can select only a single byday/bymonth/etc. ordinal because of the single-valued selection list. Thus, I cannot create a date that repeats, for example, on the 1st and 3rd Fridays of every month; I can only select the 1st Friday or 3rd Friday. I could do this with the old Date Repeat form because it had a multivalued selection list. It does not appear that there is anything in the date processing that would preclude an rrule with multiple byday values, but I cannot find any way to create such an rrule through the UI.
Still, thanks for the great work on such an essential and outstanding module!
Comments
Comment #1
KarenS CreditAttribution: KarenS commentedThat would mean reworking the UI again, and that's a bear. If it's important to you you can try to figure out a patch to make it so.
Comment #2
Abenezer CreditAttribution: Abenezer commentedHas anybody figured out an elegant solution to this issue? I would think that it is a very common occurrence when building websites for schools or other institutions that offer periodic event schedules. Right now I have two different clients that would like this feature on their websites — one is an architectural firm and one is a dance studio, but both need the same feature, so it has to be pretty common to run into this.
Any thoughts, or has anyone created a patch for this, or perhaps there are other modules that can mimic this functionality when used in tandem with date? If I were more of a programmer, I'd be happy to create a patch, but unfortunately the issue is way loftier than my skill levels : ) I really need to find more time to take some php courses!
Also, KarenS; Thanks so much for all of your time with this really, really robust module!
Comment #3
saweyer CreditAttribution: saweyer commentedI too was puzzled when the Advanced tab for Repeating went away in 7.x (and kept searching around for configuration settings).
Interestingly when I migrated my site from 6.26, I had 2 several events like this: e.g., with !st & 3rd Monday; -- and those values were preserved until I edited the events, and then they changed to just one of the days per month. So, I guess it is just a u.i. issue -- and not an underlying problem w/ storing the rule or multiple dates?
As an alternative to re-including the previous interface, might it be possible to just have the popup allow multiple selections, i.e., adding MULTIPLE attribute to the SELECT, which would use ctrl-click (or cmd-click on Mac) to add/subtract values?
thanks
Steve
Comment #4
Matthew Davidson CreditAttribution: Matthew Davidson commentedHere's my first attempt at wrestling this particular bear. I initially intended to make more elements multiple select, but when I thought about it, I couldn't see any real-world benefit in doing so to anything but days of the week, and the form's complicated enough already.
Can't see any unintended side-effects so far.
Comment #5
Matthew Davidson CreditAttribution: Matthew Davidson commentedOops. Change status.
Comment #6
nairb CreditAttribution: nairb commentedOne thing I noticed- when you return to the edit screen only the first select item is highlighted.
Comment #7
Matthew Davidson CreditAttribution: Matthew Davidson commentedAh, yes. You're quite right. Try this.
Comment #8
nairb CreditAttribution: nairb commentedLooks good to me. I have played with it a bit this morning and haven't noticed any issues.
Comment #9
Adrianm6254 CreditAttribution: Adrianm6254 commentedI have also applied the patch and it is working great for me.
Comment #10
areikiera CreditAttribution: areikiera commentedWorks great for me too!
In my case, I have events occurring on the first and third weekends. The patch allows you to select multiple values like 'First' and 'Third', but does not allow you to select both 'Saturday' and 'Sunday'. I imagine there is more complicated code needed for something like this. Someone may have an event occuring for the first and third Fridays, and the second and forth Saturdays.... so for a situation like that, it seems like having the ability to 'Add addition' (like we're able to do on 'include dates'), except for the entire fieldset of "On the _____ day of x, x, x...".
Is that possible? Is there a better approach?
Also, variable times on repeating dates would be amazing. If i have a weekend event, with times on Friday from 5pm-10pm, Saturday Noon-Midnight, and Sunday Noon-6pm, it would be great to specify those times on the repeating values. Is that separate issue worthy?
Comment #11
areikiera CreditAttribution: areikiera commentedClosed http://drupal.org/node/613614 as duplicate. The comment on http://drupal.org/node/613614#comment-4204464, is requesting the same kind of functionality as I'm requesting in my previous comment here #10.
Comment #12
nairb CreditAttribution: nairb commentedIf you set the event to start on Saturday and end on Sunday, you can set it to repeat on the 1st and 3rd Saturdays and it will cover each weekend.
Comment #13
areikiera CreditAttribution: areikiera commented@nairb: Thanks! That's a good idea, but in my case I'm grouping by day, and events that span multiple days (instead of repeating each day) only show up on the first day, and I need it to show up on each day it spans (as noted in this issue: http://drupal.org/node/1960240). Is there a way to configure the view so spanning events show up on each day that I'm not aware of?
There is also the case that an event may have different hours on Saturday than on Sunday, but that's more related to the variable hours option.
Thanks!
Comment #14
awt CreditAttribution: awt commentedI've applied the patch and it works !
Thanks :)
For those who don't know how to patch : http://drupal.org/patch/apply
(I had to look for myself first, so maybe there are others like me)
Comment #16
heddn#7: date_repeat-multiple_day_of_week_rrule_values-1437220-7.patch queued for re-testing.
Comment #18
Renee S CreditAttribution: Renee S commentedThis patch worked great for me.
Comment #19
Renee S CreditAttribution: Renee S commentedComment #20
vijaycs857: date_repeat-multiple_day_of_week_rrule_values-1437220-7.patch queued for re-testing.
Comment #23
heddn#2127497: Spam Report opened.
I think the reason #7 doesn't pass tests is because of brittle tests. The actual patch seems to operate correctly.
Comment #24
vijaycs85we may need to fix the test cases...
Comment #25
bgilhome CreditAttribution: bgilhome commentedHere's an updated patch against the latest 2.x HEAD. Some variables names had changed (lowercase).
Comment #26
bgilhome CreditAttribution: bgilhome commentedComment #28
echoz CreditAttribution: echoz commentedSince the latest patch in #25 (Sep 3), there's a newer dev (Oct 2) besides a newer release version (Sep 8). Anyone for a reroll, or is this patch still working in 7.x-2.9 except for tests?
Comment #29
suhel.rangnekar CreditAttribution: suhel.rangnekar as a volunteer and at Faichi Solutions Pvt Ltd commentedPatch is working on branch 7.x-2.x and drupal version 7.42-dev
Re-rolled!
Comment #31
echoz CreditAttribution: echoz commentedFantastic! The #29 patch applied cleanly to Date 7.x-2.9 and works great, thanks so much to all who put this together.
Since I didn't patch this on dev, my testing is not qualified for RTBC. It seems it also needs tests to pass. I just wanted to add to the reports that it works.
Comment #34
aitala CreditAttribution: aitala commentedNow if it works for Date 2.10 ...
E
Comment #35
echoz CreditAttribution: echoz commentedpatch #29 applies cleanly to Date 7.x-2.10 and seems to be working fine. I did not patch dev but that should apply too, needs tests.
I have been running #29 for a long time on 7.x-2.9 without issue.
Comment #36
aitala CreditAttribution: aitala commentedSeems to work just fine.
Of course, going through my event nodes, the first instance of multiple occurrences I found was 2nd Tuesday and 4th Friday of the month. Heh.
Thanks,
Eric
Comment #37
aitala CreditAttribution: aitala commentedNow we need to test this on the 2.11 (future) release...
E
Comment #38
echoz CreditAttribution: echoz commentedI'm unclear for what's needed to get this to RTBC in hopes to include it with #2867810: Plan for Date 7.x-2.11 release
Comment #39
steinmb CreditAttribution: steinmb as a volunteer commentedWe need to make the tests pass.