Commit f6e2d728 authored by Frédéric Buclin's avatar Frédéric Buclin

Bug 686904: Removing an invalid URL from the See Also field crashes Bugzilla

r=timello a=LpSolit
parent f856ddac
......@@ -2886,7 +2886,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} },
......@@ -3400,7 +3401,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);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment