Q: The project needs "date_api" to work but 'm unable to find it in drupal.org website.
A: It is part of the Date module.

CommentFileSizeAuthor
#3 more_specific_deps-2450191-3.patch432 bytesdddbbb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented

It is part of the Date module.

johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented
Status: Active » Closed (works as designed)
dddbbb’s picture

dddbbb CreditAttribution: dddbbb as a volunteer commented
Status: Closed (works as designed) » Needs review
FileSize
more_specific_deps-2450191-3.patch432 bytes

Installing this module via drush doesn't always work because of the way it lists its dependencies (if you don't have the date module already downloaded, drush won't be able to get it for you). This patch should fix that.

dddbbb’s picture

dddbbb CreditAttribution: dddbbb as a volunteer commented
Title: Project Date-Api is a requirement but unable to find it » Be more specific when listing dependencies so that drush en can work
Version: 7.x-1.4 » 7.x-1.x-dev
dddbbb’s picture

dddbbb CreditAttribution: dddbbb as a volunteer commented
Title: Be more specific when listing dependencies so that drush en can work » Be more specific when listing dependencies so that `drush en office_hours` can work
dddbbb’s picture

dddbbb CreditAttribution: dddbbb as a volunteer commented

More info: https://www.drupal.org/node/2299747

johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented

O wow, I didn't know this notation.
There was an issue once from someone who couldn't fine the date_api module. This is better for drush and humans!

dddbbb’s picture

dddbbb CreditAttribution: dddbbb as a volunteer commented

:) This is that issue. I changed the title.

johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented

O ja. :-)

  • johnv committed d6ec08a on 7.x-1.x authored by danbohea 
    Issue #2450191 by danbohea: Be more specific when listing dependencies...
johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented
Version: 7.x-1.x-dev » 7.x-1.4
Category: Support request » Bug report
Issue summary: View changes
Status: Needs review » Fixed

Committed. Thanks.

johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented

ATM this is not relevant for the D8-version, since date API is not ported, and the relevant code is copied into the module.

  • johnv committed 9377182 on 7.x-1.x authored by danbohea 
    Issue #2450191 by danbohea: Be more specific when listing dependencies...
johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented
Related issues: +#2607848: Date API "missing"

Apparently, this doesn't work. See #2607848: Date API "missing" The notation is also not menioned on this manual page:
https://www.drupal.org/node/542202#dependencies

The adjacent patch adds an extra dependency on the 'date' module, which is not stricly necessary.

Status: Fixed » Closed (fixed)

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

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
CreditAttribution: volkswagenchick as a volunteer commented

I tried to download this module recently, and being new to drupal, it would have been nice to have documentation that stated the dependencies. I saw that there is a patch, but it is not mentioned on the Office Hours page.

Thankyou

johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented

I updated the project page.
Thanks for the feedback. Hope you enjoy this module.