Closed (fixed)
Project:
Search API
Version:
8.x-1.x-dev
Component:
Plugins
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 May 2019 at 15:04 UTC
Updated:
6 Apr 2020 at 10:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
mkalkbrennerComment #3
drunken monkeyDo you have a stack trace for this error? (Can be enabled at
/admin/config/development/loggingif Devel module is installed (or maybe even without).)Does your index contain media entities?
Comment #4
wblancditjolicoeur commentedHi,
I got the same behaviour when trying to upgrade from search_api_solr 1.x to 3.x ( here from 1.3 to 3.8) while still using the contrib "Media Entity" Module 8.x-1.8.
Of course, Media are meant to be indexed.
any action on /admin/config/search/search-api/index/index_name/edit, /admin/config/search/search-api/index/index_name/fields ,
or drush sapi-c, sapi-r sapi-i will result in the following trace :
Is there any quick workaround to this ? Or should I definitly urge to switch to Media Entity in core ?
Comment #5
drunken monkeyBrilliant, thanks a lot for the backtrace! Seems the flaw we need to fix in this module is insufficient error handling in the “Add hierarchy” processor. Please see, test and review the attached patch!
However, from the stack trace it’s apparent that there is also an underlying issue in the Search API Attachments module, so I posted a patch there was well: #3117949: Improve exception handling in FilesExtractor. Would be great if you could review that patch (independently of this one), too! Would be great to get both of these fixed.
Comment #6
drunken monkeyPS: Hm, looking at this again, as the
FilesExtractor::getPropertyDefinitions()method doesn’t follow its contract, by propagating/throwing exceptions it shouldn’t, the patch here wouldn’t actually do anything to resolve your problem. It should still be an improvement, but you’ll probably need the Attachments patch for the problem to go away.Comment #8
drunken monkeyCommitted.
If possible, please still also give feedback on the Attachments patch in #3117949: Improve exception handling in FilesExtractor. (As said, the commit here will most likely not help with your problem.)