Opened 13 years ago
Last modified 11 years ago
#18654 new New feature
Support for model properties that are not fields in LayerMapping
Reported by: | Melvyn Sopacua | Owned by: | nobody |
---|---|---|---|
Component: | GIS | Version: | dev |
Severity: | Normal | Keywords: | LayerMapping |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | yes |
Easy pickings: | no | UI/UX: | no |
Pull Requests: | How to create a pull request | ||
Description ¶
The GIS LayerMapping does explicit testing for model fields and has logic to support Foreign keys. However, this does not allow supporting the reverse relationship of foreign keys or to support fields that need some kind of translation or need to be spread across different fields/models (f.e. when a shape file contains compound fields).
The patch provided allows for 'class properties' to be filled by the layermap and contains a test for the concrete use case that prompted this enhancement.
According to the ticket's flags, the next step(s) to move this issue forward are:
- To improve the patch as described in the pull request review comments or on this ticket, then uncheck "Patch needs improvement".
If creating a new pull request, include a link to the pull request in the ticket comment when making that update. The usual format is:
[https://github.com/django/django/pull/#### PR]
.
Change History (3)
by , 13 years ago
Attachment: | django.contrib.gis.property_fields.patch added |
---|
comment:1 by , 12 years ago
Triage Stage: | Unreviewed → Accepted |
---|---|
Type: | Uncategorized → New feature |
Add support for properties in LayerMap (fixed indentation)