-
Notifications
You must be signed in to change notification settings - Fork 0
GNIP 12 Tagging (with Migration)
It is proposed to migrate the current keywords implementation in the python (django) portion of GeoNode to use django-taggit in order to allow for tagging of _any_ model object used in the system in a flexible and extensible way and to allow for inspection of the entire set of tags as whole rather than scatted across models/tables.
Initial work has been done under this proposal by Eldarion.
https://github.com/eldarion/geonode/tree/add-tagging
Jeffrey Johnson
1.2
Accepted and merged.
Currently GeoNode handles keywords or tags for the Layer model with a TextField https://github.com/GeoNode/geonode/blob/master/src/GeoNodePy/geonode/maps/models.py#L691 ... These keywords are parsed and synchronized with GeoServer and the CSW backend (currently GeoNetwork). The keywords are stored in the database as a comma or space separated list and its not easy to query them. Other model classes like the Map do not currently have keywords, but this kind of functionality has been requested many times.
It is proposed to use django-taggit [1] to provide flexible tagging functionality inside GeoNodePy and to add TagFields to the Layer, Map and Profile classes and to provide a way of migrating sites using the old implementation to the new one.
[1] https://github.com/alex/django-taggit
Migrations (See GNIP 19) ...
The new implementation must still properly sync the tags to GeoServer and GeoNetwork (or other csw backend) which requires that the existing code that does this be tested to make sure that it still works.
Existing tests that test the keyword synchronization must pass and any new tests necessary to demonstrate that the new implementation works should be added to either/or the unit test or integration test suites.
Do nothing. GeoNodes current keyword implementation works as is.
See discussion on the geonode-dev list here.
https://groups.google.com/a/opengeo.org/group/geonode-dev/browse_thread/thread/682c77437f996e0a
Ariel +1 - (This is a post-facto +1 the branch in question was already pulled into master last week)