Re: CVS ltp/projects

From: Erik Tews (erik.tews@gmx.net)
Date: Sat May 04 2002 - 17:22:11 CEST


On Sat, May 04, 2002 at 05:06:41PM +0200, Martin Schulze wrote:
> Erik Tews wrote:
> > On Sat, May 04, 2002 at 04:23:01PM +0200, CVS User joey wrote:
> > > Update of /var/cvs/ltev/ltp/projects
> > > In directory luonnotar:/tmp/cvs-serv9170
> > >
> > > Modified Files:
> > > edit_person.wml
> > > Log Message:
> > > We need to make the insurance field optional
> > >
> > > --- /var/cvs/ltev/ltp/projects/edit_person.wml 2002/05/03 09:18:21 1.28
> > > +++ /var/cvs/ltev/ltp/projects/edit_person.wml 2002/05/04 14:23:01 1.29
> > > @@ -117,11 +117,13 @@
> > > checked_1="<? if ($row['infomail'] == 1) { echo 'checked'; } ?>"
> > > >
> > >
> > > +<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.



This archive was generated by hypermail 2.1.3 : Sat May 04 2002 - 17:23:45 CEST