I tested this module on a fresh Drupal 9 environment, by enabling Drupal 9 support. I have tested and it works. There are some issues when running the "Upgrade status" scan. It provided this result:
CUSTOM PROJECTS
--------------------------------------------------------------------------------
Recipe
Scanned on Sat, 06/06/2020 - 14:29.
55 warnings found.
web/modules/recipe/modules/ingredient/src/Feeds/Target/Ingredient.php:
┌──────────┬──────┬───────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼───────────────────────────────────────────────────────────┤
│ Check │ │ Class Drupal\feeds\Feeds\Target\EntityReference not found │
│ manually │ │ and could not be autoloaded. │
│ │ │ │
│ Check │ 18 │ Class Drupal\feeds\Feeds\Target\EntityReference not found │
│ manually │ │ and could not be autoloaded. │
│ │ │ │
└──────────┴──────┴───────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Functional/IngredientFieldSettin
gsTest.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 12 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Functional/IngredientFieldTest.p
hp:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 13 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Functional/IngredientTest.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 14 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Functional/IngredientTranslation
Test.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 15 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Kernel/IngredientAccessTest.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 18 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Kernel/IngredientSettingsTest.ph
p:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 13 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Kernel/Migrate/recipe61/MigrateI
ngredient61Test.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 14 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Kernel/Migrate/recipe61/MigrateI
ngredient61TestBase.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 10 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Kernel/Migrate/recipe61/MigrateI
ngredientSettings61Test.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 10 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Kernel/Migrate/recipe71/MigrateI
ngredient71Test.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 14 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Kernel/Migrate/recipe71/MigrateI
ngredient71TestBase.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 10 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/tests/src/Kernel/Migrate/recipe71/MigrateI
ngredientSettings71Test.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 10 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Functional/PlainTextStyleTest.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 17 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Functional/RecipeMlTest.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 13 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Functional/RecipeNodeTest.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 13 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Functional/RecipeSettingsTest.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 13 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Functional/RecipeTestBase.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 10 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Functional/RecipeTotalTimeConfigTest.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 14 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Kernel/Migrate/recipe61/MigrateRecipe61Test.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 15 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Kernel/Migrate/recipe61/MigrateRecipe61TestBase.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 10 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Kernel/Migrate/recipe61/MigrateRecipeDisplaySetting
s61Test.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 12 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Kernel/Migrate/recipe71/MigrateRecipe71Test.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 14 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Kernel/Migrate/recipe71/MigrateRecipe71TestBase.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 10 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Kernel/Migrate/recipe71/MigrateRecipeDisplaySetting
s71Test.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 12 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/tests/src/Kernel/Migrate/recipe71/MigrateRecipeFieldSettings7
1Test.php:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
│ Check │ 12 │ Class PHPUnit\Framework\TestCase not found and could not be │
│ manually │ │ autoloaded. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe:
┌──────────┬──────┬───────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼───────────────────────────────────────────────────────┤
│ Check │ 0 │ The 'recipe' extension is not installed. Cannot check │
│ manually │ │ deprecated library use. │
│ │ │ │
└──────────┴──────┴───────────────────────────────────────────────────────┘
web/modules/recipe/recipe.module:
┌──────────┬──────┬─────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼─────────────────────────────────────────────────────────────┤
│ Check │ 77 │ The 'recipe/export' library is not defined because the │
│ manually │ │ defining extension is not installed. Cannot decide if it is │
│ │ │ deprecated or not. │
│ │ │ │
└──────────┴──────┴─────────────────────────────────────────────────────────────┘
web/modules/recipe/modules/ingredient/src/Plugin/Field/FieldWidget/IngredientWid
get.php:
┌──────────┬──────┬───────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼───────────────────────────────────────────────────────────┤
│ Check │ 119 │ The 'ingredient/drupal.ingredient' library is not defined │
│ manually │ │ because the defining extension is not installed. Cannot │
│ │ │ decide if it is deprecated or not. │
│ │ │ │
└──────────┴──────┴───────────────────────────────────────────────────────────┘
I still get to add and view recipes, so it looks like it works. I would like to contribute to this module, unfortunately I am unexperienced with Drupal 8 module development. I will post a patch if I figure out how for making this module Drupal 9 ready.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 3148439-6.patch | 779 bytes | dcam |
Comments
Comment #2
Mind Design commentedComment #3
dcam commentedThe module is already D9 ready, aside from the version requirements. I didn't want to release a new version until this week, but I've been too busy to work on it. It's on my list of stuff to do this weekend. I'll use your patch as a starting point.
Comment #4
Mind Design commentedI see. Didn't realize that, sorry. Not much of a starting point though, I'll try and find something else to contribute with. Thanks for working on this module, it is a really great one.
Comment #5
dcam commentedNo need to apologize. Thank you for trying to contribute.
For the record, I didn't release a new version earlier because there are BC-breaking changes in the D9-compatibility changes that I already made. Future versions will not be compatible with older versions of D8. That's why I waited to release a new version until all those older versions were no longer supported.
I'm not sure what's up with all of those errors that Upgrade Status found. I'll look into them, but I suspect that something simply went wrong with it. Upgrade Status usually finds a bunch of deprecated, but inactive code that happens to still be in the module. If you decide to try again, then you can ignore much of what it finds.
Comment #6
dcam commentedThe dev branch is dependent on core ^8.8.
Comment #8
dcam commentedThe version compatibility has been updated and all tests are passing on D9.