Problem/Motivation

We found ourselves needing this module to be Drupal 9 compatible, Drupal 8's eol is near November 2nd 2021.
We run Update Status to see the minimum needed to support that:

FILE: modules/contrib/micro_menu/src/Form/SiteMenuForm.php
 
STATUS         LINE                           MESSAGE
--------------------------------------------------------------------------------
Fix now        71   Call to deprecated method url() of class
                    Drupal\Core\Form\FormBase. Deprecated in drupal:8.0.0 and is
                    removed from drupal:9.0.0. Use Drupal\Core\Url::fromUri()
                    instead.
--------------------------------------------------------------------------------
Fix now        72   Call to deprecated method url() of class
                    Drupal\Core\Form\FormBase. Deprecated in drupal:8.0.0 and is
                    removed from drupal:9.0.0. Use Drupal\Core\Url::fromUri()
                    instead.
--------------------------------------------------------------------------------
 
FILE: modules/contrib/micro_menu/tests/src/Functional/LoadTest.php
 
STATUS         LINE                           MESSAGE
--------------------------------------------------------------------------------
Check manually 13   Drupal\Tests\BrowserTestBase::$defaultTheme is required. See
                    https://www.drupal.org/node/3083055, which includes
                    recommendations on which theme to use.
--------------------------------------------------------------------------------
 
FILE: modules/contrib/micro_menu/micro_menu.info.yml
 
STATUS         LINE                           MESSAGE
--------------------------------------------------------------------------------
Check manually 0    Add core_version_requirement: ^8 || ^9 to designate that the
                    module is compatible with Drupal 9. See
                    https://drupal.org/node/3070687.
--------------------------------------------------------------------------------

Proposed resolution

Create a patch containing the above recommendations.

I'll try to do it soon!

CommentFileSizeAuthor
#2 micro_menu-drupal-9-compatibility-for-alpha-4-do-not-test.patch2.07 KBcurriedn
#2 micro_menu-drupal-9-compatibility-3233088-2.patch1.86 KBcurriedn

Comments

CurriedN created an issue. See original summary.

curriedn’s picture

curriedn commented
StatusFileSize
new micro_menu-drupal-9-compatibility-3233088-2.patch1.86 KB
new micro_menu-drupal-9-compatibility-for-alpha-4-do-not-test.patch2.07 KB

Here is the patch! Hope it helps :)

flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
commented

Thanks

  • flocondetoile authored b276088 on 8.x-1.x 
    Issue #3233088 by CurriedN, flocondetoile: Drupal 9 compatibility
flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
commented
Status: Active » Fixed

Thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.