Problem/Motivation

The update of the search_api module from 8.x-1.0-beta3 to 8.x-1.0-beta4 causes many Paragraphs tests to fail.

Namely:

Paragraphs.Drupal\paragraphs_demo\Tests\ParagraphsDemoTest.testConfigurationsAndCreation
Paragraphs.Drupal\paragraphs_demo\Tests\ParagraphsDemoTest.onConfigSave
Paragraphs.Drupal\paragraphs_type_permissions\Tests\ParagraphsTypePermissionsTest.testAnonymousParagraphsTypePermissions
Paragraphs.Drupal\paragraphs_type_permissions\Tests\ParagraphsTypePermissionsTest.onConfigSave
Paragraphs.Drupal\paragraphs\Tests\ParagraphsUninstallTest.testUninstall
Paragraphs.Drupal\paragraphs\Tests\ParagraphsUninstallTest.onConfigSave
Paragraphs.Drupal\paragraphs\Tests\ParagraphsTranslationTest.testParagraphTranslation
Paragraphs.Drupal\paragraphs\Tests\ParagraphsTranslationTest.onConfigSave
Paragraphs.Drupal\paragraphs\Tests\ParagraphsAccessTest.testParagraphAccessCheck
Paragraphs.Drupal\paragraphs\Tests\ParagraphsAccessTest.onConfigSave

Proposed resolution

Remaining tasks

Find the cause and fix it.

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 search_api_update-2841069-3.patch1.15 KBModernMantra

Comments

VladimirMarko created an issue. See original summary.

VladimirMarko’s picture

VladimirMarko commented
Assigned: Unassigned » VladimirMarko
ModernMantra’s picture

ModernMantra commented
Status: Active » Needs review
StatusFileSize
new search_api_update-2841069-3.patch1.15 KB

Seems that search_api.index.paragraphs_demo_index.yml had different structure that conflicted since plugin id was dropped in search_api.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Looks good.

collect_demo probably need the same fix and maybe other projects too if we have search_api integrations there, maybe core_crm? Not sure...

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

Committed, thx.

VladimirMarko’s picture

VladimirMarko commented 
+++ b/modules/paragraphs_demo/config/install/search_api.index.paragraphs_demo_index.yml
@@ -38,26 +38,21 @@ field_settings:
+  rendered_item: {  }
+  aggregated_field: {  }

These two lines are unnecessary.
They have no equivalent in the original configuration file and it works fine without them.

Status: Fixed » Closed (fixed)

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