Closed phptek closed 5 years ago
The code as it is is working for me. I can't reproduce the issue.
The suggested change would add the $summary_fields to all inheriting service implementations, even if they don't need it. This argument is rather weak because the design is far from being clean and abstract, but rather pragmatic.
getConfig() would never return false (default) and all DataObjects would be exposed through all service implementations.
The basic idea behind private static $geojsonservice
is to work like RestfulServer's private static $api_access
(and this is not spelled out in the docs): It is turned off by default. You can turn it on with default settings by just setting it to true. Or you can give it an associative array with all the custom config you want. I am working on rewriting and restructuring the docs to explain this better.
Maybe I can write a unit test to cover this bit.
Can no longer reproduce with e6a484181897d443
or 0372211b9069
closing.
property_map
is accounted for.getConfig()
including consistent return type and PSR2.