Problem/Motivation

ArgumentCountError: Too few arguments to function Solarium\Core\Client\Client::__construct(), 0 passed in docroot\modules\contrib\acquia_connector\acquia_search\src\Plugin\SolrConnector\SearchApiSolrAcquiaConnector.php on line 189 and at least 2 expected in Solarium\Core\Client\Client->__construct() (line 277 of D:\Projects\P-KWALL\P-ERWC\vendor\solarium\solarium\src\Core\Client\Client.php).

Steps to reproduce

  • Step1:- Update Acquia Connector to latest version which is 8.x-1.26
  • Step2:- Update Search API to latest version which is search_api_solr 1.10.0
  • Step3:- Go to status report page /admin/reports/status
CommentFileSizeAuthor
#7 3209105.patch400 bytesmkalkbrenner

Issue fork search_api_solr-3209105

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

someshver created an issue. See original summary.

japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry at Acquia commented
Project: Acquia Connector » Search API Solr
Version: 8.x-1.26 » 8.x-1.10
Priority: Critical » Minor

This is an issue in Search API Solr, it looks like composer.json isn't configured right, because 1.10 says its compatible with Acquia Connector 1.25+. This is false. If you're using Acquia Connector 8.x-1.x with Acquia Search, you must use Search API Solr 8.x-1.7. All newer versions of Search API Solr 8.x-1.x require Acquia Connector 3.x and Acquia Search 2.x

Remaining task:
Update composer.json to fix the conflict clause.

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
CreditAttribution: mkalkbrenner at Jarltech Europe GmbH commented

Can you provide a patch?

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
CreditAttribution: mkalkbrenner at Jarltech Europe GmbH commented

I would prefer to remove the compatibility statement entirely as it just was a temporary workaround.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Related issues: +#3197199: Compatibility with Solarium 6 and Search API Solr 8.x-1.8+

This is a problem with the Acquia search module that shows up because Search API Solr 8.x-1.x was updated to Solarium 6, the Pantheon module has the same problem in #3197199: Compatibility with Solarium 6 and Search API Solr 8.x-1.8+. This should be moved to the Acquia search module's issue queue and worked on there.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Version: 8.x-1.10 » 8.x-1.x-dev
Priority: Minor » Normal
Issue tags: -solarium, -Search API
mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
CreditAttribution: mkalkbrenner at Jarltech Europe GmbH for Acquia commented
Status: Active » Needs review
FileSize
3209105.patch400 bytes

All these Acquia modules are very confusing ;-)

  • mkalkbrenner committed 8ab4661 on 8.x-1.x 
    Issue #3209105 by mkalkbrenner, japerry: ArgumentCountError: Too few...
mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
CreditAttribution: mkalkbrenner at Jarltech Europe GmbH for Acquia commented
Status: Needs review » Fixed

  • mkalkbrenner committed 8de5766 on 8.x-1.x 
    Issue #3209105 by mkalkbrenner, japerry: ArgumentCountError: Too few...

Status: Fixed » Closed (fixed)

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

stephenrodrigo@yahoo.com made their first commit to this issue’s fork.