Problem/Motivation

The hooks in .module files should be converted to OOP hooks. They should be in a logical file structure too (if not sure ask)

Tests should pass before review.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork quiz-3571376

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

smustgrave created an issue. See original summary.

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

miroslavstankov’s picture

miroslavstankov commented
Assigned: Unassigned » miroslavstankov

miroslavstankov opened merge request !69

miroslavstankov’s picture

miroslavstankov commented
Assigned: miroslavstankov » Unassigned
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Needs work

Thanks! But MR appears to have test failures.

smustgrave’s picture

smustgrave commented

Also I'm seeing that the content of the hooks may have chanced and we really shouldn't do that in this ticket. This was just conversions, think that's what the test failures are showing.