#53 field error erases previous values

[Robert Story]

when the error check for a field fails and the page
redisplays, all previous values are erased.

[Robert Story]

Logged In: YES
user_id=76148

doesn't quite work right with merged primaries...

[Robert Story]

Logged In: YES
user_id=76148

to be more precise re: mreged screens.. if screen 1 has an
error, and screen 2 has none, screen 2's values are not
saved. If screen 2 has an error as well, values are saved.

[Robert Story]

Logged In: YES
user_id=76148

Just verified that this problem remains. This small patch
will merge the pickpolicy screen with whatever action was
selected. My test case was to select create new firewall
rule, enter a new policy name, but no priority for the rule.
The empty priority field is flagged, and the new policy name
is erased.

Reducing the priority, as there aren't many merged primaries
in use.

diff -u -r1.180 htmlmanager.pm
--- html/htmlmanager.pm 9 Apr 2003 15:38:11 -0000 1.180
+++ html/htmlmanager.pm 15 Apr 2003 16:32:01 -0000
@@ -663,10 +665,13 @@
sub { my $wiz = shift;
my @todos;
if (exists($primaries{qwparam('action')})) {
- $wiz->add_todos(qwparam('action'));
if (qwparam('action') !~ "^admin") {
- $wiz->add_todos('pickpolicy');
+ $wiz->add_todos(-merge, 'pickpolicy',
+
qwparam('action'));
}
+ else {
+
$wiz->add_todos(qwparam('action'));
+ }
}
return 'OK'
}