Hello,
It looks like search_api has a new way for getting the fields by property path. Using new function getFieldsHelper() in the addFieldValues() function will fix it.
I will attach a patch for it.

CommentFileSizeAuthor
#2 use-getFieldsHelper-in-FilesExtrator-class-2827028-2.patch808 bytestuwebo

Comments

TuWebO created an issue. See original summary.

tuwebo’s picture

tuwebo commented
Status: Active » Needs review
StatusFileSize
new use-getFieldsHelper-in-FilesExtrator-class-2827028-2.patch808 bytes

Patch attached. This should work with the last dev versions of search_api and search_api_solr.

ekes’s picture

ekes commented
Status: Needs review » Reviewed & tested by the community

API change introduced: #2782577: Fix extraction of configurable properties in processors
Patch https://www.drupal.org/files/issues/processors_property_extraction-27825... does exactly the same with other processors.

izus’s picture

izus commented
Status: Reviewed & tested by the community » Postponed (maintainer needs more info)

as https://www.drupal.org/node/2782577 is fixed, i wonder if we have something else to do here or just clode the issue. Is the issue solved for you or it's accurate ?
Thanks

tuwebo’s picture

tuwebo commented

Hi @izus,
I've to take a look at it. Probably in the next weeks. I am a little bit outdated with the search_api code now. As soon as I know something I'll post it here.

tuwebo’s picture

tuwebo commented
Status: Postponed (maintainer needs more info) » Closed (duplicate)

Hi @izus,
I've just been taken a look and this issue has been fixed in this other one #2846467: Search API beta4 removes filterForPropertyPath from ProcessorPluginBase which breaks FilesExtractor::addFieldValues leading to error on indexing which was posted later than this one, but got commited. So I guess we can mark this one as duplicated.

Thanks for the review.

tunic’s picture

tunic
he/him
Primary language Spanish
Location Madrid
commented

This issue has been addressed in #2846467: Search API beta4 removes filterForPropertyPath from ProcessorPluginBase which breaks FilesExtractor::addFieldValues leading to error on indexing. Note that patch from this current issue in #2 was posted two months earlier and IMHO is clearer (it uses a temporary variable instead a big line in the foreach sentence), issue may have been fixed in shorter time :-P

izus’s picture

izus commented

Thanks all for the feedback