From: David <da...@df...> - 2004-05-19 15:50:15
|
The suggestion about autoSave brought me to one idea: why not auto-save the form data whenever a form is submitted and use save() only for saving the data in the session? That would solve some frequent mistakes: - people forget to call saveAll() and wonder why getValue() doesn't return anything - people don't reset() the form after processing the data, so all values are still filled in when the visitor comes back to the form again. What I suggest is not really to auto-save the form data but rather change getValue() to read from the form data and only fall back to the session data if the form was not submitted. save() would only be needed if the form data is neede on another page again. I would also remove the field saving from check(), since form data can be read without being saved before. I think that would be a major improvement and increase usability a lot. About the password field: I would not change anything about saving but rather change only the smarty function to not add the value attribute (unless the keepvalue or whatever attribute is preset). - david On Sat, 15 May 2004 20:42:08 +0200 Katana <ka...@ka...> wrote: > > we've been talking about not keeping values in password fields for > > security reason time ago. > > > > I'll change that in the next days. I'll also add an attribute to > > tell if the value should be kept (default no). How should it be > > called? I'm for keepvalue. > This attribute would only exist for password fields right ? > Maybe we could add it for all fields but default it to yes for all the > > fields but password. > what about autoSave=(bool) ? The only thing I don't like about > autoSave is that it is not auto since you have to run a validator on > the field first... Any better idea ? > > Bertrand > > > ------------------------------------------------------- > This SF.Net email is sponsored by: SourceForge.net Broadband > Sign-up now for SourceForge Broadband and get the fastest > 6.0/768 connection for only $19.95/mo for the first 3 months! > http://ads.osdn.com/?ad_id=2562&alloc_id=6184&op=click > _______________________________________________ > Formsess-devel mailing list > For...@li... > https://lists.sourceforge.net/lists/listinfo/formsess-devel > |