I ran into some issues trying to get the D8 version to work.
While through the GeoJsonExport class fixing the errors that were arising, it became clear that it was essentially a copy of the RestExport class with some modifications.
I found success by changing it to just derive from the RestExport class itself (instead of their common PathPluginBase ancestor) and making just a couple small modifications.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | views_geojson-use_RestExport-2943351-5.patch | 8.76 KB | jeffschuler |
| #4 | views_geojson-use_RestExport-2943351-4.patch | 7.31 KB | jeffschuler |
| #3 | views_geojson-use_RestExport-2943351-2.patch | 9.53 KB | jeffschuler |
| #2 | views_geojson-use_RestExport-2943351-1.patch | 9.47 KB | jeffschuler |
Comments
Comment #2
jeffschulerComment #3
jeffschulerFor Drupal 8.5.x compatibility:
Add
$serializer_format_providerstoGeoJsonExportclass constructor (to be in-line withRestExport).Comment #4
jeffschulerRe-rolled for latest 8.x-1.x-dev, mostly to account for changes from #3083020: Cannot use object of type Drupal\Core\Cache\CacheableResponse as array. Still just deleting most of the class implementation, as it's just code copied from core's RestExport class.
Comment #5
jeffschulerFixes previous.
Comment #7
jeffschulerCommit: 29121e1
Comment #8
jeffschuler