Ability to create a menu item with the config page

Here is a patch, which creates menu links.

I confirm that this patch works for me.

Thx for this patch, already committed to dev version, so will be available with new release.

Strange... The issue is not with how the link is created, it has to do it with how you clear cache. If you run drush cr, then hook_menu_links_discovered_alter is not called, hence menu link is not created. But if you clear cache inside an administration, then this hook is called and menu link is created.

hook_menu_links_discovered_alter is the right way to do it, as this is one of the options written in drupal documentation "How to add dynamic menu links". And the other one is, as you said, with using deriver class in *.links.menu.yml file. I tested both options, but if you rebuild cache with drush menu links are not created.

Hello,

I have the same problem. The menu links are not created. The parent is not detected.

I confirm that when using drush cr, the hook_menu_links_discovered_alter is triggered

On the line:

$route = \Drupal::service('path.validator')->getUrlIfValid($path);

When in the execution of "drush cr", no route is found, but when executing using devel/php, it founds the right route.

I can't reproduce this issue in my local environment. @Grimreaper or @rodrigoaguilera anyone of you still able to reproduce it? If so, please share STR with me. Thx.

Fixed.

I found a bug related to this commit.

If you create a custom path, let's say /admin/config/cerema/activites, for a config page, then if you flush routing and links cache (with the custom link provided by admin_toolbar_tools), you've got a fatal error.

Le site Web a rencontré une erreur inattendue. Veuillez essayer de nouveau plus tard.</br></br><em class="placeholder">Symfony\Component\Routing\Exception\ResourceNotFoundException</em>: No routes found for &quot;/admin/flush/menu&quot;. in <em class="placeholder">Drupal\Core\Routing\Router-&gt;matchRequest()</em> (line <em class="placeholder">159</em> of <em class="placeholder">core/lib/Drupal/Core/Routing/Router.php</em>). <pre class="backtrace">Drupal\Core\Routing\Router-&gt;match(&#039;/admin/config/cerema&#039;) (Line: 162)
config_pages_get_route_name_from_path(&#039;/admin/config/cerema&#039;) (Line: 150)
config_pages_find_parent_route_name(&#039;/admin/config/cerema/activites&#039;) (Line: 126)
config_pages_menu_links_discovered_alter(Array, NULL, NULL) (Line: 501)
Drupal\Core\Extension\ModuleHandler-&gt;alter(&#039;menu_links_discovered&#039;, Array) (Line: 167)
Drupal\Core\Menu\MenuLinkManager-&gt;getDefinitions() (Line: 190)
Drupal\Core\Menu\MenuLinkManager-&gt;rebuild() (Line: 129)
Drupal\admin_toolbar_tools\Controller\ToolbarController-&gt;flush_menu()
call_user_func_array(Array, Array) (Line: 123)
Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber-&gt;Drupal\Core\EventSubscriber\{closure}() (Line: 576)
Drupal\Core\Render\Renderer-&gt;executeInRenderContext(Object, Object) (Line: 124)
Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber-&gt;wrapControllerExecutionInRenderContext(Array, Array) (Line: 97)
Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber-&gt;Drupal\Core\EventSubscriber\{closure}()
call_user_func_array(Object, Array) (Line: 153)
Symfony\Component\HttpKernel\HttpKernel-&gt;handleRaw(Object, 1) (Line: 68)
Symfony\Component\HttpKernel\HttpKernel-&gt;handle(Object, 1, 1) (Line: 57)
Drupal\Core\StackMiddleware\Session-&gt;handle(Object, 1, 1) (Line: 47)
Drupal\Core\StackMiddleware\KernelPreHandle-&gt;handle(Object, 1, 1) (Line: 99)
Drupal\page_cache\StackMiddleware\PageCache-&gt;pass(Object, 1, 1) (Line: 78)
Drupal\page_cache\StackMiddleware\PageCache-&gt;handle(Object, 1, 1) (Line: 50)
Drupal\ban\BanMiddleware-&gt;handle(Object, 1, 1) (Line: 47)
Drupal\Core\StackMiddleware\ReverseProxyMiddleware-&gt;handle(Object, 1, 1) (Line: 50)
Drupal\Core\StackMiddleware\NegotiationMiddleware-&gt;handle(Object, 1, 1) (Line: 23)
Stack\StackedHttpKernel-&gt;handle(Object, 1, 1) (Line: 657)
Drupal\Core\DrupalKernel-&gt;handle(Object) (Line: 19)
</pre>

Because the parent /admin/config/cerema/ do not correspond to any existing routes.

As a workaround I added a custom route which correspond to the parent path.

Should we reopen this issue or create a new one ?

I think that it would be better to create a new issue.