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.uzkuras
#2 facets-2656220-2-processor-order-save-8.x.patch793 bytesevaldas.uzkuras

Comments

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

evaldas.uzkuras’s picture

evaldas.uzkuras
Primary language Lithuanian
Location Kaunas, LT
commented
StatusFileSize
new facets-2656220-2-processor-order-save-8.x.patch793 bytes

Patch for quick solution.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
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.uzkuras’s picture

evaldas.uzkuras
Primary language Lithuanian
Location Kaunas, LT
commented
StatusFileSize
new 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, 🇧🇪
commented
StatusFileSize
new facet_s_processors-2656220-5--test-only.patch1.82 KB
new facet_s_processors-2656220-5.patch3.99 KB

Added a test. (test-only is interdiff)

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
StatusFileSize
new 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, 🇧🇪
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, 🇧🇪
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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