Skip to content

[raster] Fix bad layer properties not saved - #66607

Merged
nyalldawson merged 2 commits into
qgis:masterfrom
elpaso:bugfix-gh-66324-edit-broken-raster-layer
Jul 1, 2026
Merged

[raster] Fix bad layer properties not saved#66607
nyalldawson merged 2 commits into
qgis:masterfrom
elpaso:bugfix-gh-66324-edit-broken-raster-layer

Conversation

@elpaso

@elpaso elpaso commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Fix #66324 by partially reverting #9563 : I couldn't replicate the original crash so that might have been fixed.

Not recommended for backport.

Fix qgis#66324 by partially reverting qgis#9563 : I couldn't replicate
the original crash so that might have been fixed.

Not recommended for backport.
@elpaso elpaso added Bug Either a bug report, or a bug fix. Let's hope for the latter! NOT FOR BACKPORT! Too risky and unsuitable for backporting labels Jun 29, 2026
@github-actions github-actions Bot added this to the 4.2.0 milestone Jun 29, 2026
@github-actions github-actions Bot added the Rasters Related to general raster layer handling (not specific data formats) label Jun 29, 2026
@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

🪟 Windows Qt6 builds

Download Windows Qt6 builds of this PR for testing.
To execute locally, unzip the downloaded zip file and run bin\qgis-bin.exe in the extracted directory.
You might be prompted by Windows Defender click "Run anyway"
(Built from commit 0cbdcd4)

🍎 MacOS Qt6 builds

Download MacOS Qt6 builds of this PR for testing.
This app is not notarized, run sudo xattr -d com.apple.quarantine /Applications/QGIS*.app to avoid the warning
(Built from commit 0cbdcd4)

@nyalldawson

Copy link
Copy Markdown
Collaborator

@elpaso interesting -- this has triggered the test for checking for crashes when applying layer properties dialog to invalid layers. So there's definitely something else still trying to access broken layers when applying this dialog...

@elpaso elpaso closed this Jul 1, 2026
@elpaso elpaso reopened this Jul 1, 2026
@nyalldawson
nyalldawson merged commit 0055723 into qgis:master Jul 1, 2026
65 of 67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Either a bug report, or a bug fix. Let's hope for the latter! NOT FOR BACKPORT! Too risky and unsuitable for backporting Rasters Related to general raster layer handling (not specific data formats)

2 participants