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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

whiplashomega created an issue. See original summary.

nicholas.alipaz’s picture

nicholas.alipaz CreditAttribution: nicholas.alipaz commented
FileSize
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 CreditAttribution: nicholas.alipaz commented
Status: Active » Needs review
f0ns’s picture

f0ns CreditAttribution: f0ns as a volunteer and at EntityOne commented

Same issue, the patch posted in #2 works!

blazey’s picture

blazey CreditAttribution: blazey at Amazee Labs commented
FileSize
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 CreditAttribution: Leksat at Amazee Labs commented
Status: Needs review » Reviewed & tested by the community

Tested manually - works!
Patch looks good.

ekes’s picture

ekes CreditAttribution: ekes as a volunteer 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 CreditAttribution: 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.