Kinky Calendar Leap Year Update

By Ben Nadel

Published 2008-03-04 in ColdFusion — Comments (8)

Just a quick update - Peter Bell pointed out to me that my ColdFusion Kinky Calendar system was breaking on my site. I took a quick look, and indeed it was. Oh No! So, I threw on my ColdFusion ninja suit and jumped into action. The problem was a little mysterious to debug, but I finally narrowed it down to a leap year problem. The leap year itself was not causing the problem; the problem was that I had some code that looked like this:

CreateDate(
	(Year( start_date ) - 1),
	Month( start_date ),
	Day( start_date )
	)

The problem here is that on February 29th of this year, the above code tried to create the 29th of last year. Of course, there was no February 29th of last year. This was an easy fix with some conditional logic.

Thanks Peter for bringing that to my attention. I have updated the Kinky Calendar project page with a new ZIP download.

Want to use code from this post? Check out the license.

Short link: https://bennadel.com/1173

Reader Comments

Sami Hoda Mar 4, 2008 at 4:06 PM

20 Comments

Can you tell us which file (line numbers) were changed so we can update our copies?

Steve Bryant Mar 4, 2008 at 4:55 PM

59 Comments

Why not use DateAdd("yyyy",-1,start_date) ?

Werbeagentur Mar 5, 2008 at 4:14 AM

2 Comments

Thanks! Sometimes February has only 28 days :-)

Ben Nadel Mar 6, 2008 at 8:57 AM

15,902 Comments

@Sami,

It was the _functions.cfm file. It was several different lines for the month repeat and the year repeat. It's probably just easiest to download the ZIP and replace that function. Or, if you just want to swap out those case statements:

<cfcase value="4">

and

<cfcase value="5">

@Steve,

Hmmm, not sure. I think you might be on to something :) I love when I overlook the most obvious solution! Thanks Steve.

Ben Nadel Mar 6, 2008 at 10:10 AM

15,902 Comments

@Steve,

Thanks for the tip. I updated the code with your DateAdd() suggestion. Seems to work like a charm.