Problem/Motivation

When editing an existing Event Series, if you try to change the "Recur on Day of the Week" setting, within a Monthly recurring event, the change/data is not changed and no confirmation is displayed.

Steps to reproduce

  • Create a new Event Series
  • Set it to recur Monthly, and set a day for "Recur on Day of the Week"
  • Fill in other fields and save
  • Return to the Event Series you created
  • Change the day of the recurrence to a different day than what was first selected
  • Note that you are not able to save and there is no confirmation of changes message

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork recurring_events-3238591

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

hotsaucedesign created an issue. See original summary.

owenbush’s picture

owenbush commented
Issue tags: +rc eligible

Thanks for the issue. I'll get this looked at asap.

owenbush opened merge request !23

owenbush’s picture

owenbush commented
Status: Active » Needs review

I have created a merge request to address this issue, and the problem with throwing lots of PHP Notices in error logs when it comes to generating the diff tables.

You can use the following as a patch URL to test the MR:

https://git.drupalcode.org/project/recurring_events/-/merge_requests/23....

owenbush’s picture

owenbush commented

Crediting hotsaucedesign for the issue.

hot_sauce’s picture

hot_sauce commented

Thank you owenbush, tested the patch locally and it works as expected!

owenbush’s picture

owenbush commented
Status: Needs review » Reviewed & tested by the community

Confirmed reviewed & tested by hotsaucedesign

  • owenbush committed 93e789c on 2.0.x 
    Issue #3238591 by owenbush, hotsaucedesign: Unable to save changed "...

  • owenbush committed 3203429 on 8.x-1.x 
    Issue #3238591 by owenbush, hotsaucedesign: Unable to save changed "...
owenbush’s picture

owenbush commented
Status: Reviewed & tested by the community » Fixed

This has now been fixed, thanks to hotsaucedesign for the find and help testing.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.