Currently Special:Search does multiple kinds of searches, and each one adds another level of complexity to the class. Before this can be done we will need to review what the default (sql based) search engines do, and if it is reasonable to use SearchEngine::searchText() for all cases. If it is rip out the support and simplify handling down to the single search type.
Description
Description
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | EBernhardson | T150217 [Epic] De-Spaghettify SpecialSearch class | |||
| Declined | None | T150392 Consider removing usage of SearchEngine::searchTitle from Special:Search |
Event Timeline
Comment Actions
After doing a variety of other cleanups, the extra complexity of this is now much less. There is probably more harm in trying to make sure the various other search engine implementations behave in a particular manner than there is maintaining this code.