- Issue created by @svendecabooter
- First commit to issue fork.
- π§πͺBelgium svendecabooter Gent
MR introduces PHPCS / PHPStan issues, which would ideally be resolved before merging.
- π§πͺBelgium aren33k
@svendecabooter agreed on the constructor promotion, but won't this cause issues for drupal 9 installations?
- π§πͺBelgium svendecabooter Gent
Drupal 9 is no longer officially supported, and OOP hooks only work for 10.1+ anyway
- Assigned to aren33k
- Status changed to Needs work
2 days ago 12:33pm 30 June 2025 - π³π±Netherlands eelkeblok Netherlands π³π±
Ah, I had not noticed this issue before. I added a hook_cron implementation in β¨ Add a cleanup function Active and did not use this pattern because of the Drupal 9 support (although I may have used some PHP features that rule out D9, not sureπ«’). Will probably depend on the sequence of merging what needs to be adapted to what.
-
svendecabooter β
committed 75a4340a on 1.0.x authored by
aren33k β
Issue #3519842 by aren33k, svendecabooter: Support OOP hooks
-
svendecabooter β
committed 75a4340a on 1.0.x authored by
aren33k β
- π§πͺBelgium svendecabooter Gent
Remaining issues have been resolved, and this has been added to 1.0.x branch.
@eelkeblok would be good if you could rebase your MR branch on 1.0.x and also add support for OOP hooks. Sorry for the extra work.