Convert several test modules and remove .module files [2]

Created on 10 June 2025, 22 days ago

Problem/Motivation

Preprocess hooks in module can now be OOP so we can convert the remaining test modules.
Convert modules until about 200 lines are changed since these are small.

  • layout_test

Steps to reproduce

Proposed resolution

Convert any remaining functions in the .module file.
If they are hooks name the file according to the structure recommended in 📌 [meta] Clean up hook classes in core Active
Organize existing hooks according to the above referenced issue.
Add dependency injection.

Remaining tasks

N/A

User interface changes

N/A

Introduced terminology

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

📌 Task
Status

Active

Version

11.0 🔥

Component

base system

Created by

🇺🇸United States nicxvan

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

Comments & Activities

  • Issue created by @nicxvan
  • 🇺🇸United States nicxvan

    I forgot to link the parent issue some of those cannot be converted and that is too many

  • 🇮🇳India annmarysruthy

    I found .module files in these modules too. should these be changed too

    • views_test_data
    • views_test_rss
    • toolbar_test
    • common_test
    • deprecation_test
    • entity_test
    • hook_collector_on_behalf_procedural
    • hook_collector_skip_procedural
    • hook_collector_skip_procedural_attribute
    • module_implements_alter_test
    • module_test
    • olivero_test
    • pager_test
    • system_test
    • theme_region_test
    • theme_test
    • twig_theme_test
  • 🇺🇸United States nicxvan

    I have added the parent issue, the ones in the evaluate list that do not have an issue are able to be worked on.

    A couple of notes, try to keep changes to around 200 lines so if a module is small you can do another, but if there is a lot to convert then do not.

    Don't convert template_preprocess_HOOKs in this issue either.

    Thanks!

Production build 0.71.5 2024