This is a difficult to decide. For me it is both ways. Since half of
my forms require zero=None and the other half require zero='', either
way it goes there will be no option that is more "convenient" for my
development.
I stand in support of zero=None. Web2py is a data/business logic
oriented fra
On Feb 4, 2010, at 7:34 AM, mdipierro wrote:
> Mind that currently zero='' should be interpreted as zero='(please
> choose a value)'. the value '' is not in the db and will not pass
> validation. It s just a place holder. Only if
> IS_NULL_OR(IS_IN_DB(...zero='')) then the value '' can pass
> vali
On Feb 4, 2010, at 7:08 AM, mdipierro wrote:
> I do not really have an opinion on this. I am only reluctant to
> changes in behavior.
> Anyway, I am prepared to make this change (zero=None by default). I
> would like to know that all major contributors are on board so that
> you can answer the ema
Excellent suggestion on adding the comments with a link to the
discussion thread Iceberg
-Thadeus
On Sat, Jan 30, 2010 at 2:07 AM, Iceberg wrote:
> When saying "some people ask for zero=None, some for zero='', some for
> zero='Please choose a value'", ok, I am the one who once suggest the
>
+1 supporting DenesL argument. (but we already know this)
note.
default.
breaks app.
we like the feature
but it shouldn't break an app
no reason to have to specifically DISABLE something that hasn't
existed in web2py before ( you know, the empty select or "please
choose a value" breaks my app
Yes, the option is important. But the case for a default of None is
pretty convincing.
On 1/29/10, mdipierro wrote:
> Let's hear a few more opinions. I don't just want to say no. The
> problem is that we had this discussion already and people insisted
> such option should be there.
>
> Massimo
>
My thoughts exactly :)
-Thadeus
On Thu, Jan 28, 2010 at 12:55 PM, DenesL wrote:
> Thanks, I've seen it.
>
> The problem with format and IS_IN_DB can be easily fixed in sql.py but
> I would be against setting the default of zero to '' in IS_IN_DB and
> IS_IN_SET if it creates an empty option
7 matches
Mail list logo
