Can anyone help me in getting basic event nodes working? I don't see any place to enable them. I'm running the current CVS HEAD as of 11/15/05 and am excited that event.module is working again for me. I do have the basic event module enabled. However, under administer/settings/content types, I see only forum topic, page, and story. So, I can't create any events. (Don't have flexinode installed.)

Thanks for any help!

CommentFileSizeAuthor
#2 basicevent.module.patch5.49 KBdrewish

Comments

drewish’s picture

The current basic node is pretty broken. At a minimum, you'll need to replace event_node_name() with:

/**
 * Implementation of hook_node_info().
 */
function basicevent_node_info() {
  return array('event' => array('name' => t('event'), 'base' => 'event'));
}

There's a bunch of code for the location module that seems un-needed, the location module should be able to do the same thing by way hook_nodeapi().

I'll roll a patch that cleans out all the borked stuff.

drewish’s picture

Status: Active » Needs review
StatusFileSize
new5.49 KB

Here's the patch, it pretty much replaces the old module. Let me know if it works for you.

killes@www.drop.org’s picture

The current basicevent is the result of me too much listening to certain people. ;)

There was a lengthy discussion leading to this being in cvs. I still am wondering why I did that.

The original issue is here:

http://drupal.org/node/27053

Your patch looks like it would make the basicevent module more like the other suggestions (no location etc). Care to look at those and see if there are good ideas hiding in them? In either case, please commit this patch (and respond to any issues that might come up wrt it ;).

People who want to have an event type that integrates with location.module should make a event_location module and maintain that in parallel.

drewish’s picture

I'll take a look at them. I got the feeling that the module was designed for 4.6.

I've been using this version of basicevent with the CVS version of location module and it works great. You have to enable it in the content types screen but it works fine. It seems like the new forms API has made the modules much more... modular ;)

Do you want to do a branch of the module for 4.6?

killes@www.drop.org’s picture

The idea was to have this for 4.6 too, yes. Can you just pick one of the non-locale enabled versions which would not require an upgrade path and add that for 4.6?

drewish’s picture

I've commited the patched version to CVS.

gtcaz, go ahead and update your version of the module. You'll need to make sure that:
* the basicevent module is enabled (on the admin/modules page)
* the event node is shown in the calendar (on the admin/settings/content-types/event page)
If you're still having problems, you can switch this issue's status back to active.

drewish’s picture

Status: Needs review » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)