Fix configuring search when search settings are empty.

Review Request #11427 — Created Feb. 2, 2021 and submitted — Latest diff uploaded

Information

Review Board
release-4.0.x

Reviewers

The recent work to fix search settings and bring compatibility to modern
Haystack ended up regressing configuring a search engine if there
weren't already stored settings. The old code conditionally ensured,
rather than assumed, there would be a key in settings for the search
backend, but the new code regressed that.

This change updates the search configuration setting code to always set
a default entry if it doesn't exist, and to populate that.

Unit tests were added to ensure this doesn't regress again.

Unit tests pass.

Verified the original failure case with the unit tests, without the
fix.

Commits

Files

    Loading...