Problem/Motivation

pathauto.api.php refers to 'pathauto.pathauto.inc', but the file doesn't exist.

 * It may be helpful to review some examples of integration from
 * pathauto.pathauto.inc.

Steps to reproduce

view pathauto.api.php

Proposed resolution

remove or update the documentation

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 3165993-5.patch552 bytespflora
#2 updated-3165993-2.patch485 bytessamvada_jain_m

Issue fork pathauto-3165993

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

oknate created an issue. See original summary.

samvada_jain_m’s picture

samvada_jain_m commented
Status: Active » Needs review
StatusFileSize
new updated-3165993-2.patch485 bytes

Please review

deaom’s picture

deaom commented
Version: 8.x-1.8 » 8.x-1.x-dev
Status: Needs review » Reviewed & tested by the community

Patch applies and replaces the 'pathauto.pathauto.inc' with the 'pathauto.tokens.inc', so marking this as RTBC and changing version to dev.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Needs work

tokens.inc is a file that exists, but this is not the same thing. A more useful thing to say would be to look at \Drupal\pathauto\Plugin\pathauto\AliasType\EntityAliasTypeBase and subclasses. That's where this logic now lives.

pflora’s picture

pflora commented
Status: Needs work » Needs review
StatusFileSize
new 3165993-5.patch552 bytes

I've made the requested changes! Moving it to needs review.

ngkoutsaik’s picture

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

I reviewed the changes. The patch was applied without issues and the tests were run. The patch has the suggested changes.

Manoj Raj.R’s picture

Manoj Raj.R commented

#5 looks good to me since that addresses proposed solutions.

nitapawar’s picture

nitapawar commented

#5 looks good to me!

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Needs work

@see must only be used as a standalone element with a single reference and not within sentence, it's just see then.

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

mably’s picture

mably commented
Status: Needs work » Closed (duplicate)
Related issues: +#3104119: Functions on pathauto.api.php documentation do not exist

Closing it as a duplicate of #3104119: Functions on pathauto.api.php documentation do not exist.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.