Problem/Motivation

I tried to update core 8.7 to core 8.8 and pathauto from 1.3 to 1.8 together with the bibcite module and got following error:

[notice] Update started: bibcite_entity_update_8025
[error]  Argument 2 passed to Drupal\pathauto\AliasStorageHelper::__construct() must be an instance of Drupal\path_alias\AliasRepositoryInterface, instance of Drupal\Core\Path\AliasRepository given 
[error]  Update failed: bibcite_entity_update_8025 

Proposed resolution

A workaround is to update core and pathauto first and run the database updates. After the update you can update bibcite and run the bibcite database updates.

Comments

Corn696 created an issue. See original summary.

jienckebd’s picture

Attached patch seems to fix the issue.

antongp’s picture

Status: Active » Closed (won't fix)

Hello @Corn696,
Thank you for your issue.

For some reason I could not reproduce the issue actually. So, I updated simultaneously:

  • Drupal 8.7.14 --> 8.8.8
  • Pathauto 8.x-1.3 --> 8.x-1.8
  • Bibliography & Citation 8.x-1.0-alpha10 --> latest 8.x-1.x-dev

Code was updated via Composer, database updates were applied via Drush (10.3.0).

Once I actually got different error when tried to update Bibliography & Citation to latest dev on Drupal 8.7 and it was caused by the same bibcite_entity_update_8025() hook. Looks this hook is not compatible with Drupal 8.7, but anyway Drupal 8.7 is not maintained anymore.

Also, there are Drupal 8.8.0 release notes that ask to update Pathauto before or at the same time as you update to Drupal 8.8.0: https://www.drupal.org/project/drupal/releases/8.8.0 . And Pathauto 8.x-1.7 release notes recommend to use pathauto 1.6 while updating from Drual 8.7 and only update to 1.7 afterwards: https://www.drupal.org/project/pathauto/releases/8.x-1.7 .

I think the best way would be:

  1. Update Pathauto to 8.x-1.6 within Drupal 8.7
  2. Update Drupal to 8.8
  3. Update Pathauto to latest
  4. Update Bibliography & Citation to latest dev.

It doesn't seem that there is something to fix in Bibliography & Citation.

@jienckebd is this patch for alpha10? Latest dev uses EntityTypeManagerInterface already...

Thanks!