diff options
author | David Lawrence <dlawrence@mozilla.com> | 2011-09-30 23:50:01 +0200 |
---|---|---|
committer | David Lawrence <dlawrence@mozilla.com> | 2011-09-30 23:50:01 +0200 |
commit | 77f1cffd26e4e306b46558142a432acdbb67b9f1 (patch) | |
tree | e79f3e29d233c0afbc8c7928b408b8a6119847dc /Bugzilla/Bug.pm | |
parent | f6e81df17967ab414235bcc12d3432df69192f1a (diff) | |
parent | f517974618ba835014e9d2a3565109af469f5d34 (diff) | |
download | bugzilla-77f1cffd26e4e306b46558142a432acdbb67b9f1.tar.gz bugzilla-77f1cffd26e4e306b46558142a432acdbb67b9f1.tar.xz |
merged with bugzilla/4.2
Diffstat (limited to 'Bugzilla/Bug.pm')
-rw-r--r-- | Bugzilla/Bug.pm | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 8beecdcd2..175bcd050 100644 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -1130,9 +1130,7 @@ sub remove_from_db { # The bugs_fulltext table doesn't support transactions. $dbh->do("DELETE FROM bugs_fulltext WHERE bug_id = ?", undef, $bug_id); - # Now this bug no longer exists - $self->DESTROY; - return $self; + undef $self; } ##################################################################### @@ -2891,7 +2889,8 @@ sub remove_see_also { # Since we remove also the url from the referenced bug, # we need to notify changes for that bug too. $removed_bug_url = $removed_bug_url->[0]; - if ($removed_bug_url->isa('Bugzilla::BugUrl::Bugzilla::Local') + if ($removed_bug_url + and $removed_bug_url->isa('Bugzilla::BugUrl::Bugzilla::Local') and defined $removed_bug_url->ref_bug_url) { push @{ $self->{see_also_changes} }, @@ -3385,7 +3384,7 @@ sub reporter { sub see_also { my ($self) = @_; return [] if $self->{'error'}; - if (!defined $self->{see_also}) { + if (!exists $self->{see_also}) { my $ids = Bugzilla->dbh->selectcol_arrayref( 'SELECT id FROM bug_see_also WHERE bug_id = ?', undef, $self->id); |