Skip to content

[Backport release_3_10] Fix #5934: Display legend for external WMS layers#6558

Merged
rldhont merged 2 commits intorelease_3_10from
backport-6114-to-release_3_10
Feb 10, 2026
Merged

[Backport release_3_10] Fix #5934: Display legend for external WMS layers#6558
rldhont merged 2 commits intorelease_3_10from
backport-6114-to-release_3_10

Conversation

@3liz-bot
Copy link
Copy Markdown
Contributor

@3liz-bot 3liz-bot commented Feb 10, 2026

Backport #6114
Authored by: @meyerlor

meyerlor and others added 2 commits February 10, 2026 06:15
@meyerlor @github-actions
Fix #5934: Display legend for external WMS layers
ab97a9c 
External WMS layers were showing empty legend icons because QGIS Server
returns empty icon fields in JSON GetLegendGraphic responses.

- Add getLegendGraphicPNG() method to WMS class
- Detect external WMS layers via layerConfig.externalWmsToggle
- External WMS layers fetch PNG legend URL directly
- Update icon getter to handle URLs (http/https/relative paths)
- Set externalWmsToggle for WMS layers (not just EPSG:3857)
- Add LAYERTITLE=FALSE parameter to suppress redundant title
@rldhont @github-actions
JS: Using image symbology for layer
6e9f1fb
@github-actions github-actions bot added this to the 3.10.0 milestone Feb 10, 2026
@github-actions github-actions bot added the legend/layer tree Tool which displays the layer tree with legends label Feb 10, 2026
@3liz-bot
Copy link
Copy Markdown
Contributor Author

3liz-bot commented Feb 10, 2026 •
edited
Loading

All Playwright tests LEGACY

Summary

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
517 514 1 2 0 5 14m 47s

Failed Tests

Failed Tests ❌ Fail Message

Flaky Tests

Flaky Tests 🍂 Retries
Test JSON data for plot 0 - Municipalities 1
#2 Layer export with user_in_group_a user logged in 2
Popup with map click & attribute table 1
Switcher from map to map 1
Build permalink, reload and apply one 1

Flaky Rate

Overall Flakiness 1.15%    ±0
Test 📝 Flaky Attempts 🍂 Total Attempts 🎯 Flaky Rate (%) 📈
#2 Layer export with user_in_group_a user logged in 2 4 50.00    ↑50.00
Test JSON data for plot 0 - Municipalities 1 2 50.00    ↑50.00
#2 Layer export with user_in_group_a user logged in 2 4 50.00    ↑50.00
Switcher from map to map 1 2 50.00    ↑50.00
Build permalink, reload and apply one 1 2 50.00    ↑50.00
Popup with map click & attribute table 1 4 25.00    ↑25.00
Popup with map click & attribute table 1 4 25.00    ↑25.00
Popup with map click & attribute table 1 4 25.00    ↑25.00

Github Test Reporter by CTRF 💚

All Playwright tests BLEEDING_EDGE

Summary

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
517 514 1 2 0 5 15m

Failed Tests

Failed Tests ❌ Fail Message

Flaky Tests

Flaky Tests 🍂 Retries
Test JSON data for plot 0 - Municipalities 1
#2 Layer export with user_in_group_a user logged in 1
Layer export request ACL 1
Switcher from map to map 1
Build permalink, reload and apply one 1

Flaky Rate

Overall Flakiness 0.96%    ±0
Test 📝 Flaky Attempts 🍂 Total Attempts 🎯 Flaky Rate (%) 📈
Test JSON data for plot 0 - Municipalities 1 2 50.00    ↑50.00
Layer export request ACL 1 2 50.00    ↑50.00
Switcher from map to map 1 2 50.00    ↑50.00
Build permalink, reload and apply one 1 2 50.00    ↑50.00
#2 Layer export with user_in_group_a user logged in 1 3 33.33    ↑33.33
#2 Layer export with user_in_group_a user logged in 1 3 33.33    ↑33.33

Github Test Reporter by CTRF 💚

@rldhont rldhont merged commit d066a1c into release_3_10 Feb 10, 2026
18 checks passed
@rldhont rldhont deleted the backport-6114-to-release_3_10 branch February 10, 2026 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

legend/layer tree Tool which displays the layer tree with legends

Projects

None yet

Milestone

3.10.0

Development

Successfully merging this pull request may close these issues.

3 participants

@3liz-bot @rldhont @meyerlor