From: Jonathan Prykop <jonathan@freeside.biz>
Date: Tue, 26 Jul 2016 20:04:54 +0000 (-0500)
Subject: Revert "RT#38217: Send email when logging conditions are met"
X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=commitdiff_plain;h=a1930173f49200333e347b87c01c3edabc1ebb9c

Revert "RT#38217: Send email when logging conditions are met"

This reverts commit 5f563d5ac7e6e1e93cca382baa42ee106f3db5a0.
---

diff --git a/FS/FS/Log.pm b/FS/FS/Log.pm
index 9d6ce6a57..2fd002093 100644
--- a/FS/FS/Log.pm
+++ b/FS/FS/Log.pm
@@ -9,7 +9,7 @@ use vars qw(@STACK @LEVELS);
 
 # override the stringification of @_ with something more sensible.
 BEGIN {
-  @LEVELS = qw(debug info warning error critical);
+  @LEVELS = qw(debug info notice warning error critical alert emergency);
 
   foreach my $l (@LEVELS) {
     my $sub = sub {
diff --git a/FS/FS/Upgrade.pm b/FS/FS/Upgrade.pm
index 3faf47e24..65b83bd5f 100644
--- a/FS/FS/Upgrade.pm
+++ b/FS/FS/Upgrade.pm
@@ -352,9 +352,6 @@ sub upgrade_data {
 
   tie my %hash, 'Tie::IxHash', 
 
-    #remap log levels
-	'log' => [],
-
     #cust_main (remove paycvv from history, locations, cust_payby, etc)
     'cust_main' => [],
 
diff --git a/FS/FS/log.pm b/FS/FS/log.pm
index b8f51cd13..1d4df730a 100644
--- a/FS/FS/log.pm
+++ b/FS/FS/log.pm
@@ -6,7 +6,6 @@ use FS::Record qw( qsearch qsearchs dbdef );
 use FS::UID qw( dbh driver_name );
 use FS::log_context;
 use FS::log_email;
-use FS::upgrade_journal;
 
 =head1 NAME
 
@@ -384,52 +383,6 @@ sub search {
   };
 }
 
-sub _upgrade_data {
-  my ($class, %opts) = @_;
-
-  return if FS::upgrade_journal->is_done('log__remap_levels');
-
-  tie my %levelmap, 'Tie::IxHash', 
-#    0 => 0, #debug
-#    1 => 1, #info
-    2 => 1, #notice -> info
-    3 => 2, #warning
-    4 => 3, #error
-    5 => 4, #critical
-    6 => 4, #alert -> critical
-    7 => 4, #emergency -> critical
-  ;
-
-  # this method should never autocommit
-  # should have been set in upgrade, but just in case...
-  local $FS::UID::AutoCommit = 0;
-
-  # FS::log has no replace method
-  # in practice, only debug/info/warning/error were used,
-  #   so this should only hit warning/error
-  foreach my $old (keys %levelmap) {
-    my $sql = 'UPDATE log SET level=' . dbh->quote($levelmap{$old}) . ' WHERE level=' . dbh->quote($old);
-    warn $sql unless $opts{'quiet'};
-    my $sth = dbh->prepare($sql) or die dbh->errstr;
-    $sth->execute() or die $sth->errstr;
-    $sth->finish();
-  }
-
-  foreach my $log_email (
-    qsearch('log_email',{ 'min_level' => { 'op' => '>=', 'value' => '2' } })
-  ) {
-    $log_email->min_level($levelmap{$log_email->min_level});
-    my $error = $log_email->replace;
-    if ($error) {
-      dbh->rollback;
-      die $error;
-    }
-  }
-
-  FS::upgrade_journal->set_done('log__remap_levels');
-
-}
-
 =back
 
 =head1 BUGS
diff --git a/httemplate/edit/log_email.html b/httemplate/edit/log_email.html
index 4b7d09fb4..0c98046d3 100644
--- a/httemplate/edit/log_email.html
+++ b/httemplate/edit/log_email.html
@@ -16,8 +16,8 @@
                             },
                             { 'field' => 'min_level',
                               'type'  => 'select',
-                              'options' => [ 0..4 ],
-                              'labels' => { map {$_ => $FS::Log::LEVELS[$_]} 0..4 },
+                              'options' => [ 0..7 ],
+                              'labels' => { map {$_ => $FS::Log::LEVELS[$_]} 0..7 },
                               'curr_value' => scalar($cgi->param('min_level')),
                             },
                             'to_addr',
diff --git a/httemplate/search/log.html b/httemplate/search/log.html
index cefa399a2..111200f55 100644
--- a/httemplate/search/log.html
+++ b/httemplate/search/log.html
@@ -81,15 +81,15 @@ a:visited {text-decoration: none}
   <TD>Level
     <& /elements/select.html,
       field => 'min_level',
-      options => [ 0..4 ],
-      labels => { map {$_ => $FS::Log::LEVELS[$_]} 0..4 },
+      options => [ 0..7 ],
+      labels => { map {$_ => $FS::Log::LEVELS[$_]} 0..7 },
       curr_value => $cgi->param('min_level'),
     &>
      to
     <& /elements/select.html,
       field => 'max_level',
-      options => [ 0..4 ],
-      labels => { map {$_ => $FS::Log::LEVELS[$_]} 0..4 },
+      options => [ 0..7 ],
+      labels => { map {$_ => $FS::Log::LEVELS[$_]} 0..7 },
       curr_value => $cgi->param('max_level'),
     &>
   </TD>
@@ -192,11 +192,14 @@ my $object_link_sub = sub {
 };
 
 my @colors = (
-  '404040', #debug, gray
-  '000000', #info, black
-  '0000aa', #warning, blue
-  'aa0066', #error, purple
-  'ff0000', #critical, red
+  '404040', #debug
+  '0000aa', #info
+  '00aa00', #notice
+  'aa0066', #warning
+  '000000', #error
+  'aa0000', #critical
+  'ff0000', #alert
+  'ff0000', #emergency
 );
 
 my $color_sub = sub { $colors[ $_[0]->level ]; };