Hi!

I had a problem with taxonomy_place's dependencies.

Long story short, not specifying the project fixes it:

...
dependencies:
- address
- taxonomy
...

From what I've read, the project should be specified in the .info.yml file, but eh... removing it makes it work so...

CommentFileSizeAuthor
#8 3030130-add-geolocation-to-info.patch448 bytesKarenS
#7 errors_in_dependencies_3030130_7.patch359 bytesbrooke_heaton
#6 composer.json_.patch342 bytesjoy29
#6 3030130-5.patch342 bytesjoy29
#2 3030130-2.patch357 bytesveerabhadra
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ncg777 created an issue. See original summary.

veerabhadra’s picture

veerabhadra CreditAttribution: veerabhadra at Valuebound commented
Status: Active » Needs review
FileSize
3030130-2.patch357 bytes

Apply this patch. It will work fine.

ncg777’s picture

ncg777 CreditAttribution: ncg777 commented

Indeed... It works.

brooke_heaton’s picture

brooke_heaton CreditAttribution: brooke_heaton as a volunteer commented

Furthermore, there is a missing composer.json file which should stipulate a dependency on the geolocation module.

joy29’s picture

joy29 CreditAttribution: joy29 commented
Assigned: Unassigned » joy29
joy29’s picture

joy29 CreditAttribution: joy29 commented
FileSize
3030130-5.patch342 bytes
composer.json_.patch342 bytes
brooke_heaton’s picture

brooke_heaton CreditAttribution: brooke_heaton as a volunteer commented
FileSize
errors_in_dependencies_3030130_7.patch359 bytes

we also need to add geolocation as a dependency in the .info file or install will fail.

https://cgit.drupalcode.org/taxonomy_place/tree/config/install/core.enti...

dependencies:
config:
- field.field.taxonomy_term.place.field_geolocation
- field.field.taxonomy_term.place.field_place
- field.field.taxonomy_term.place.field_short_name
- field.field.taxonomy_term.place.field_sortable_name
- taxonomy.vocabulary.place
module:
- address
- geolocation <----------------------------------------------------- This is a dependency
- path
- text

KarenS’s picture

KarenS CreditAttribution: KarenS at Lullabot commented
FileSize
3030130-add-geolocation-to-info.patch448 bytes

That patch does not use the correct format for dependencies, I re-rolled it.

  • KarenS committed 818b991 on 8.x-1.x authored by brooke_heaton 
    Issue #3030130 by joy29, brooke_heaton, KarenS, veerabhadra, ncg777:...
KarenS’s picture

KarenS CreditAttribution: KarenS at Lullabot commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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