summaryrefslogtreecommitdiffstats
path: root/Bugzilla
diff options
context:
space:
mode:
authormkanat%bugzilla.org <>2006-08-23 07:30:27 +0200
committermkanat%bugzilla.org <>2006-08-23 07:30:27 +0200
commit2de6abfda1e684d218b8d9b251a1153ce30f89e2 (patch)
treeeba27d9ed2512908a2f136a893846aa76b044f5e /Bugzilla
parent579565465390391b24027475bbadcbcf3a230111 (diff)
downloadbugzilla-2de6abfda1e684d218b8d9b251a1153ce30f89e2.tar.gz
bugzilla-2de6abfda1e684d218b8d9b251a1153ce30f89e2.tar.xz
Bug 349558: Move validations of global enum fields from post_bug into Bugzilla::Bug
Patch By Max Kanat-Alexander <mkanat@bugzilla.org> r=bkor, a=justdave
Diffstat (limited to 'Bugzilla')
-rwxr-xr-xBugzilla/Bug.pm32
1 files changed, 32 insertions, 0 deletions
diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index 9c9e2828d..406e70aca 100755
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -262,6 +262,13 @@ sub _check_bug_file_loc {
return $url;
}
+sub _check_bug_severity {
+ my ($severity) = @_;
+ $severity = trim($severity);
+ check_field('bug_severity', $severity);
+ return $severity;
+}
+
sub _check_bug_status {
my ($status, $product) = @_;
my $user = Bugzilla->user;
@@ -363,6 +370,31 @@ sub _check_product {
return $obj;
}
+sub _check_op_sys {
+ my ($op_sys) = @_;
+ $op_sys = trim($op_sys);
+ check_field('op_sys', $op_sys);
+ return $op_sys;
+}
+
+sub _check_priority {
+ my ($priority) = @_;
+ if (!Bugzilla->params->{'letsubmitterchoosepriority'}) {
+ $priority = Bugzilla->params->{'defaultpriority'};
+ }
+ $priority = trim($priority);
+ check_field('priority', $priority);
+
+ return $priority;
+}
+
+sub _check_rep_platform {
+ my ($platform) = @_;
+ $platform = trim($platform);
+ check_field('rep_platform', $platform);
+ return $platform;
+}
+
sub _check_short_desc {
my ($short_desc) = @_;
# Set the parameter to itself, but cleaned up