On Wed, 2008-05-28 at 16:47 -0400, Robert Cummings wrote: > Out of curiosity.. have you thought about reworking your premise? Using > prev, current, and next is extremely sloppy IMHO. Additionally, it > prevents bookmarking a specific calendar date. Why don't you have a date > parameter that indicates the year and month? If no such field exists > then you fallback to current date. Then the situation becomes trivial > and doesn't require dependence on the session. Having looked at your > code it strikes me as very poorly thought out which makes it difficult > to ascertain your current problem. At any rate, it's obviously more > complex than is necessary. > > <?php > > $date = isset( $_GET['date'] ) ? $_GET['date'] : date( 'Y-m' ); > list( $year, $month ) = explode( '-', $date ); > $timestampCurr = mktime( 0, 0, 0, $month, 1, $year ); > > $timestampPrev = strtotime( '-1 month', date( 'Y-m-d', $timestamp ) ); > $timestampNext = strtotime( '+1 month', date( 'Y-m-d', $timestamp ) ); Those two lines above should be: $timestampPrev = strtotime( '-1 month', $timestampCurr ); $timestampNext = strtotime( '+1 month', $timestampCurr ); Cheers, Rob. -- http://www.interjinn.com Application and Templating Framework for PHP -- PHP General Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
