It's all in the title.

Issue fork domain_path-3573947

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

mably created an issue. See original summary.

mably opened merge request !77

mably’s picture

mably commented
Status: Active » Needs review

Patch converting all procedural hooks to OOP #[Hook] classes with #[LegacyHook] wrappers for Drupal 10.6 backward compatibility.

New files

  • src/Hook/DomainPathEntityHooks.php — entity_delete, entity_base_field_info, entity_type_alter, validation_constraint_alter
  • src/Hook/DomainPathFormHooks.php — form_alter, domain_source_alter
  • modules/domain_path_pathauto/src/Hook/DomainPathPathautoHooks.php — field_info_alter, field_widget_info_alter, entity_delete, domain_delete

Modified files

  • domain_path.services.yml — added class aliases for DomainPathHelper and DomainAliasManagerInterface (needed for autowiring), plus hook class registrations
  • domain_path.module — replaced 6 hook bodies with #[LegacyHook] + delegation
  • domain_path_pathauto.services.yml — added _defaults block (autoconfigure/autowire) and hook class registration
  • domain_path_pathauto.module — replaced 4 hook bodies with #[LegacyHook] + delegation

Not converted (intentionally)

Install/update hooks remain procedural as per Drupal convention.

Tests

All 11 existing functional tests pass (149 assertions), no regressions.

  • mably committed 6f8810b0 on 2.x 
    feat: #3573947 Convert all hooks to OOP format

By: mably
mably’s picture

mably commented
Status: Needs review » Fixed

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.