On Sat, Jun 14, 2008 at 10:42 AM, Christopher J. Bottaro
<cjbottaro at alumni.cs.utexas.edu> wrote:> I noticed two little bugs when using your vpim module in a Rails
application.
>
> 1) The vpim module overwrites Rail''s implementation of
Date#to_time.
> 2) The vpim module''s implementation of Date#to_time causes
> Date.today.to_time to return a Time that is a day in the past.
>
> I don''t know how to fix issue 2, but I have attached a small patch
> that makes vpim''s Date#to_time use any existing implementation
before
> just defining it itself.
>
> Please let me know if the patch is accepted.
Vpim''s Date#to_time may break rails'', but the opposite is also
true.
With your patch Rails will work, but Vpim will be broken.
If you can give me a patch that renames Vpim''s Date#to_time (and the
calls to it) with
something like Date#vpim_to_time, I would very happily merge that.
Thanks,
Sam