Skip to content
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

Fix broken filter on Relation Editor #51914

Merged
merged 1 commit into from
Feb 27, 2023

Conversation

signedav
Copy link
Contributor

@signedav signedav commented Feb 17, 2023

Recreates the QgsFilteredSelectionManager on request-change, when the layer persists. With this there is the current and valid FeatureRequest set on QgsFilteredSelectionManager and this fixes the broken filter in the Relation Editor.

Bug

In the Attribute Table you could not select the child features anymore in the Relation Editor from the second feature.
image

Video:
selectionnotconsideredinfilter.webm

The feature got selected, but because it has not been considered in the QgsFilteredSelectionManager it was neither "yellow" nor the buttons reacted accordingly.

LTR has not been affected

…hen the layer persists. With this there is the current and valid FeatureRequest set on QgsFilteredSelectionManager and this fixes the brocken selection in the RelationEditor.
@signedav
Copy link
Contributor Author

Would be great when you can have a look at it @troopa81

@troopa81
Copy link
Contributor

I'm away from keyboard for the next week. If nobody steps in the mean time, I would take a look when I come back.

@github-actions github-actions bot added this to the 3.30.0 milestone Feb 17, 2023
Copy link
Contributor

@troopa81 troopa81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for fixing

@troopa81 troopa81 merged commit 1a77b16 into qgis:master Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants