Problem/Motivation

Drupal Core has a new API for form validation using #config_target, see https://www.drupal.org/node/3373502

Proposed resolution

Update \Drupal\domain_path\Form\DomainPathSettingsForm to use #config_target

Remaining tasks

  1. Write a merge request
  2. Review
  3. Commit

User interface changes

None

API changes

None

Data model changes

None

Issue fork domain_path-3565027

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

idebr created an issue. See original summary.

idebr opened merge request !60

idebr’s picture

idebr commented
Status: Active » Needs review

The merge request updates \Drupal\domain_path\Form\DomainPathSettingsForm to use #config_target

Note it includes a database update to rewrite the entity_type setting:

Before:

entity_types:
  media: '0'
  menu_link_content: '0'
  node: '1'
  paragraphs_library_item: '0'
  taxonomy_term: '0'
  user: '0'
  webform_submission: '0'

After:

entity_types:
- node

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

  • mably committed ac43b12d on 2.x authored by idebr 
    task: #3565027 Use #config_target for the settings form

By: idebr
mably’s picture

mably commented
Status: Needs review » Fixed

That's some great work, thanks @idebr! Merged.

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.

Status: Fixed » Closed (fixed)

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