Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Really minor issue, I doubt many people using this module wouldn't already have Date installed, but I didn't, this time.
I used Drush to install this, and the dependencies called for date_popup, but since that is a module within Date, it isn't directly found and there might be some poor, lost soul out there not familiar with the Date module to know where to find date_popup. Might be a good idea to include Date in the dependencies.
Comments
Comment #1
Mark TrappI took a look at how core handles dependencies of dependencies, and it only specifies the immediate dependency (e.g., comment.info only specifies text as a dependency, not field), so I'm classifying this a documentation task rather than a bug report.
I'm tempted to close this, though: both the project page and the README state Date Popup is part of Date, thus making Date required. I feel like it's pretty clear, but if there's a way to make it clearer, I'd be open to improving it.
Comment #2
Mark TrappNo movement on this in 2 weeks, so I'm closing this as "cannot reproduce" as I think the dependency on Date is already pretty clear. If there is a better way to express the dependency in the documentation, please feel free to re-open this.