In #2656668: Facet's results sorting with multiple sorts., @StryKaizer noticed that Facet::getProcessorsByStage doesn't return the correct processor configuration.
The real issue was that we initiate another instance, while we should use the instances as returned by getProcessorsByStage. This will be fixxed in #2724171: Create a new processor stage 'Sort'.
This issue refactors the getProcessorsByStage implementation to reuse getProcessors do reduce duplicate code.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | fix-2722267-4.patch | 1.08 KB | strykaizer |
Comments
Comment #2
borisson_This is a bug, not a task.
Comment #3
strykaizerIt looks like getProcessorsByStage does return the correct configuration, but on the next line we created a new instance, which is not required, as getProcessorsByStage already returns instances.
Refactored code a bit and fixxed this.
This patch will need a reroll once #2724171: Create a new processor stage 'Sort' is in
Comment #4
strykaizerDecided to fix the actual issue in #2724171: Create a new processor stage 'Sort' instead, this patch contains only the refactor in Facet.php to re-use loadProcessors now, and does no longer needs to be rerolled depending on #2724171
Comment #5
borisson_Comment #7
borisson_Thanks!