summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormkanat%bugzilla.org <>2008-12-16 22:16:25 +0100
committermkanat%bugzilla.org <>2008-12-16 22:16:25 +0100
commitbfa4c5aede92951fa43d3bee1b15b9773e3d20f2 (patch)
tree6c5784a55dcf5b074192219fad94581fd35fa702
parent82262256f09894f934af6a40a53de80c376e6df2 (diff)
downloadbugzilla-bfa4c5aede92951fa43d3bee1b15b9773e3d20f2.tar.gz
bugzilla-bfa4c5aede92951fa43d3bee1b15b9773e3d20f2.tar.xz
Bug 467701: USAGE_MODE_CMDLINE should be the default if not i_am_cgi()
Patch By Max Kanat-Alexander <mkanat@bugzilla.org> r=LpSolit, a=LpSolit
-rw-r--r--Bugzilla.pm2
-rwxr-xr-xchecksetup.pl1
-rwxr-xr-xcollectstats.pl3
-rwxr-xr-xcontrib/bzdbcopy.pl2
-rwxr-xr-ximportxml.pl4
-rw-r--r--sanitycheck.pl2
6 files changed, 1 insertions, 13 deletions
diff --git a/Bugzilla.pm b/Bugzilla.pm
index e80f184ed..354d05148 100644
--- a/Bugzilla.pm
+++ b/Bugzilla.pm
@@ -372,7 +372,7 @@ sub usage_mode {
$class->request_cache->{usage_mode} = $newval;
}
return $class->request_cache->{usage_mode}
- || Bugzilla::Constants::USAGE_MODE_BROWSER;
+ || (i_am_cgi()? USAGE_MODE_BROWSER : USAGE_MODE_CMDLINE);
}
sub installation_mode {
diff --git a/checksetup.pl b/checksetup.pl
index d7a787f48..74b21dfda 100755
--- a/checksetup.pl
+++ b/checksetup.pl
@@ -115,7 +115,6 @@ require Bugzilla::Template;
require Bugzilla::Field;
require Bugzilla::Install;
-Bugzilla->usage_mode(USAGE_MODE_CMDLINE);
Bugzilla->installation_mode(INSTALLATION_MODE_NON_INTERACTIVE) if $answers_file;
Bugzilla->installation_answers($answers_file);
diff --git a/collectstats.pl b/collectstats.pl
index 08f1c1131..761c648c8 100755
--- a/collectstats.pl
+++ b/collectstats.pl
@@ -59,9 +59,6 @@ if (chdir("graphs")) {
chdir($cwd);
}
-# This is a pure command line script.
-Bugzilla->usage_mode(USAGE_MODE_CMDLINE);
-
my $dbh = Bugzilla->switch_to_shadow_db();
diff --git a/contrib/bzdbcopy.pl b/contrib/bzdbcopy.pl
index eade75683..b4f1fffd2 100755
--- a/contrib/bzdbcopy.pl
+++ b/contrib/bzdbcopy.pl
@@ -48,8 +48,6 @@ use constant TARGET_DB_HOST => 'localhost';
# MAIN SCRIPT
#####################################################################
-Bugzilla->usage_mode(USAGE_MODE_CMDLINE);
-
print "Connecting to the '" . SOURCE_DB_NAME . "' source database on "
. SOURCE_DB_TYPE . "...\n";
my $source_db = Bugzilla::DB::_connect(SOURCE_DB_TYPE, SOURCE_DB_HOST,
diff --git a/importxml.pl b/importxml.pl
index d2ff9cc67..fd8c27723 100755
--- a/importxml.pl
+++ b/importxml.pl
@@ -92,10 +92,6 @@ use Getopt::Long;
use Pod::Usage;
use XML::Twig;
-# We want to capture errors and handle them here rather than have the Template
-# code barf all over the place.
-Bugzilla->usage_mode(Bugzilla::Constants::USAGE_MODE_CMDLINE);
-
my $debug = 0;
my $mail = '';
my $attach_path = '';
diff --git a/sanitycheck.pl b/sanitycheck.pl
index 2ef0eea7d..e0128b58a 100644
--- a/sanitycheck.pl
+++ b/sanitycheck.pl
@@ -42,8 +42,6 @@ my $result = GetOptions('verbose' => \$verbose,
pod2usage({-verbose => 1, -exitval => 1}) if $help;
-Bugzilla->usage_mode(USAGE_MODE_CMDLINE);
-
# Be sure a login name if given.
$login || ThrowUserError('invalid_username');