As of #2656668: Facet's results sorting with multiple sorts., sort processors are separated in the build phase and fired together, AFTER every other processor in the build phase.

We should move sorts into a separate stage "Sort", which fires right after the build stage, to make this more explicit, as ordering sorts and non-sort processors will not behave as a user expects at this moment.

CommentFileSizeAuthor
#7 create_a_new_processor-2724171-7.patch9.58 KBborisson_
#7 interdiff.txt617 bytesborisson_
#5 create_a_new_processor-2724171-5.patch9.19 KBStryKaizer
#5 interdiff.txt2.21 KBStryKaizer
#4 create_a_new_processor-2724171-4.patch8.82 KBStryKaizer
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

StryKaizer created an issue. See original summary.

StryKaizer’s picture

StryKaizer
Primary language Dutch
Location Belgium
CreditAttribution: StryKaizer as a volunteer and commented
Issue summary: View changes
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ commented
Title: Create new stage 'Sort' » Create a new processor stage 'Sort'
Issue summary: View changes
StryKaizer’s picture

StryKaizer
Primary language Dutch
Location Belgium
CreditAttribution: StryKaizer as a volunteer and commented
Status: Active » Needs review
FileSize
create_a_new_processor-2724171-4.patch8.82 KB
StryKaizer’s picture

StryKaizer
Primary language Dutch
Location Belgium
CreditAttribution: StryKaizer as a volunteer and commented
FileSize
interdiff.txt2.21 KB
create_a_new_processor-2724171-5.patch9.19 KB

Cleanup manager, do not create new instances for processors which already exist in getProcessorByStage

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ commented
Status: Needs review » Reviewed & tested by the community
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ commented
FileSize
interdiff.txt617 bytes
create_a_new_processor-2724171-7.patch9.58 KB

  • borisson_ committed ae38a8e on 8.x-1.x authored by StryKaizer 
    Issue #2724171 by StryKaizer, borisson_: Create a new processor stage '...
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ commented
Status: Reviewed & tested by the community » Fixed

Awesome, thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.