[Bps-public-commit] RT-BugTracker branch, rt4, updated. 8dcc3000e6d078e90c5022b2cd7dc30662d8f874

Thomas Sibley trs at bestpractical.com
Thu Apr 4 20:25:59 EDT 2013


The branch, rt4 has been updated
       via  8dcc3000e6d078e90c5022b2cd7dc30662d8f874 (commit)
      from  8816187b4bf40357e2ed9f1d6291cb8e26034f8a (commit)

Summary of changes:
 lib/RT/BugTracker.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

- Log -----------------------------------------------------------------
commit 8dcc3000e6d078e90c5022b2cd7dc30662d8f874
Author: Thomas Sibley <trs at bestpractical.com>
Date:   Thu Apr 4 17:24:37 2013 -0700

    Accurately log our refusal (not failure) to set non-HTTP/HTTPs bugtracker URIs
    
    Since the behaviour is expected and there's nothing for the RT admin to
    do about it, it is really an informational message not an error.

diff --git a/lib/RT/BugTracker.pm b/lib/RT/BugTracker.pm
index b977f06..3643e2e 100644
--- a/lib/RT/BugTracker.pm
+++ b/lib/RT/BugTracker.pm
@@ -125,10 +125,10 @@ sub SetDistributionBugtracker {
             }
 
             else {
-                my $error_msg = "Failed to set external bugtracker website";
+                my $error_msg = "Refused to set external bugtracker website";
                 $error_msg   .= " on distribution (" . $self->Name() .  ").";
                 $error_msg   .= " Unsupported scheme (" . $uri->scheme() . ").";
-                $RT::Logger->error($error_msg);
+                $RT::Logger->info($error_msg);
             }
         }
         else {

-----------------------------------------------------------------------



More information about the Bps-public-commit mailing list