Problem/Motivation

Currently, the code in PathAliasProcessor detects entity from path parts. However, for taxonomy/term/ paths, "taxonomy" is not an entity name.

Steps to reproduce

Try generating an alias for taxonomy term. The functionality does not succeed.

Proposed resolution

Use url validator to convert path to url, extract url parameters, check if they work.

CommentFileSizeAuthor
#3 3426601-2-fix-taxonomy-term-detection.patch2.99 KBalexrayu
#2 3426601-fix-taxonomy-term-detection.patch3.07 KBalexrayu

Comments

alexrayu created an issue. See original summary.

alexrayu’s picture

alexrayu commented
Status: Active » Needs review
StatusFileSize
new 3426601-fix-taxonomy-term-detection.patch3.07 KB
alexrayu’s picture

alexrayu commented
StatusFileSize
new 3426601-2-fix-taxonomy-term-detection.patch2.99 KB

Remove unused use statement.

macsim’s picture

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

Nice catch. Thanks for the patch.
I tested it and it works like a charm

  • MacSim committed 6f237245 on 3.1.x authored by alexrayu 
    Issue #3426601 by alexrayu: Entity type "taxonomy" does not exist
macsim’s picture

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

  • MacSim committed 8de681dc on 2.0.x 
    Issue #3426601 by alexrayu: Entity type taxonomy does not exist

  • MacSim committed 676210d3 on 1.0.x 
    Issue #3426601 by alexrayu: Entity type "taxonomy" does not exist

  • MacSim committed 8346ec97 on 2.0.x 
    Issue #3426601 by alexrayu: Entity type "taxonomy" does not exist

  • MacSim committed 629393c0 on 1.0.x 
    Issue #3426601 by alexrayu: Entity type "taxonomy" does not exist

macsim’s picture

macsim commented
macsim’s picture

macsim commented
Status: Fixed » Closed (fixed)