There are schema errors for geocoder_field:
- weight: missing
- reverse_geocode_field: invalid type (should be sequence)
- hidden: missing
- disabled: missing

CommentFileSizeAuthor
#5 invalid-geocoder_field-schema-2971970-5-D8.patch853 bytesszato
#2 invalid-geocoder_field-schema-2971970-2-D8.patch922 bytesszato
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

szato created an issue. See original summary.

szato’s picture

szato CreditAttribution: szato at Brainsum for Tieto commented
Status: Active » Needs review
FileSize
invalid-geocoder_field-schema-2971970-2-D8.patch922 bytes
itamair’s picture

itamair CreditAttribution: itamair as a volunteer commented

I can agree that those indexes (weight, hidden, disabled) in the scheme are missing.
But:
- weight.label shouldn't be called Geocode method, right?
- should the geocode_field be sequence so as the reverse_geocode_field needs to be?

Sorry, but may you carefully correct your patch please?

itamair’s picture

itamair CreditAttribution: itamair as a volunteer commented
Status: Needs review » Needs work
szato’s picture

szato CreditAttribution: szato at Brainsum for Tieto commented
Issue summary: View changes
Status: Needs work » Needs review
FileSize
invalid-geocoder_field-schema-2971970-5-D8.patch853 bytes

Thank you for your review, you are right.
Label and type (string is correct) fixed, patch attached.

  • itamair committed 460c11d on 8.x-2.x authored by szato 
    Issue #2971970 by szato: Invalid geocoder_field schema
itamair’s picture

itamair CreditAttribution: itamair as a volunteer commented
Status: Needs review » Fixed

You are welcome @szato ... you patch added to dev.
Thank for your contribution

Status: Fixed » Closed (fixed)

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