Problem/Motivation

Error appears on page /admin/reports/site-audit "Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException: You have requested a non-existent service "site.path". in Drupal\Component\DependencyInjection\Container->get() (line 157 of core/lib/Drupal/Component/DependencyInjection/Container.php)."

Steps to reproduce

- Install and enable Vardot support module
- Try open page /admin/reports/site-audit

Proposed resolution

Fix the code issue.

Remaining tasks

  • ✅ File an issue about this project
  • ✅ Addition/Change/Update/Fix to this project
  • ✅ Testing to ensure no regression
  • ➖ Automated unit/functional testing coverage
  • ➖ Developer Documentation support on feature change/addition
  • ➖ User Guide Documentation support on feature change/addition
  • ➖ Accessibility and Readability
  • ✅ Code review from 1 Varbase core team member
  • ✅ Full testing and approval
  • ✅ Credit contributors
  • ➖ Review with the product owner
  • ✅ Update Release Notes and Update Helper on new feature change/addition
  • ❌ Release vardot_support-1.0.0-beta8

Varbase update type

  • ✅ No Update
  • ➖ Optional Update
  • ➖ Forced Update
  • ➖ Forced Update if Unchanged

User interface changes

  • N/A

API changes

  • N/A

Data model changes

  • N/A

Release notes snippet

  • Issue #3376407 by Qusai Taha: Fixed error You have requested a non-existent service site.path
CommentFileSizeAuthor
#2 3376407-2.patch798 bytesqusai taha
localhost-32778-admin-reports-site-audit-gf-gdhg.png197.34 KBqusai taha

Issue fork vardot_support-3376407

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

Qusai Taha created an issue. See original summary.

qusai taha’s picture

qusai taha commented
Assigned: qusai taha » Unassigned
Status: Active » Needs review
StatusFileSize
new 3376407-2.patch798 bytes
keshavv’s picture

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

I have reviewed the patch and confirmed that we can merge it, As method is deprecated.
https://stefvanlooveren.me/errors/how-fix-sitepath-service-deprecated-dr...

rajab natshah’s picture

rajab natshah
he/him
commented
Status: Reviewed & tested by the community » Needs review
rajab natshah’s picture

rajab natshah
he/him
commented
Title: Fix error You have requested a non-existent service "site.path" » Fix error You have requested a non-existent service site.path
Status: Needs review » Needs work

MR is needed

rajab natshah’s picture

rajab natshah
he/him
commented
Priority: Major » Critical

Qusai Taha opened merge request !3

qusai taha’s picture

qusai taha commented
Status: Needs work » Needs review

I've created the merge request. Could you please check it?

rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
Issue tags: +vardot_support-1.0.0-beta8
rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes

✅ Released vardot_support-1.0.0-beta8

rajab natshah’s picture

rajab natshah
he/him
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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