Setting ckan.search.solr_allowed_query_parsers in ini results in error

[bzar]

https://github.com/ckan/ckanext-spatial/blob/master/ckanext/spatial/plugin/__init__.py#L165 assumes that config["ckan.search.solr_allowed_query_parsers"] contains a list, but if it is set in ini file (and not by the above default value) it contains a string, causing an error. Also if the string already contains the value in qp, adding a duplicate is unnecessary.

[amercader]

@bzar I assume this is pre CKAN 2.10? Otherwise the config declaration parsing would have enforced a list.
But you are right that the code should check if the value is already on the list

[bzar]

Yes, in CKAN 2.9.

[amercader]

The latest spatial version doesn't support 2.9 officially but I'll be happy to merge a PR that makes the handling of this conf option work better on it