Begin forwarded message:

From: "David T. Lewis" <lewis@mail.msen.com> Date: June 17, 2008 2:50:58 AM CEDT To: Seaside - general discussion <seaside@lists.squeakfoundation.org> Subject: Re: [Seaside] Timespan bug Reply-To: Seaside - general discussion <seaside@lists.squeakfoundation.org

...

On Mon, Jun 16, 2008 at 01:50:43PM +1000, Steve Aldred wrote:

...

Sorry for posting this here but I don't use Squeak itself and hence aren't on the main squeak list.

As a VW Seaside user I do use Squeak extensions. There is a bug in Timespan as shown below:

(Year starting: (DateAndTime year: 2008 month: 1 day:1)) previous

The result should be 2007 but it returns 2006.

Year should implement previous the same as Month does.

Steve,

Thanks for reporting this issue. I was going to enter a report on your behalf in the Squeak bug tracking system, but it looks like Brent is already on top of it :)

http://bugs.squeak.org/view.php?id=7097

The bug report includes a proposed fix, which is straightforward if you want to include it in your image:

http://bugs.squeak.org/file_download.php?file_id=3386&type=bug

Brent's fix is as follows:

Year>>previous "This implementation handles leap years correctly" ^ self class year: (self year - 1)

Dave

_______________________________________________ seaside mailing list seaside@lists.squeakfoundation.org http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside