From: jeff Date: Fri, 19 Jan 2007 22:02:21 +0000 (+0000) Subject: blank password on error X-Git-Tag: TRIXBOX_2_6~747 X-Git-Url: http://git.freeside.biz/gitweb/?a=commitdiff_plain;h=32860d44e17b53f220cd98e041219d7d5b8a4619;p=freeside.git blank password on error --- diff --git a/httemplate/edit/process/access_user.html b/httemplate/edit/process/access_user.html index 7aab79954..9f7c4ddbf 100644 --- a/httemplate/edit/process/access_user.html +++ b/httemplate/edit/process/access_user.html @@ -6,6 +6,7 @@ 'table' => 'access_user', 'viewall_dir' => 'browse', 'copy_on_empty' => [ '_password' ], + 'clear_on_error' => [ '_password', '_password2' ], 'process_m2m' => { 'link_table' => 'access_usergroup', 'target_table' => 'access_group', }, diff --git a/httemplate/edit/process/elements/process.html b/httemplate/edit/process/elements/process.html index 5d40bba50..fb5b352d2 100644 --- a/httemplate/edit/process/elements/process.html +++ b/httemplate/edit/process/elements/process.html @@ -24,7 +24,9 @@ % # 'edit_ext' => 'html', #defaults to 'html', you might want 'cgi' while the % # #naming is still inconsistent % # -% # 'copy_on_empty' => [ old_field_name, another_old_field, ... ], +% # 'copy_on_empty' => [ 'old_field_name', 'another_old_field', ... ], +% # +% # 'clear_on_error' => [ 'form_field1', 'form_field2', ... ], % # % # 'process_m2m' => { 'link_table' => 'link_table_name', % # 'target_table' => 'target_table_name', @@ -57,7 +59,7 @@ % } @$fields % } ); % -% if ($old && scalar($opt{'copy_on_empty'})) { +% if ($old && scalar(@{$opt{'copy_on_empty'}})) { % foreach my $field (@{$opt{'copy_on_empty'}}) { % $new->set($field, $old->get($field)) % unless scalar($cgi->param($field)); @@ -88,6 +90,11 @@ % % if ( $error ) { % $cgi->param('error', $error); +% if (scalar(@{$opt{'clear_on_error'}})) { +% foreach my $field (@{$opt{'clear_on_error'}}) { +% $cgi->param($field, '') +% } +% } % my $edit_ext = $opt{'edit_ext'} || 'html'; % my $url = $opt{'error_redirect'} || popurl(2)."$table.$edit_ext?"; % print $cgi->redirect($url. $cgi->query_string );