Problem/Motivation

When installing an existing codebase with the Visitors module enabled we get the following error:
The service "visitors.visibility" has a dependency on a non-existent service "path_alias.manager".

Proposed resolution

Add the core path_alias module as a dependency in the info file.

CommentFileSizeAuthor
#2 3395217-dependency-on-path-alias.patch244 bytesmichel.g

Issue fork visitors-3395217

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

michel.g created an issue. See original summary.

michel.g’s picture

michel.g commented
StatusFileSize
new 3395217-dependency-on-path-alias.patch244 bytes
michel.g’s picture

michel.g commented
Status: Active » Needs review
bluegeek9’s picture

bluegeek9 commented
Status: Needs review » Active

@michel.g,

Thank you for reporting this bug. I don't want to require the path_alias module. I can update the constructor of visitors.visibility to check if path_alias is installed and only add path_alias.manager if it is.

bluegeek9’s picture

bluegeek9 commented
Assigned: michel.g » Unassigned
bluegeek9’s picture

bluegeek9 commented
Assigned: Unassigned » bluegeek9

bluegeek9 opened merge request !116

bluegeek9’s picture

bluegeek9 commented
Assigned: bluegeek9 » Unassigned
Status: Active » Fixed

  • bluegeek9 committed 81c9df74 on 8.x-2.x 
    Issue #3395217 by bluegeek9: On a fresh install of a Drupal site with...

Status: Fixed » Closed (fixed)

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

bluegeek9’s picture

bluegeek9 commented

This issue was resolved. A new release, 8.x-2.17, will be made soon, May 31st.

This is an excellent opportunity to beta test the dev branch, and report any issues.

Contributors (5)

bluegeek9, roshni27, abhishek_gupta1, sarwan_verma, SandeepSingh199

Changelog

Issues: 32 issues resolved.

Changes since 8.x-2.16:

Bug

Feature

Task