No update of CRS cache when modifying a custom CRS

[qgib]

Author Name: Leyan Ouyang (Leyan Ouyang)
Original Redmine Issue: 8058
Affected QGIS version: master
Redmine category:projection_support
Assignee: Marco Hugentobler

---

In 1.8, if I update a custom CRS, I can directly use the new value for this CRS, while in master, I have to restart QGIS to be able to use the updated parameters. They are displayed when I choose the CRS, but not used during the actual transformation, which is a severe regression in my opinion as it makes the whole custom CRS management very confusing.

This is due to the recent use of a cache to store CRS parameters (since commit 8dc0135). There is currently no way to update the cache, short of restarting QGIS.

I tried to write a simple updateCRSCache function, but it isn't giving the expected result. The CRS seems updated in the debug output, but nothing is redrawn. Here is what I wrote so far: https://github.com/leyan/Quantum-GIS/commits/custom_CRS

[qgib]

Author Name: Marco Hugentobler (@mhugent)

---

The updateCRSCache function is fine. However, there is also a crs transform cache that copies the crs and that needs to be updated as well (sorry, the crs / transform cache have mainly been made with the server usage in mind...). It should now work after 4061445, please test.

[qgib]

Author Name: Leyan Ouyang (Leyan Ouyang)

---

This works for me. Actually the behavior is not exactly the same, as the layers are reprojected automatically as soon as they are redrawn. Previously, the CRS had to be selected again for each layer before the new parameters can be used. I think the new behavior is better and that is a change I was planning to add anyway.

I added a pull request to deal with deleted custom CRS: #661

The whole custom CRS management still is a bit flaky, with a bad UI (I know, I did it ;) ) and a lot of things that should be more robust. Anyway, this was already the case in 1.8 so no regression there, I guess improvements will have to wait for the next version.

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

can this be closed?

---

• status_id was changed from Open to Feedback

[qgib]

Author Name: Marco Hugentobler (@mhugent)

---

I added a pull request to deal with deleted custom CRS

Applied. It would also be possible to just delete the crs cache entry for update and delete (if the crs is not cached, the entry will be created on next usage).

The whole custom CRS management still is a bit flaky, with a bad UI (I know, I did it ;) ) and a lot of things that should be more >robust. Anyway, this was already the case in 1.8 so no regression there, I guess improvements will have to wait for the next version.

I agree. It will be greatly appreciated if you improve custom crs handling and ui for the next version.

Thanks for your input and patches,
Marco

[qgib]

Author Name: Leyan Ouyang (Leyan Ouyang)

---

Yes, can be closed for me, the main issue is solved.

---

• status_id was changed from Feedback to Closed