Move networklogger to gui

[domi4484]

To make QgsNetworkLoggerPanelWidget accessible to applications that builds against QGIS gui

[github-actions]

🍎 MacOS Qt6 builds

Download MacOS Qt6 builds of this PR for testing.
This installer is not signed, control+click > open the app to avoid the warning
(Built from commit 6bbc1d1)

🪟 Windows Qt6 builds

Download Windows Qt6 builds of this PR for testing.
(Built from commit 6bbc1d1)

[troopa81]

What are you motivations to move it in gui module ? especially when you don't enable sip API.

[domi4484]

@troopa81 thanks for the review! I would like to use the network logger widget inside an external application that uses QGIS as a library.
As we build directly against the c++ library, we don't need the sip bindings. I thought it was better to not expose all those classes to python if there is no need for it at the moment

[troopa81]

As we build directly against the c++ library, we don't need the sip bindings. I thought it was better to not expose all those classes to python if there is no need for it at the moment

Maybe, yes. I don't know the policy exactly in this kind of situation.

[nyalldawson]

I thought it was better to not expose all those classes to python if there is no need for it at the moment

I think keeping them out of the bindings is the right move -- there's too much risk of abuse / nasty behavior if we expose these classes for use in Python.

[github-actions]

Tests failed for Qt 6 (ALL_BUT_PROVIDERS - ubuntu)

One or more tests failed using the build from commit ef1ad53

layout_export_marker_masking_w_effects

layout_export_marker_masking_w_effects

Test failed at test_layout_export_marker_masking_w_effects at tests/src/python/test_selective_masking.py:1295

Rendered image did not match tests/testdata/control_images/selective_masking/layout_export_marker_masking_w_effects/layout_export_marker_masking_w_effects.png (found 20 pixels different)

layout_export_marker_masking_w_transparency

layout_export_marker_masking_w_transparency

Test failed at test_layout_export_marker_masking_w_transparency at tests/src/python/test_selective_masking.py:1383

Rendered image did not match tests/testdata/control_images/selective_masking/layout_export_marker_masking_w_transparency/layout_export_marker_masking_w_transparency.png (found 23 pixels different)

layout_export_text_masking_w_transparency

layout_export_text_masking_w_transparency

Test failed at test_layout_export_text_masking_w_transparency at tests/src/python/test_selective_masking.py:1414

Rendered image did not match tests/testdata/control_images/selective_masking/layout_export_text_masking_w_transparency/layout_export_text_masking_w_transparency.png (found 12 pixels different)

layout_export_w_effects

layout_export_w_effects

Test failed at test_layout_export_w_effects at tests/src/python/test_selective_masking.py:1204

Rendered image did not match tests/testdata/control_images/selective_masking/layout_export_w_effects/layout_export_w_effects.png (found 6 pixels different)

layout_export_force_raster_render

layout_export_force_raster_render

Test failed at test_layout_export_w_force_raster_render at tests/src/python/test_selective_masking.py:1357

Rendered image did not match tests/testdata/control_images/selective_masking/layout_export_force_raster_render/layout_export_force_raster_render.png (found 94 pixels different)

The full test report (included comparison of rendered vs expected images) can be found here.

Further documentation on the QGIS test infrastructure can be found in the Developer's Guide.

[nyalldawson]

Can you also move this to Qgis namespace? (and rename to DevToolsNodeRole, and remove "Role" prefix from the values)

[domi4484]

Done thanks