Is it possible to configure GCal Events to display the end date as well as the start date of the event?

CommentFileSizeAuthor
#6 gcal_events_50.patch3.19 KBjdwfly
#9 gcal_events_51.patch4.25 KBjdwfly

Comments

Se7enLC’s picture

Se7enLC commented

it's not currently possible, but it should be a trivial change to add that. That feature will definitely be in a later release as soon as I find time to code it in.

frednwright’s picture

frednwright commented

Thanks Se7enLC!

It sounds silly, but I have several clients who love the module, but they all want the end date to display as well. It's a great module and I'm encouraging people who use it to buck up and donate.

Fred

Se7enLC’s picture

Se7enLC commented

Check out the latest CVS revision of gcal_events.module - I just added in simple support for #ENDTIME# is that what you're looking for, or do you want an end-date as well?

frednwright’s picture

frednwright commented

Hey Se7enLC!

Thanks for the update - I am indeed looking for an "end date" variable, but the #ENDTIME# variable is incredibly handy as well.

sorry for the delay on responding - I just got back from a trip and saw your post.

fred

jdwfly’s picture

jdwfly commented

I need this functionality as well. I am currently working on it and hopefully will have a patch soon.

jdwfly’s picture

jdwfly commented
Version: 7.x-1.x-dev » 6.x-1.4
Status: Needs work » Needs review
StatusFileSize
new gcal_events_50.patch3.19 KB

I have never made a patch before so I hope this works. I used the endtime that was already made to extend functionality for all day events spanning multiple days. With the proper templates you can have multi-day events.

Check this out and tell me what I did wrong. :)

Se7enLC’s picture

Se7enLC commented
Status: Active » Needs review

Looks like a good patch! I didn't try to apply it, yet. I just looked over it quickly. I'll put that in cvs once I get time to test it out. Good call on making endtime only include the date when the event is > 1 day, I think that will probably handle most people's needs.

jdwfly’s picture

jdwfly commented

I am having a slight issue, but I am not sure that it is related to what I have done.

I have an event that is 3/5/09 - 3/6/09, but I can't get it to show up as the multiple days. It only shows the 3/5/09. Any ideas?

jdwfly’s picture

jdwfly commented
StatusFileSize
new gcal_events_51.patch4.25 KB

Figured it out. I was using a variable that had not yet been defined. So I moved that earlier in the loop and here is the new patch.

jdwfly’s picture

jdwfly commented

There seems to be some faulty logic in this because I am having an issue where it is still showing the end time when the event is a single day. I will be working on this today and should have a fix out soon.

jdwfly’s picture

jdwfly commented
Version: 6.x-1.4 » 7.x-1.x-dev
Status: Needs review » Needs work

Ouch, should have committed this long ago. Anyway, bumping to 7.x and then I will back port it to 6.x.

jdwfly’s picture

jdwfly commented
Version: 6.x-1.4 » 7.x-1.x-dev
Status: Needs review » Patch (to be ported)

committed to 7.x