diff --git a/useradmin/edit_group.cgi b/useradmin/edit_group.cgi index 257f3dc90..1db61e265 100755 --- a/useradmin/edit_group.cgi +++ b/useradmin/edit_group.cgi @@ -100,8 +100,8 @@ if ($n ne "") { @uprilinks = ( ); foreach $u (@upri) { if (&can_edit_user(\%access, $u)) { - push(@uprilinks, "".&html_escape($u->{'user'}).""); + push(@uprilinks, &ui_link("edit_user.cgi?". + "user=$u->{'user'}", &html_escape($u->{'user'}) ) ); } else { push(@uprilinks, $u->{'user'}); diff --git a/useradmin/index.cgi b/useradmin/index.cgi index 041ddfe97..d62681d03 100755 --- a/useradmin/index.cgi +++ b/useradmin/index.cgi @@ -221,16 +221,15 @@ if ($access{'ucreate'}) { } else { $cancreate = 1; } if ($cancreate) { - push(@rv, "". - "$text{'index_createuser'}"); + push(@rv, &ui_link("edit_user.cgi", $text{'index_createuser'}) ); } else { push(@rv, $text{'index_nomoreusers'}); } } -push(@rv, "$text{'index_batch'}") +push(@rv, &ui_link("batch_form.cgi", $text{'index_batch'}) ) if ($access{'batch'}); -push(@rv, "$text{'index_export'}") +push(@rv, &ui_link("export_form.cgi", $text{'index_export'}) ) if ($access{'export'}); return @rv; } @@ -259,15 +258,15 @@ if ($access{'gcreate'} == 1) { } else { $cancreate = 1; } if ($cancreate) { - push(@rv, "$text{'index_creategroup'}"); + push(@rv, &ui_link("edit_group.cgi", $text{'index_creategroup'}) ); } else { push(@rv, $text{'index_nomoregroups'}); } } -push(@rv, "$text{'index_batch'}") +push(@rv, &ui_link("gbatch_form.cgi", $text{'index_batch'}) ) if ($access{'batch'}); -push(@rv, "$text{'index_export'}") +push(@rv, &ui_link("gexport_form.cgi", $text{'index_export'}) ) if ($access{'export'}); return @rv; } diff --git a/useradmin/list_who.cgi b/useradmin/list_who.cgi index b7faf84fb..6ebce3347 100755 --- a/useradmin/list_who.cgi +++ b/useradmin/list_who.cgi @@ -17,8 +17,8 @@ foreach $w (@whos) { $tm = defined(&mailboxes::parse_mail_date) ? &mailboxes::parse_mail_date($w->{'when'}) : undef; push(@table, [ - "". - &html_escape($w->{'user'})."", + &ui_link("list_logins.cgi?username=".&urlize($w->{'user'}), + &html_escape($w->{'user'}) ), &html_escape($w->{'tty'}), &html_escape($tm ? &make_date($tm) : $w->{'when'}), $w->{'from'} ? &html_escape($w->{'from'}) diff --git a/useradmin/my_group_chooser.cgi b/useradmin/my_group_chooser.cgi index 892162216..b7913bc0b 100755 --- a/useradmin/my_group_chooser.cgi +++ b/useradmin/my_group_chooser.cgi @@ -68,7 +68,9 @@ if ($in{'multi'}) { foreach $u (&get_groups_list()) { if ($in{'group'} eq $u->[0]) { print "\n"; } else { print "\n"; } - print "$u->[0]\n"; + print ""; + print &ui_link("#", $u->[0], undef, "onClick='return addgroup(\"$u->[0]\", \"$u->[3]\");'"); + print "\n"; print "$u->[3] \n"; $cnt++; } @@ -155,7 +157,9 @@ else { foreach $u (&get_groups_list()) { if ($in{'group'} eq $u->[0]) { print "\n"; } else { print "\n"; } - print "".&html_escape($u->[0])."\n"; + print ""; + print &ui_link("#", &html_escape($u->[0]), undef, "onClick='return select(\"$u->[0]\");'"); + print "\n"; print "".&html_escape($u->[3])." \n"; $cnt++; } diff --git a/useradmin/my_user_chooser.cgi b/useradmin/my_user_chooser.cgi index 830b858ee..d76e62d2b 100755 --- a/useradmin/my_user_chooser.cgi +++ b/useradmin/my_user_chooser.cgi @@ -61,7 +61,9 @@ if ($in{'multi'}) { if ($in{'user'} eq $u->[0]) { print "\n"; } else { print "\n"; } $u->[6] =~ s/'/'/g; - print "$u->[0]\n"; + print ""; + print &ui_link("#", $u->[0], undef, "onClick='return adduser(\"$u->[0]\", \"$u->[6]\");'"); + print "\n"; print "$u->[6] \n"; $cnt++; } @@ -148,7 +150,9 @@ else { foreach $u (&get_users_list()) { if ($in{'user'} eq $u->[0]) { print "\n"; } else { print "\n"; } - print "$u->[0]\n"; + print ""; + print &ui_link("#", $u->[0], undef, "onClick='return select(\"$u->[0]\");'"); + print "\n"; print "$u->[6] \n"; $cnt++; } diff --git a/useradmin/user-lib.pl b/useradmin/user-lib.pl index 06294373c..4ef717c44 100755 --- a/useradmin/user-lib.pl +++ b/useradmin/user-lib.pl @@ -2520,12 +2520,10 @@ if ($_[0]->{'noedit'}) { return $dis; } elsif ($_[0]->{'dn'}) { - return "". - "$dis"; + return &ui_link("edit_user.cgi?dn=".&urlize($_[0]->{'dn'}), $dis); } else { - return "". - "$dis"; + return &ui_link("edit_user.cgi?user=".&urlize($_[0]->{'user'}), $dis); } } @@ -2540,12 +2538,10 @@ if ($_[0]->{'noedit'}) { return &html_escape($_[0]->{'group'}); } elsif ($_[0]->{'dn'}) { - return "". - &html_escape($_[0]->{'group'}).""; + return &ui_link("edit_group.cgi?dn=".&urlize($_[0]->{'dn'}), &html_escape($_[0]->{'group'}) ); } else { - return "". - &html_escape($_[0]->{'group'}).""; + return &ui_link("edit_group.cgi?group=".&urlize($_[0]->{'group'}), &html_escape($_[0]->{'group'}) ); } }