diff options
author | ivan <ivan> | 2003-07-04 00:51:29 +0000 |
---|---|---|
committer | ivan <ivan> | 2003-07-04 00:51:29 +0000 |
commit | af213c494294f73750b6b5b07ca828782ff3a9e4 (patch) | |
tree | c647965dfee1436b1e91665da42821601992666c | |
parent | d3eefd3591dbd54a1626cca247c7a34b57d89f96 (diff) |
optimize javascript to handle large numbers of POPs
-rw-r--r-- | FS/FS/svc_acct_pop.pm | 5 | ||||
-rwxr-xr-x | fs_signup/FS-SignupClient/cgi/signup.cgi | 8 |
2 files changed, 5 insertions, 8 deletions
diff --git a/FS/FS/svc_acct_pop.pm b/FS/FS/svc_acct_pop.pm index 3c9ea0130..404816b47 100644 --- a/FS/FS/svc_acct_pop.pm +++ b/FS/FS/svc_acct_pop.pm @@ -142,8 +142,7 @@ sub popselector { function popstate_changed(what) { state = what.options[what.selectedIndex].text; - for (var i = what.form.popnum.length;i > 0;i--) - what.form.popnum.options[i] = null; + what.form.popnum.options.length = 0 what.form.popnum.options[0] = new Option("", "", false, true); END @@ -182,7 +181,7 @@ END =head1 VERSION -$Id: svc_acct_pop.pm,v 1.7 2002-04-10 13:42:48 ivan Exp $ +$Id: svc_acct_pop.pm,v 1.8 2003-07-04 00:51:29 ivan Exp $ =head1 BUGS diff --git a/fs_signup/FS-SignupClient/cgi/signup.cgi b/fs_signup/FS-SignupClient/cgi/signup.cgi index de142a1c4..09bdd43fa 100755 --- a/fs_signup/FS-SignupClient/cgi/signup.cgi +++ b/fs_signup/FS-SignupClient/cgi/signup.cgi @@ -1,6 +1,6 @@ #!/usr/bin/perl -Tw # -# $Id: signup.cgi,v 1.37 2003-06-24 15:34:45 ivan Exp $ +# $Id: signup.cgi,v 1.38 2003-07-04 00:51:29 ivan Exp $ use strict; use vars qw( @payby $cgi $locales $packages $pops $init_data $error @@ -401,8 +401,7 @@ sub popselector { function acstate_changed(what) { state = what.options[what.selectedIndex].text; - for (var i = what.form.popac.length;i > 0;i--) - what.form.popac.options[i] = null; + what.form.popac.options.length = 0 what.form.popac.options[0] = new Option("Area code", "-1", false, true); END @@ -422,8 +421,7 @@ END $text .= <<END; function popac_changed(what) { ac = what.options[what.selectedIndex].text; - for (var i = what.form.popnum.length;i > 0;i--) - what.form.popnum.options[i] = null; + what.form.popnum.options.length = 0; what.form.popnum.options[0] = new Option("City", "-1", false, true); END |