Patch for bug 287325: we should use scalar instead of length for finding out how...

diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index 22f62f1864f1ce5ee5c961739e01cd5f16af0728..290c4e7ca82d8528e0a10ddde9be128dc596633d 100755 (executable)
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -128,7 +128,7 @@ sub initBug  {
   $self->{'who'} = new Bugzilla::User($user_id);
 
     my $custom_fields = "";
-    if (length(Bugzilla->custom_field_names) > 0) {
+    if (scalar(Bugzilla->custom_field_names) > 0) {
         $custom_fields = ", " . join(", ", Bugzilla->custom_field_names);
     }
 
@@ -742,7 +742,6 @@ sub AppendComment {
     $dbh->do("UPDATE bugs SET delta_ts = ? WHERE bug_id = ?",
              undef, $timestamp, $bugid);
 }
-\r
 # This method is private and is not to be used outside of the Bug class.
 sub EmitDependList {
     my ($myfield, $targetfield, $bug_id) = (@_);