Problem/Motivation

The drush script of bs_base uses an obsolete service site.path.

Steps to reproduce

I get the error

You have requested a non-existent service "site.path".

when running
drush bs-tc

CommentFileSizeAuthor
#3 3346593-missing_service-3.patch605 bytesytsurk

Comments

ytsurk created an issue. See original summary.

ytsurk’s picture

ytsurk
Location Switzerland
commented
Issue summary: View changes
ytsurk’s picture

ytsurk
Location Switzerland
commented
Status: Active » Needs review
StatusFileSize
new 3346593-missing_service-3.patch605 bytes

Here a patch replacing the service with \Drupal::getContainer()->getParameter('site.path')

pivica’s picture

pivica commented
Status: Needs review » Reviewed & tested by the community

Original core issue #3259024: Remove deprecated 'app.root' and 'site.path' services.

I've checked Drupal core 9.3.x and it is already using new `\Drupal::getContainer()->getParameter('site.path')` call and we are stating that current bs_base dev is compatible with 9.3 and up so we can commit this and continue supporting both D9 and D10.

  • pivica committed 55667f14 on 8.x-1.x authored by ytsurk 
    Issue #3346593 by ytsurk: "site.path" service no longer existing in D10
pivica’s picture

pivica commented
Status: Reviewed & tested by the community » Fixed

And committed, thx :)

ytsurk’s picture

ytsurk
Location Switzerland
commented

Howdy :D

Status: Fixed » Closed (fixed)

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