Changeset 139 for trunk/dns.cgi


Ignore:
Timestamp:
10/07/11 16:01:55 (13 years ago)
Author:
Kris Deugau
Message:

/trunk

Reduce the abuse of "actions" as a form input name; make the
domain and user list status changes use their own custom input
names to reduce the mixups when checking "global" actions

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/dns.cgi

    r138 r139  
    228228# hmm.  seeing problems in some possibly-not-so-corner cases.
    229229# this currently only handles "domain on", "domain off"
    230   if (defined($webvar{action})) {
    231     my $stat = domStatus($dbh,$webvar{id},$webvar{action});
     230  if (defined($webvar{domstatus})) {
     231    my $stat = domStatus($dbh,$webvar{id},$webvar{domstatus});
    232232    logaction($webvar{id}, $session->param("username"), parentID($webvar{id}, 'dom', 'group'),
    233233        "Changed ".domainName($dbh, $webvar{id})." state to ".($stat ? 'active' : 'inactive'));
     
    797797} elsif ($webvar{page} eq 'useradmin') {
    798798
    799   if (defined($webvar{action})) {
    800     userStatus($dbh,$webvar{id},$webvar{action});
     799  if (defined($webvar{userstatus})) {
     800    userStatus($dbh,$webvar{id},$webvar{userstatus});
    801801  }
    802802
Note: See TracChangeset for help on using the changeset viewer.