UpdateFetcher and UpdateProcessor services throw an ArgumentCountError with drupal 11.0.9:

ArgumentCountError: Too few arguments to function Drupal\update\UpdateFetcher::__construct(), 3 passed in /app/web/core/lib/Drupal/Component/DependencyInjection/Container.php on line 259 and exactly 4 expected in Drupal\update\UpdateFetcher->__construct() (line 65 of /app/web/core/modules/update/src/UpdateFetcher.php)

ArgumentCountError: Too few arguments to function Drupal\update\UpdateProcessor::__construct(), 7 passed in /app/web/modules/contrib/drd/src/UpdateProcessor.php on line 44 and exactly 8 expected in Drupal\update\UpdateProcessor->__construct() (line 107 of /app/web/core/modules/update/src/UpdateProcessor.php)

CommentFileSizeAuthor
#3 3491607_drd_argument_count_errors_3.patch2.21 KBadinac

Issue fork drd-3491607

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

Comments

adinac created an issue. See original summary.

adinac’s picture

adinac commented
Issue summary: View changes
adinac’s picture

adinac commented
StatusFileSize
new 3491607_drd_argument_count_errors_3.patch2.21 KB
jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented
Status: Active » Needs work

Thanks for your contribution.

No patches, please. Turn this into an issue fork with an MR so that tests can run. Once all tests are green, please set the status to "Needs review"

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented

@adinac do you want to follow-up on this?

atul_ghate made their first commit to this issue’s fork.

atul_ghate opened merge request !28

atul_ghate’s picture

atul_ghate commented
Status: Needs work » Needs review

Hi @jurgenhaas i have converted patch to MR please review.

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented
Status: Needs review » Fixed

This is looking good and tests are all green as well. Thank you @atul_ghate, I've merged that and will publish another release.

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented
Status: Fixed » Needs work

This code breaks on PHP 8.2 and Drupal 10.4 - the protected in the constructor needs to be removed.

  • jurgenhaas committed 83c84058 on 4.1.x 
    Issue #3491607 by jurgenhaas, atul_ghate, adinac: Fix PHP 8.2...
jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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