Closed (fixed)
Project:
Node Link Report
Version:
8.x-1.20
Component:
Documentation
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
30 Jul 2025 at 10:10 UTC
Updated:
30 Jul 2025 at 15:18 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
jparmar commentedYes aggree to declare Drupal-level (runtime) dependency — so that Drupal knows to enable that dependency module when enabling your module.
Added patch for it.
Comment #3
jparmar commentedComment #4
bhogue commentedI can confirm that after applying the patch, the dependency is noted on the "extend" page and when installing the module, the modules it depends upon are also installed.
Comment #7
swirtThanks sirclickalot, jparmar, and bhogue for all your work on this and contributing. Unfortunately it took the problem in the wrong direction. node_link_report does not use views_data_export. It was mistakenly added to the composer.json as a result of copy and paste.
I created an MR to remove it and will merge that shortly.
The nice thing about MR rather than patches is that they generate tugboat preview instances so it makes it really quick to review things on vanilla installs.
Comment #9
swirtComment #10
swirtComment #11
swirtReleased as 8.x-1.21