diff options
| author | lpsolit%gmail.com <> | 2009-04-06 22:02:51 +0200 |
|---|---|---|
| committer | lpsolit%gmail.com <> | 2009-04-06 22:02:51 +0200 |
| commit | 1633d9835c4ab45d583ba4b6d8b585d66925a5b2 (patch) | |
| tree | a59da1c5899387fc43a553a41810e2fe4aea1bc4 | |
| parent | 2ee6d9a1b8ebde208a1ff11e97c909925fe87303 (diff) | |
| download | bugzilla-1633d9835c4ab45d583ba4b6d8b585d66925a5b2.tar.gz bugzilla-1633d9835c4ab45d583ba4b6d8b585d66925a5b2.tar.xz | |
Bug 486881: A powerless user can still see a restricted bug when he removes all his roles related to the bug - Patch by Frédéric Buclin <LpSolit@gmail.com> r/a=mkanat
| -rw-r--r-- | Bugzilla/Bug.pm | 4 | ||||
| -rw-r--r-- | Bugzilla/User.pm | 2 |
2 files changed, 6 insertions, 0 deletions
diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 5038275c8..79cf1dd5e 100644 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -876,6 +876,10 @@ sub update { delete $self->{'_old_assigned_to'}; delete $self->{'_old_qa_contact'}; + # Also flush the visible_bugs cache for this bug as the user's + # relationship with this bug may have changed. + delete Bugzilla->user->{_visible_bugs_cache}->{$self->id}; + return $changes; } diff --git a/Bugzilla/User.pm b/Bugzilla/User.pm index 74289fe56..cf4cb5fa6 100644 --- a/Bugzilla/User.pm +++ b/Bugzilla/User.pm @@ -588,6 +588,8 @@ sub visible_bugs { # We only check the visibility of bugs that we haven't # checked yet. + # Bugzilla::Bug->update automatically removes updated bugs + # from the cache to force them to be checked again. my $visible_cache = $self->{_visible_bugs_cache} ||= {}; my @check_ids = grep(!exists $visible_cache->{$_}, @bug_ids); |