Leaflet GeoJSON is currently hard coded to look for the views argument bbox_argument which is fine for default views. I'm working on getting this to work with search_api solr and we need a different views argument handler. Patch coming soon.

CommentFileSizeAuthor
#1 2061629-1-views_argument_bbox.patch1.06 KBrandallknutson
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

randallknutson’s picture

randallknutson CreditAttribution: randallknutson commented
FileSize
2061629-1-views_argument_bbox.patch1.06 KB

Here is a patch that looks for "bbox" in the views argument handler name instead of hard coding it.

randallknutson’s picture

randallknutson CreditAttribution: randallknutson commented

See #2061639: Add views argument for geojson bbox for where this is used.

randallknutson’s picture

randallknutson CreditAttribution: randallknutson commented
Status: Active » Needs review

Changing status.

randallknutson’s picture

randallknutson CreditAttribution: randallknutson commented

Also see: #1641870: Bounding Box support with Search API views

  • Commit 9a186fc on 7.x-2.x authored by randallknutson, committed by mpgeek: 
    Issue #2061629 by randallknutson: allow other arguments to provide bbox...
mpgeek’s picture

mpgeek CreditAttribution: mpgeek commented
Issue summary: View changes
Status: Needs review » Closed (fixed)