Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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!