diff --git a/mozilla/webtools/bugzilla/Bug.pm b/mozilla/webtools/bugzilla/Bug.pm index 96ff0b8af6a..5ae83d29525 100755 --- a/mozilla/webtools/bugzilla/Bug.pm +++ b/mozilla/webtools/bugzilla/Bug.pm @@ -248,7 +248,7 @@ sub emitXML { $xml .= "\n"; - foreach my $field ("bug_id", "urlbase", "bug_status", "product", + foreach my $field ("bug_id", "bug_status", "product", "priority", "version", "rep_platform", "assigned_to", "delta_ts", "component", "reporter", "target_milestone", "bug_severity", "creation_ts", "qa_contact", "op_sys", "resolution", "bug_file_loc", diff --git a/mozilla/webtools/bugzilla/Bugzilla/Bug.pm b/mozilla/webtools/bugzilla/Bugzilla/Bug.pm index 96ff0b8af6a..5ae83d29525 100644 --- a/mozilla/webtools/bugzilla/Bugzilla/Bug.pm +++ b/mozilla/webtools/bugzilla/Bugzilla/Bug.pm @@ -248,7 +248,7 @@ sub emitXML { $xml .= "\n"; - foreach my $field ("bug_id", "urlbase", "bug_status", "product", + foreach my $field ("bug_id", "bug_status", "product", "priority", "version", "rep_platform", "assigned_to", "delta_ts", "component", "reporter", "target_milestone", "bug_severity", "creation_ts", "qa_contact", "op_sys", "resolution", "bug_file_loc", diff --git a/mozilla/webtools/bugzilla/bugzilla.dtd b/mozilla/webtools/bugzilla/bugzilla.dtd index f9e9c448000..459755ccd85 100644 --- a/mozilla/webtools/bugzilla/bugzilla.dtd +++ b/mozilla/webtools/bugzilla/bugzilla.dtd @@ -5,7 +5,7 @@ maintainer CDATA #REQUIRED exporter CDATA #IMPLIED > - + diff --git a/mozilla/webtools/bugzilla/sanitycheck.cgi b/mozilla/webtools/bugzilla/sanitycheck.cgi index 902557bb8ae..80b88d5fd90 100755 --- a/mozilla/webtools/bugzilla/sanitycheck.cgi +++ b/mozilla/webtools/bugzilla/sanitycheck.cgi @@ -140,6 +140,24 @@ if (exists $::FORM{'rebuildvotecache'}) { print "OK, now running sanity checks.

\n"; +# This one goes first, because if this is wrong, then the below tests +# will probably fail too + +# This isn't extensible. Thats OK; we're not adding any more enum fields +Status("Checking for invalid enumeration values"); +foreach my $field (("bug_severity", "bug_status", "op_sys", + "priority", "rep_platform", "resolution")) { + # undefined enum values in mysql are an empty string which equals 0 + SendSQL("SELECT bug_id FROM bugs WHERE $field=0 ORDER BY bug_id"); + my @invalid; + while (MoreSQLData()) { + push (@invalid, FetchOneColumn()); + } + if (@invalid) { + Alert("Bug(s) found with invalid $field value: ".join(', ',@invalid)); + } +} + CrossCheck("keyworddefs", "id", ["keywords", "keywordid"]);