There is still an issue with DateAPI, but this patch fixes the issue with the schema change.

CommentFileSizeAuthor
bookingsapi.module.patch3.88 KBkreynen

Comments

dwees’s picture

dwees commented

This is just renaming rrule to ical_rrule. I have all of my module stuff labelled rrule and my version (with the small patch to the Date API module) seems to run smoothly. Why is this fix necessary?

Dave

kreynen’s picture

kreynen commented
Title: schema change from rrule to ical_rrule not reflected in module » schema change from ical_rrule to just rrule not reflected in module

If you're doing a clean install of 6.x-1.x.dev, this patch isn't necessary. If you installed the alpha1 release ical is dropped as a prefix to the rrule tables. Running update.php doesn't correct for this schema change, but it could also be corrected with an update to the install file.

I've changed the title of this issue to better reflect the what the patch actually does.

dwees’s picture

dwees commented

Yeah the alpha release of this module needs to be updated, or a new release created the .dev is much, much better.

dwees’s picture

dwees commented

Yeah the alpha release of this module needs to be updated, or a new release created the .dev is much, much better.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Closed (won't fix)

As far as I understood the patch and this thread, this patch is obsolete if a clean install of dev is done or if the new update method in dev has been called - I'm setting this to "won't fix" without committing the patch.
Please feel free to reopen if you believe, that some changes are still necessary. (And please - feel free to continue committing any patches you like - I promise to have a look at them!)