Then you rearrange the order of processors and hit save it returns to previous state.
New order is not saved.

CommentFileSizeAuthor
#6 facet_s_processors-2656220-6.patch1.82 KBborisson_
#5 facet_s_processors-2656220-5.patch3.99 KBborisson_
#5 facet_s_processors-2656220-5--test-only.patch1.82 KBborisson_
#4 facets-2656220-4-processor-order-save-8.x.patch2.16 KBEvaldas Užkuras
#2 facets-2656220-2-processor-order-save-8.x.patch793 bytesEvaldas Užkuras
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Evaldas Užkuras created an issue. See original summary.

Evaldas Užkuras’s picture

Evaldas Užkuras
Primary language Lithuanian
Location Kaunas, LT
CreditAttribution: Evaldas Užkuras at WEB Partners commented
FileSize
facets-2656220-2-processor-order-save-8.x.patch793 bytes

Patch for quick solution.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ for Acquia commented
Status: Active » Needs review

Setting to NR for the testbot.

I'm not sure if we can write an integration test for this but I can give it a try.

Evaldas Užkuras’s picture

Evaldas Užkuras
Primary language Lithuanian
Location Kaunas, LT
CreditAttribution: Evaldas Užkuras at WEB Partners commented
FileSize
facets-2656220-4-processor-order-save-8.x.patch2.16 KB

Adding new patch.
Processor weight form fields' values was not set correctly - default values instead of new ones.

For making test, you could try to check value of hidden weight field.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ for Acquia commented
FileSize
facet_s_processors-2656220-5--test-only.patch1.82 KB
facet_s_processors-2656220-5.patch3.99 KB

Added a test. (test-only is interdiff)

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ for Acquia commented
FileSize
facet_s_processors-2656220-6.patch1.82 KB

Re-uploading --test-only to trigger testbot.

Status: Needs review » Needs work

The last submitted patch, 6: facet_s_processors-2656220-6.patch, failed testing.

The last submitted patch, 6: facet_s_processors-2656220-6.patch, failed testing.

borisson_’s picture

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

Committed and pushed, thanks for the issue and patch.

  • borisson_ committed 5c49f5a on 8.x-1.x 
    Issue #2656220 by borisson_, Evaldas Užkuras: Facet's processors orders...
borisson_’s picture

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

Status: Fixed » Closed (fixed)

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