-
-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[bugfix] Attribute table model not updated when a feature is changed. fixes #17280 #17187 #5403
[bugfix] Attribute table model not updated when a feature is changed. fixes #17280 #17187 #5403
Conversation
please @pblottiere you are the best to review this backport ;) |
I have just tested the patch and indeed it fixes an issue happening on the first row of the attribute table in 2.18 👍. However, I'm not sure that this PR fully fixes the issue https://issues.qgis.org/issues/17187. In fact, the ticket has been opened 25 days ago and:
However, the original PR is merged in master for a while now. So it seems that even with this patch, the issue #17187 is still happening on master. What do you think? |
I'll check better 17187... in the meantime I removed it from the title and description |
@pblottiere in the meantime we can merge this that for sure fix 17280. what do you think? |
@pblottiere Hi, I tested the patch applying it to 2.18-dev code, and checked both #17187 and #17280, both seems fixed to me now. |
@pblottiere may you merge? (I can't) |
Done. Thank you for this patch @luipir! |
Description
Backport of a 3.0 fix by @pblottiere. This would fix #17280 #17187
Checklist
fixes #11111
in the commit message next to the description[FEATURE]
in the commit message[needs-docs]
in the commit message and containt sufficient information in the commit message to be documentedscripts/prepare-commit.sh
script before each commit