According to Drupal standards, modules should include dependencies in the .info.yml file.
Dependencies should be namespaced in the format {project}:{module}, where {project} is the project name as it appears in the Drupal.org URL (e.g. drupal.org/project/views) and {module} is the module's machine name.

https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-...

Patch to follow, thanks!

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

Assigned: volkswagenchick » Unassigned
Status: Active » Needs review
StatusFileSize
new360 bytes

I uploaded a patch that aligns the dependency namespacing to Drupal standards. Thanks!

volkswagenchick’s picture

Assigned: Unassigned » volkswagenchick
Status: Needs review » Needs work

I just noticed an empty space. patch forthcoming!

volkswagenchick’s picture

Assigned: volkswagenchick » Unassigned
Status: Needs work » Needs review
StatusFileSize
new329 bytes

Okay updated the patch. Please review

volkswagenchick’s picture

StatusFileSize
new0 bytes

ok - This one is really the right one :)

volkswagenchick’s picture

StatusFileSize
new319 bytes

ugh. ok - let's try this one last time :)

alonaoneill’s picture

Status: Needs review » Reviewed & tested by the community

Code looks good. The dependency namespacing aligns with Drupal documentation standards.
Patch applied via simplytest.me
Marking as RTBC
Thank you for working on module!

chipway’s picture

Assigned: Unassigned » chipway
Status: Reviewed & tested by the community » Needs work

Patch applies but it lacks a space.

chipway’s picture

Assigned: chipway » Unassigned
Status: Needs work » Needs review
StatusFileSize
new320 bytes

Here is the new patch. Please review.

volkswagenchick’s picture

Status: Needs review » Reviewed & tested by the community

The code looks good. The dependency namespacing was addressed. Patch applied using simplytest.me

Marking RTBC, and thanks for the fix @chipway. Good catch.

sanduhrs’s picture

Version: 8.x-1.x-dev » 2.0.x-dev

  • sanduhrs committed 401a708 on 8.x-1.x authored by chipway
    Issue #3005175 by volkswagenchick, chipway: Dependency namespacing in ....

  • sanduhrs committed d322802 on 2.0.x authored by chipway
    Issue #3005175 by volkswagenchick, chipway: Dependency namespacing in ....
sanduhrs’s picture

Status: Reviewed & tested by the community » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

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

sanduhrs’s picture

Status: Closed (fixed) » Fixed

Status: Fixed » Closed (fixed)

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