Per https://www.drupal.org/node/2000204, specify module dependencies in project:module form, a la:

dependencies:
  - datetime:datetime
  - link:link
  - drupal:views
CommentFileSizeAuthor
#4 use_namespaced_dependencies_2795829_4.patch908 byteslomasr
#2 2795829_2.patch425 byteslomasr

Comments

mikeryan created an issue. See original summary.

lomasr’s picture

lomasr commented
StatusFileSize
new 2795829_2.patch425 bytes

As mentioned in the description have used namespaced dependencies. In the comments in wordpress_migrate.info.yml file .@todo
https://www.drupal.org/node/2560795 is mentioned , which is fixed now , So I have removed the dependency on migrate_drupal.Please review the patch Thanks.

lomasr’s picture

lomasr commented
Status: Active » Needs review
lomasr’s picture

lomasr commented
StatusFileSize
new use_namespaced_dependencies_2795829_4.patch908 bytes

Adding a patch for 'WordPress migration support' & 'WordPress Migration UI' Modules.

Thanks

hongpong’s picture

hongpong commented
Status: Needs review » Reviewed & tested by the community

This works correctly. I rolled this patch and the other longstanding approved ones on the issue queue here, if you'd like to try it: https://github.com/HongPong/wordpress_migrate
I would request this get added. (note it overlaps with #2742271: Remove dependency on migrate_drupal - this one has the right namespacing on the pertinent line though.)

  • HongPong committed ca02690 on 8.x-3.x authored by lomasr 
    Issue #2795829 by lomasr, HongPong: Use namespaced dependencies in .info...
hongpong’s picture

hongpong commented
Status: Reviewed & tested by the community » Fixed

With tweaking to account for the other patch it is now rolled in. Thank you lomasr :)

lomasr’s picture

lomasr commented

:)

Status: Fixed » Closed (fixed)

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