
All dependencies must be prefixed by project name, So please apply new {project}:{module} format for dephttps://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-... in info.yml filehttps://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-about-your-module-with-an-infoyml-file
It is supported since 8.0 and 7.40 Change Record: Project namespaces can now be added for module dependencies, and is now a Best Practice Define project dependencies in core module .info files).
It is useful for Installation profiles to work well. So sooner is better.
Hope this helps.
Comment | File | Size | Author |
---|---|---|---|
#3 | Translatable menu link uri.png | 21.48 KB | zalak.addweb |
#2 | Update_dependencies_format_in_info.yml_3003431-2.patch | 553 bytes | chipway |
Comments
Comment #2
chipway commentedHere is the patch. Please review.
I also also capitalized module name per coding standards.
Comment #3
zalak.addweb commented@chipway, Your patch is not working for me showing an error while using it in my Local & Simplytest.me. Can you please help me with the same?
Comment #4
chipway commentedThanks Pranjal,
For the patch applies cleanly and do expected changes.
git clone --branch 8.x-1.x https://git.drupal.org/project/translatable_menu_link_uri.git
wget https://www.drupal.org/files/issues/2018-10-01/Update_dependencies_forma...
cd translatable_menu_link_uri/
git apply -v ../Update_dependencies_format_in_info.yml_3003431-2.patch
Vérification du patch translatable_menu_link_uri.info.yml...
Patch translatable_menu_link_uri.info.yml appliqué proprement.
Comment #6
jsobiecki commentedPatch applied cleanly for me, and it's working fine.
Not big fan of namespacing in such global context, but it doesn't hurt :)
Change merged and commited to 1.x branch.
Thanks!