Search API beta4 removed the method filterForPropertyPath from ProcessorPluginBase, which is called in FilesExtractor::addFieldValues. This leads to an undefined function error on site indexing. It could be fixed either by refactoring addFieldValues to remove filterForPropertyPath, or copying the filterForPropertyPath from the old ProcessorPluginBase into FilesExtractor.

CommentFileSizeAuthor
#5 2846467-5.patch763 bytesblazey
#2 search_api_attachments-filterForPropertyPath_method_undefined-2846467-2.patch1.24 KBnicholas.alipaz

Comments

whiplashomega created an issue. See original summary.

nicholas.alipaz’s picture

nicholas.alipaz commented
StatusFileSize
new search_api_attachments-filterForPropertyPath_method_undefined-2846467-2.patch1.24 KB

Here is a patch against dev which just copies the method from search_api.

nicholas.alipaz’s picture

nicholas.alipaz commented
Status: Active » Needs review
f0ns’s picture

f0ns commented

Same issue, the patch posted in #2 works!

blazey’s picture

blazey commented
StatusFileSize
new 2846467-5.patch763 bytes

Hey. First of all thanks for your work! Good effort. IMHO there's no need to reimplement filterForPropertyPath. It's not been REmoved but just moved to Drupal\search_api\Utility\FieldsHelper :).

leksat’s picture

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

Tested manually - works!
Patch looks good.

ekes’s picture

ekes commented

NB This is also a dupe of the also RTBC #2827028: FilesExtrator processor should use new getFieldsHelper() function for adding fields values

  • izus committed ea4e4cf on 8.x-1.x authored by blazey 
    Issue #2846467 by nicholas.alipaz, blazey, whiplashomega, Fons Vandamme...
izus’s picture

izus commented
Status: Reviewed & tested by the community » Fixed

Thanks all and sorry for the long time
this will be part of next beta that will be available today

Status: Fixed » Closed (fixed)

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