Summary

Derive the search index bug field from get_bug_list().

Review Request #15307 — Created Sept. 14, 2026 and submitted Sept. 17, 2026, 12:06 p.m.

Information

Owner

david

Repository

Review Board

Branch

release-9.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

This change modifies the bug field for the search index moves from
model_attr to a prepare method, so migrated review requests index
their linked bugs rather than the frozen bugs_closed column.
extra_data is added to the indexed fields since get_bug_list() reads
the migration marker from it.

A reindex will be needed on upgrade. This should be noted in the
release notes.

Testing Done

Ran unit tests.

Commits

Summary	ID
Derive the search index bug field from get_bug_list(). This change modifies the bug field for the search index moves from `model_attr` to a `prepare` method, so migrated review requests index their linked bugs rather than the frozen `bugs_closed` column. `extra_data` is added to the indexed fields since `get_bug_list()` reads the migration marker from it. A reindex will be needed on upgrade. This should be noted in the release notes. Testing Done: Ran unit tests.	qukvmvyqkwtwwtvkkokplzwpxwnqxyor

Summary

Derive the search index bug field from get_bug_list().

This change modifies the bug field for the search index moves from `model_attr` to a `prepare` method, so migrated review requests index their linked bugs rather than the frozen `bugs_closed` column. `extra_data` is added to the indexed fields since `get_bug_list()` reads the migration marker from it. A reindex will be needed on upgrade. This should be noted in the release notes. Testing Done: Ran unit tests.

qukvmvyqkwtwwtvkkokplzwpxwnqxyor

Issues

Description	From	Last Updated
Shouldn't we derive the search index bug field from review_request.bugs so that it includes all bugs (from the default tracker, …	maubin	Sept. 17, 2026, 11:10 a.m.

flake8 passed.

JSHint passed.

The issue has been dropped. Show all issues

Shouldn't we derive the search index bug field from review_request.bugs so that it includes all bugs (from the default tracker, the sentinel, and any configured trackers)? Doesn't get_bug_list() just return the legacy bugs from the default tracker and the sentinel?

Ship it!

Nevermind this makes sense with the new TrackedBugsField which I missed when reviewing the stack of changes.

Status:: Completed
Change Summary:: Pushed to release-9.x (7394d62)