Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
system_config_form() function has been removed in Drupal 8 instead it is necessary to use Use \Drupal\system\SystemConfigFormBase or system_config_form(). Reference page - https://drupal.org/node/1910694
Comment | File | Size | Author |
---|---|---|---|
#15 | admin_menu-convert-config-form-2012194-15.patch | 10.14 KB | kerby70 |
Comments
Comment #1
sergeypavlenko CreditAttribution: sergeypavlenko commentedReplaced the function.
Comment #2
dmouse#1 the system_config_form has removed.
I'm working on this
Comment #3
jibranI have created a fork of admin_menu 8.x-3.x on github. Please help me fix it for Drupal 8
PRs are welcome. https://github.com/jibran/admin_menu
Comment #4
kerby70 CreditAttribution: kerby70 at Blink Reaction (now part of FFW) commentedAttached is a patch converting the form builder system_settings_form to the new form class. This lines up with the changes proposed in #2402185: Convert hook_menu() to routing system https://www.drupal.org/node/2402185#comment-9689639
The variable names have been shortened and placed under the $config = $this->config('admin_menu.admin_config'); since variable_get() has been removed and I am not so sure the use of system.admin_menu is appropriate from #2.
Comment #5
kerby70 CreditAttribution: kerby70 commentedComment #6
kerby70 CreditAttribution: kerby70 commentedComment #7
kerby70 CreditAttribution: kerby70 at Blink Reaction (now part of FFW) commentedI am looking into include the admin_menu.settings.yml and changes to the 'admin_menu.settings' naming instead of 'admin_menu.admin_config' for the settings.
Comment #8
kerby70 CreditAttribution: kerby70 at Blink Reaction (now part of FFW) commentedAttached are the changes including admin_menu.settings.yml. Included are two additional fixes resolving the change of module_exist() and the other url(). And fixed a bug in my previous for save arrays.
Comment #9
kerby70 CreditAttribution: kerby70 commentedComment #10
kerby70 CreditAttribution: kerby70 at Blink Reaction (now part of FFW) commentedWhile reviewing #2401457: replace system variables with configuration API I realized I am duplicating the admin_menu.settings.yml that is already there. I am removing that section from my changes.
Comment #11
kerby70 CreditAttribution: kerby70 at Blink Reaction (now part of FFW) commentedMy last patches didn't capture the correct changes and both #8 and #10 had lost the form.
Attached are the changes including the two additional fixes resolving the change of module_exist() and the other url(). And fixed a bug in [#4] form save arrays. It does not include the settings.yml which is included in #2401457: replace system variables with configuration API
Comment #12
kerby70 CreditAttribution: kerby70 at Blink Reaction (now part of FFW) commentedRemoving /module/admin_menu from the file path of #11.
Comment #13
kerby70 CreditAttribution: kerby70 commentedComment #14
kerby70 CreditAttribution: kerby70 commentedComment #15
kerby70 CreditAttribution: kerby70 commentedRealized the form call in #2402185: Convert hook_menu() to routing system was inline with the core modules calling ModuleSettingsForm instead of ModuleConfigForm. Here is that change.
Comment #17
Andrew.Mikhailov CreditAttribution: Andrew.Mikhailov at DrupalJedi commentedI try to fix this issue asap.
Best regards.
Comment #18
Andrew.Mikhailov CreditAttribution: Andrew.Mikhailov at DrupalJedi commentedComment #20
thallesThis module has been deprecated for Drupal 8.
For Drupal 8: the Admin Toolbar provides an experience similar to admin_menu with the core toolbar.
Comment #21
thalles