Problem/Motivation
This module is still calling variable_get() in the code, which doesn't exist anymore. It also uses functions that don't exist, such as ultimate_cron_job_load(), ultimate_cron_job_get_status(), ultimate_cron_job_set_status(), module_implements(), etc.
Many of these are in QueueSettings.php
It'd also be nice if the module worked towards Drupal coding standards compliance (such as using short array syntax and using doc comments). Something as simple as running drupalmoduleupgrader and phpcs or phpcbf would help a bunch.
Comments
Comment #2
guilhermevp commentedComment #3
guilhermevp commentedModule is stated as Drupal9 compatible in #3141993: Automated Drupal Rector fixes
Comment #4
solideogloria commentedThat doesn't mean that it hasn't been completely converted. Just because it happens to work so far doesn't mean there isn't something broken. Is the QueueSettings.php file unused then? Non-existent functions shouldn't be used in the project, so whether it's working or not, this piece needs to be cleaned up.
When I open this module in VS Code, files from this module are full of errors and standards violations. Here's a piece:
Comment #5
berdir> Many of these are in QueueSettings.php
That is dead, unused code.
Yes, those things would be nice, but this module is only alpha for a reason, sadly I have limited resources to spend on improving it.
Open for specific patches and contributes, but dong code style improvements like this is generally hard, because it is a lot of work to review it and conflicts often with other issues.