Move domain path logic from node submit handler to entity API level

Created on 3 November 2021, over 2 years ago
Updated 7 November 2023, 8 months ago

Problem/Motivation

Domain path logic is done in node form submission. Therefore, it does not act on contents created by other ways, such as via REST or programmatically.

Steps to reproduce

In a installation with the domain_path and domain_path_pathauto modules enabled and some pathauto pattern set, create a new content programmatically. The domain path alias is not created.

Proposed resolution

Move domain path logic to entity API based events, as pathauto does.

Remaining tasks

TBD.

User interface changes

None.

API changes

TBD.

Data model changes

None.

✨ Feature request
Status

Active

Version

1.0

Component

Code

Created by

πŸ‡ͺπŸ‡ΈSpain manuel.adan 🌌

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡ΊπŸ‡¦Ukraine _tarik_ Lutsk

    Hi quadrexdev
    This module is used in one of my projects with JSON:API and has problems mentioned in the task description.
    I plan to start work on this in the next few days. Would you mind assigning this issue to me?
    I plan to do these steps to fix the problem:

    1. Create a field item and field widget objects for the domain path.
    2. Add code for altering base field info for entities that support domain path. I think it will be the same solution as we already have in the path auto module.
    3. Update the domain path auto module to make its fields available on the entity API level.
  • πŸ‡ΊπŸ‡¦Ukraine _tarik_ Lutsk

    Hi, I have pushed part of the required changes that provide the ability to use domain path fields on the entity API level.
    However, changes for the domain_path_auto aren't finished. We need to cover new code by tests and refactor it.
    But that is the start of work on this issue.

Production build 0.69.0 2024