All dependencies must be prefixed by project name, So please apply new {project}:{module} format for dependencies in info.yml 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 DrupalCi to download the right dependencies and Installation profiles to work well. So sooner is better.
Hope this helps.

Comments

chipway created an issue. See original summary.

chipway’s picture

Title: Namespace Dependencies in .info.yml » Namespace Dependencies in .info.yml + capitalize
Status: Needs work » Needs review
StatusFileSize
new580 bytes

Here is the patch. Please review.
I also capitalized some module name per coding standards.

chipway’s picture

Assigned: chipway » Unassigned
Aanal.addweb’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new31.29 KB
new29.11 KB
new29.34 KB

@chipway, Thanks for providing the patch. Your patch is working fine for me. As per the mentioned changes, I matched it with Pareview and tested in simplytest.me over Extend page. PFA.

dakwamine’s picture

Hello,

What is the correct fix for the info file?

There is #2895949: field_ui module isn't dependence which fixes the dependency, but does not prefix the dependency with the drupal namespace.

dakwamine’s picture

Assigned: Unassigned » dakwamine
dakwamine’s picture

Rolliroll.

  • Dakwamine authored 50a77f8 on 8.x-2.x
    Issue #3011712 by chipway, Dakwamine, Aanal.addweb: Namespace...
dakwamine’s picture

Commited. Thanks for fix and testing.

dakwamine’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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