Re: CVS ltp/projects

From: Martin Schulze (joey@infodrom.org)
Date: Sat May 04 2002 - 19:46:10 CEST


Erik Tews wrote:
> > > > +<when <not <string-eq "$(INSURANCE)" "" />>>
> > > > <row_radiobox name=insurance
> > > > title="<b>Insurance</b>"
> > > > checked_0="<? if ($row['insurance'] == 0) { echo 'checked'; } ?>"
> > > > checked_1="<? if ($row['insurance'] == 1) { echo 'checked'; } ?>"
> > > > >
> > > > +</when>
> > >
> > > What about updating update.wml?
> >
> > Well, always inserting 0 doesn't hurt, right?
>
> Well, should this code be secure? With this patch I am still able to
> update insurance if I want to.

Shouldn't matter since it isn't used. But you're correct. Something
like the following will insert proper defaults, without rendering
the queries unreadable.

<when <string-eq "$(INSURANCE)" "" />>
   $insurance = 0;
</when>

Regards,

        Joey

-- 
We all know Linux is great... it does infinite loops in 5 seconds.
        -- Linus Torvalds



This archive was generated by hypermail 2.1.3 : Sat May 04 2002 - 19:57:14 CEST