- First commit to issue fork.
- π§πͺBelgium mschudders
Created the branch + applied the patch from: ΓΈkse to that branch.
Credits to ΓΈkse
- Merge request !12Issue #3332952: Make it possible to index attachments from index only fields that don't exist on the entity β (Open) created by mschudders
- π§πͺBelgium ΓΈkse
The patch is a first proposition to come to a solution but isn't perfect by any means hence why I didn't upload it so far.
Remaining room for improvement:
- All the fields on the index (also the content type based added fields) are available to get the attachments indexed from. This should be improved to only allow non content type relevant fields
- Changing the machine name of the search api attachments source field on the index does cause issues. This should definitly be improved to have a more flexible behaviour.
- π§πͺBelgium ΓΈkse
I made an update to only allow non content type relevant fields (layout builder specific) to be indexed instead of all the fields coming from the index.
Because of this, you don't longer need to use the Layout Builder Search API β contrib module (sorry maintainer) in order to get the reference as a field onto the search index. The update makes sure that referenced fields that have a reference to a file/media entity within block types in layout builder are available as indexable fields to get possible attachments indexed from.
After fixing this, the machine name naming convention is no longer an issue.
- Status changed to Needs review
about 2 years ago 12:39pm 3 April 2023 - π§πͺBelgium tim-diels Belgium π§πͺ
When you provide an (updated) patch, you should set the issue to Needs Review to make it clear people can test this.
- π§πͺBelgium tim-diels Belgium π§πͺ
Removed not needed use statements.
- π§πͺBelgium tim-diels Belgium π§πͺ
Fixed the PHP CS issues form this issue.
- Status changed to RTBC
about 2 years ago 1:28pm 3 April 2023 - π§πͺBelgium tim-diels Belgium π§πͺ
Tested and works as expected. Thanks for the work done.
- π§πͺBelgium kriboogh
Tested, works.
MR diff patch added for use in composer. - Status changed to Needs work
about 2 months ago 4:42pm 18 February 2025 - First commit to issue fork.
- π§πͺBelgium RandalV
Merged latest changes from 9.0.x into MR branch, seems to apply and work fine on my end.
Setting back to 'Needs review' so someone else can have a test before it's merged! - π«π·France mably
Wondering if it shouldn't rather be done in its own FileExtractor processor.
- π¬π§United Kingdom joachim
The code in this MR should be moved into a plugin once β¨ Plugin type to allow other ways to determine files to be indexed Active is in.