-
-
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
Avoid updating classification widgets which the user is editing #51720
Conversation
const QgsProcessingParameterDefinition *def = ppww->parameterDefinition(); | ||
QVariant value = method->parameterValues().value( def->name(), def->defaultValueForGui() ); | ||
ppww->setParameterValue( value, context ); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nyalldawson I've tested your branch again and debugged the code, it looks to me a no-op: the only case in which the mBlockClassificationMethodUpdates
is != 0 is when it is called from the constructor.
In all other occasions it is called from QgsGraduatedSymbolRendererWidget::classificationMethodWidgetChanged
when it is increased before the call.
I've even added an else branch and placed a breakpoint right here and it is never reached.
I must be missing something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe it's because you were testing with your fix applied too -- indeed with your fix mBlockClassificationMethodUpdates will always be zero, because it's reset immediately before the deferred timer is called to perform the classification... I'll rework to avoid that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I've updated accordingly (at a complexity cost 🙃 )
The QGIS project highly values your contribution and would love to see this work merged! Unfortunately this PR has not had any activity in the last 14 days and is being automatically marked as "stale". If you think this pull request should be merged, please check
|
dc3df04
to
d7b5481
Compare
The QGIS project highly values your contribution and would love to see this work merged! Unfortunately this PR has not had any activity in the last 14 days and is being automatically marked as "stale". If you think this pull request should be merged, please check
|
@elpaso are you happy with this? |
The QGIS project highly values your contribution and would love to see this work merged! Unfortunately this PR has not had any activity in the last 14 days and is being automatically marked as "stale". If you think this pull request should be merged, please check
|
While we hate to see this happen, this PR has been automatically closed because it has not had any activity in the last 21 days. If this pull request should be reconsidered, please follow the guidelines in the previous comment and reopen this pull request. Or, if you have any further questions, just ask! We love to help, and if there's anything the QGIS project can do to help push this PR forward please let us know how we can assist. |
Fixes #51687
Possible alternative to #51714