[Nitro] [PATCH] Unset integer properies should be nil not 0, which is a value.
rob at motionpath.com
Sat Feb 25 04:05:12 EST 2006
Which one? I know can be slap-dash with running the glycerin test
suite (i.e. not run it), especially with something like this since
the old behavior is clearly broken and caused two of my projects to
explode when I upgraded them from 0.23 because scaffold wanted to put
0 into relation foreign key fields where I was using a hidden text
box for the relation control and left it blank (see why it has to
There would be other places I could change the behavior, I'll take a
look Monday to see if glue was the most appropriate place (and if it
was, fix the other problems caused by this).
On 25 Feb 2006, at 08:03, Bryan Soto wrote:
> On 2/20/06, Rob Pitt <rob at motionpath.com> wrote:
> Empty integer properties should be nil not zero. Zero is valid number
> that you may wish to explicitly set, nil is unmistakably empty.
> This one does break a test in Glue's test suite. Not sure if it's a
> big deal or not.
> Nitro-general mailing list
> Nitro-general at rubyforge.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Nitro-general