The per filter settings (admin/config/search/search-api/index/[filtername]/filters),
coming from the FieldsProcessorPluginBase settings form (Fields to run on),
affect always all filters, instead of only the desired one.

CommentFileSizeAuthor
#7 2257113-07.search_api.processor-fields.patch1.41 KBekes

Comments

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

Could you please elaborate?

ytsurk’s picture

ytsurk
Location Switzerland
commented
Issue summary: View changes
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Issue summary: View changes

I still don't know what you mean. I just tested this, and everything seems to work fine. What steps are you executing exactly, what do you expect to happen and what does?

ytsurk’s picture

ytsurk
Location Switzerland
commented

the fields to run on property (if existing for the processor - fe. Transliteration & Ignore case) share this property.
both have it in their settings tab - but a change always affects both - clearer now ?

ekes’s picture

ekes commented

Having just done a clean install (with at least) Ignore case & HTML filter on indexes I can't save the settings for which fields they apply to at all. Other settings for the filter are saved.

ekes’s picture

ekes commented
Assigned: Unassigned » ekes
ekes’s picture

ekes commented
StatusFileSize
new 2257113-07.search_api.processor-fields.patch1.41 KB

The fields a processor/filter operate on are stored with the index entity. So are specific to that entity.

However the index was now 'field' => 0; #disabled 'field' => 1; #enabled. The form was expecting only 'field' where enabled.

If this wants tests they are tests of the form?

  • Commit 4538e24 on 2257113-Index-Processors-Fields by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • Commit 4538e24 on master, 2257113-Index-Processors-Fields by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.
ekes’s picture

ekes commented
Status: Active » Needs review

Pushed to master.

  • Commit 4538e24 on master, 2230925-server-tasks, 2257113-Index-Processors-Fields by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • Commit 4538e24 on master, 2230925-server-tasks, 2257113-Index-Processors-Fields, transliteration by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • Commit 4538e24 on master, 2230925-server-tasks, 2257113-Index-Processors-Fields, transliteration, ignore-case-test-fix by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • Commit 4538e24 on master, 2230925-server-tasks, 2257113-Index-Processors-Fields, transliteration, ignore-case-test-fix, 2281233-fix-stopwords by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • Commit 4538e24 on master, 2230925-server-tasks, 2257113-Index-Processors-Fields, transliteration, ignore-case-test-fix, 2281233-fix-stopwords, ignorecase-nick, renamefields by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • Commit 4538e24 on master, 2230925-server-tasks, 2257113-Index-Processors-Fields, transliteration, ignore-case-test-fix, 2281233-fix-stopwords, ignorecase-nick, renamefields by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • Commit 4538e24 on master, transliteration, ignore-case-test-fix, ignorecase-nick, renamefields, highlight by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • Commit 4538e24 on master, transliteration, ignore-case-test-fix, ignorecase-nick, renamefields, highlight, add-aggregation-nick by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • Commit 4538e24 on master, transliteration, ignore-case-test-fix, ignorecase-nick, renamefields, highlight, add-aggregation-nick, 2247923-Test_the_RenderedItem_processor by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • Commit 4538e24 on master, transliteration, ignore-case-test-fix, ignorecase-nick, renamefields, highlight, add-aggregation-nick, 2247923-Test_the_RenderedItem_processor, 2286813-fields-processor-plugin-base-test by ekes: 
    Issue #2257113 Save and check processor enabled fields with indexes.

  • ekes committed 4538e24 on add_aggregatedfield_test 
    Issue #2257113 Save and check processor enabled fields with indexes.
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs review » Fixed

This seems to be fixed?

Status: Fixed » Closed (fixed)

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