search

linz / QGIS-AIMS-Plugin

QGIS Plugin for managing LINZ address information
BSD 3-Clause "New" or "Revised" License
3 stars 4 forks source link

Proposed addresses can not be updated #213

Open SPlanzer opened 6 years ago

SPlanzer commented 6 years ago

as per https://github.com/linz/QGIS-AIMS-Plugin/blob/dd04fe167603b41e1a91eabe389e7b090e01c117/AimsUI/AimsClient/Gui/UiUtility.py#L382

proposed address can not be updated. There is a flaw in the logic here. This check can simply be removed providing it is okay to switch addresse from "Current" to "Proposed"

Deb-Jones commented 6 years ago

During AIMS development I asked to be able to change an address from current to proposed, but e-Spatial said that this would corrupt the data integrity. I would have thought however that a limitation such as this would belong in the API rather than the UI. I would like to know the impact on data of editing an address from current to proposed.

SPlanzer commented 6 years ago

Hi @Deb-Jones. I can not recall why this check was added, so your comment helps makes some sense of this bug. I knew there had to be a reason for the above check. It is the sort of thing we tried to avoid hard coding into the ui many times.

To get an answer to your question you will need to direct this at eSpatial