For school I needed to commit and upgrade for a drupal 7 module to drupal 8. I choose this module. Thanks alot.

Files: 
CommentFileSizeAuthor
menu_hide_disabled_parents_drupal8.zip8.71 KBLucasP

Comments

LucasP created an issue. See original summary.

LucasP’s picture

Issue summary: View changes
jiv_e’s picture

Hi, LucasP!

Thank you for your contribution! What kind of criteria do you have to get your assignment accepted? I may be able to help you if you have any issues.

Have you tested your code works? I noticed that you are using hook_translated_menu_link_alter in the .module file. It is not a part of Drupal 8.

See: https://api.drupal.org/api/drupal/modules!system!system.api.php/function...
See also: http://drupal.stackexchange.com/questions/209017/d8-what-is-the-equivale...

Please fix the code to use Drupal 8 API! After this you should test it actually works. I suppose it doesn't work at the moment.

Best regards,
Juho

LucasP’s picture

Hi

First of all I want to thank you for your reply. The code I uploaded works on drupal 8 site bu it indeed contains code that is forbidden. I made a RouteSubscriber.php to replace the hook_translated_menu_link_alter. But is the right way of fixing it?

Thanks alot!

Lucas P