Changeset 283 for trunk/dns.cgi

Timestamp:

03/22/12 16:12:32 (12 years ago)

Author:

Kris Deugau

Message:

/trunk

Convert zone status change logging to DNSDB-internal. See #35
Add a global for holding general result messages for subs that
return booleans. This allows us to easily display the same text
as was added to the log.
Fix docucomment on mailNotify(); it (still!) referred to its
copy-paste parent in the IP Database.

File:

• trunk/dns.cgi (modified) (2 diffs)

trunk/dns.cgi

@@ -330 +330 @@
     if ($flag && ($permissions{admin} || $permissions{domain_edit})) {
       my $stat = zoneStatus($dbh,$webvar{id},'n',$webvar{zonestatus});
-##fixme  switch to more consise "Enabled <domain"/"Disabled <domain>" as with users?
-      logaction($webvar{id}, $session->param("username"),
-        parentID($dbh, (id => $webvar{id}, type => 'domain', revrec => $webvar{revrec})),
-        "Changed ".domainName($dbh, $webvar{id})." state to ".($stat ? 'active' : 'inactive'));
-      $page->param(resultmsg => "Changed ".domainName($dbh, $webvar{id})." state to ".
-        ($stat ? 'active' : 'inactive'));
+      $page->param(resultmsg => $DNSDB::resultstr);
     } else {
       $page->param(errmsg => "You are not permitted to view or change the requested domain");
@@ -450 +445 @@
     if ($flag && ($permissions{admin} || $permissions{domain_edit})) {
       my $stat = zoneStatus($dbh,$webvar{id},'y',$webvar{zonestatus});
-##fixme  switch to more consise "Enabled <domain"/"Disabled <domain>" as with users?
-      logaction($webvar{id}, $session->param("username"),
-        parentID($dbh, (id => $webvar{id}, type => 'revzone', revrec => $webvar{revrec})),
-        "Changed ".revName($dbh, $webvar{id})." state to ".($stat ? 'active' : 'inactive'));
-      $page->param(resultmsg => "Changed ".revName($dbh, $webvar{id})." state to ".
-        ($stat ? 'active' : 'inactive'));
+      $page->param(resultmsg => $DNSDB::resultstr);
     } else {
       $page->param(errmsg => "You are not permitted to view or change the requested reverse zone");