While doing some complex queries involving events, CCK and a few custom tables I found that a vid field in the event table would be quite useful to simplify my SQL writing.

This patch adds the vid field to event table in the original create table as well as an update function to alter the existing table.

It also inserts vid data in event_nodeapi.

I've tested creating and updating events and everything looks ok, actually the vid is not used at all in all event related modules.

CommentFileSizeAuthor
event_vid.patch4.1 KBalexis
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gerhard Killesreiter’s picture

Are you sure you want to add that to the 5.1 version? 49st is using 5.2 (and I only want to add new features to that branch).

alexis’s picture

Version: 5.x-1.0 » 5.x-1.x-dev

Oh, that was my mistake, sure, I want to add to event 5.2 but I was not too sure about the version in CVS, I used this to checkout:

cvs -z6 -d:pserver:anonymous:anonymous@cvs.drupal.org:/cvs/drupal-contrib co -r DRUPAL-5 contributions/modules

If you tell me the correct branch or checkout command I can redo the change and submit a reviewed patch.

Thanks!

Gerhard Killesreiter’s picture

Version: 5.x-1.x-dev » 5.x-1.0

-r DRUPAL-5--2

japerry’s picture

Status: Needs review » Closed (outdated)

Event for Drupal 8 is unrelated to older versions. If an issue similar to this one exists, please open a new issue with the 8.x branch.