Problem/Motivation

I have run into a need of figuring out what index is being acted on when the AlterSettingsEvent is dispatched but this information is not present.

Steps to reproduce

N/A

Proposed resolution

Expose the index on AlterSettingsEvent as a read-only property so subscribers know which index is being acted on, without being able to mutate it.

Remaining tasks

N/A

User interface changes

N/A

API changes

Third parameter to the event and a new getIndex getter.

Data model changes

N/A

Issue fork search_api_opensearch-3591981

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

achap created an issue. See original summary.

achap opened merge request !143

achap’s picture

achap
Location 🇦🇺
commented
Status: Active » Needs review

kim.pepper made their first commit to this issue’s fork.

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Title: AlterSettingsEvent should also include the index » Add read-only index to AlterSettingsEvent
Issue summary: View changes

Updated title and IS

  • kim.pepper committed 9b408fec on 3.x authored by achap 
    task: #3591981 Add read-only index to AlterSettingsEvent

By: achap
By:...

  • kim.pepper committed ab7f3798 on 2.x 
    task: #3591981 Add read-only index to AlterSettingsEvent

By: achap
By:...
kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Status: Needs review » Fixed

Committed to 2.x and 3.x. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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