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

[Bug]: Print tool shows even if option is not enabled in LM Plugin #557

Open
1 task
gioman opened this issue Feb 12, 2024 · 8 comments
Open
1 task

[Bug]: Print tool shows even if option is not enabled in LM Plugin #557

gioman opened this issue Feb 12, 2024 · 8 comments

Comments

@gioman
Copy link

gioman commented Feb 12, 2024

What is the bug?

LMWC

image

LM Plugin

image

Steps to reproduce the issue

See images above.

Versions, safeguards, check summary etc

LMWC 3.7.3 and QGIS Desktop/Server 3.28.15 LM Plugin 4.2.0

Check Lizmap plugin

  • I have done the step just before in the Lizmap QGIS desktop plugin before opening this ticket. Otherwise, my ticket is not considered valid and might get closed.

Operating system

Ubuntu 22.04

Browsers

Firefox, Chrome

Browsers version

Latest updates

Relevant log output

Nothing relevant.
@gioman gioman added the Bug label Feb 12, 2024
@Gustry
Copy link
Member

Gustry commented Feb 12, 2024

Please follow the template , do you have the correct server filled in your table ?
The "Print" combobox mustn't be visible anymore if the target version is 3.7.

Is-it a target LWC 3.6 file ?

@gioman
Copy link
Author

gioman commented Feb 12, 2024

Is-it a target LWC 3.6 file ?

@Gustry no, is a project for testing 3.7.3 created from scratch, and its CFG too

image

LM plugin is up to date. I understand that now the layouts are manged on a layout basis and not all together, I was just noticing that the checkbox was still there and has no effect. Are you saying that the checkbox should not even show anymore?

@Gustry
Copy link
Member

Gustry commented Feb 12, 2024

Are you saying that the checkbox should not even show anymore?

Yes, depending of the server selected in the comboxbox.
Maybe just the next run of the plugin, the checkbox will not be shown.

I just tried, it's not visible for me on the 3.7 server.

@gioman
Copy link
Author

gioman commented Feb 12, 2024

Yes, depending of the server selected in the comboxbox. Maybe just the next run of the plugin, the checkbox will not be shown.

I just tried, it's not visible for me on the 3.7 server.

@Gustry I'm definitely testing on a clean 3.7.3 installation with QGIS Desktop and Server LTR and the latest version of LM plugin: that combobox is always there, even on a new QGIS profile.

@Gustry
Copy link
Member

Gustry commented Feb 13, 2024

How many servers do you have in your table ? Which versions are they ?

Please give your QGIS desktop logs (the comic strip bubble). You should have some info about the LWC version used for the Lizmap UI (defining which widgets are in light blue, hiding checkbox etc).

Like Saving new value about the LWC target version...

@gioman
Copy link
Author

gioman commented Feb 13, 2024

How many servers do you have in your table ? Which versions are they ?

@Gustry just 1. I'm using a clean QGIS profile with LM plugin which points to just one fresh LMWC 3.7.3 installation.

Please give your QGIS desktop logs (the comic strip bubble). You should have some info about the LWC version used for the Lizmap UI (defining which widgets are in light blue, hiding checkbox etc).

Like Saving new value about the LWC target version...

image

That "critical" log message is curious, as you can see I'm running the latest version of the plugin available on qgis.org's public repo.

2024-02-13T19:32:50     CRITICAL    Version checker : running a higher version than on plugins.qgis.org
2024-02-13T19:32:50     INFO    Loading all layouts from the QGIS project :
2024-02-13T19:32:50     INFO    Dialog has been loaded successful, with Lizmap configuration file
2024-02-13T19:32:50     INFO    Reading configuration from dictionary for layer Enquadramento_Ortos2018
2024-02-13T19:32:50     INFO    Reading configuration from dictionary for layer Cont_NUTSIII_CAOP2022
2024-02-13T19:32:50     INFO    Reading configuration from dictionary for layer Google Satellite
2024-02-13T19:32:50     INFO    Reading configuration from dictionary for layer baselayers
2024-02-13T19:32:50     INFO    Reading configuration from dictionary for layer OpenStreetMap
2024-02-13T19:32:50     INFO    Reading configuration from dictionary for layer Bing Satellite
2024-02-13T19:32:52     INFO    Lizmap Web Client target version 3.7, let's try to make the project valid.
2024-02-13T19:32:52     INFO    Existing shortnames detected before in project : Enquadramento_Ortos2018, Cont_NUTSIII_CAOP2022, Google_Satellite, baselayers, OpenStreetMap, Bing_Satellite
2024-02-13T19:32:52     INFO    New shortnames added : 0
2024-02-13T19:32:52     INFO    Existing shortnames detected before in project : Enquadramento_Ortos2018, Cont_NUTSIII_CAOP2022, Google_Satellite, baselayers, OpenStreetMap, Bing_Satellite
2024-02-13T19:32:52     INFO    Setting a project shortname : teste2
2024-02-13T19:32:52     INFO    Project has been detected : VALID according to OGC validation.
2024-02-13T19:32:52     INFO    Project has been detected : VALID according to OGC validation.
2024-02-13T19:32:52     INFO    Writing Lizmap configuration file for LWC version 3.7
2024-02-13T19:32:52     INFO    Skipping key 'noLegendImage' because of max_version.
2024-02-13T19:32:52     INFO    Skipping key 'noLegendImage' because of max_version.
2024-02-13T19:32:52     INFO    Skipping key 'noLegendImage' because of max_version.
2024-02-13T19:32:52     INFO    Skipping key 'noLegendImage' because of max_version.
2024-02-13T19:32:52     INFO    Skipping key 'noLegendImage' because of max_version.
2024-02-13T19:32:52     INFO    Skipping key 'noLegendImage' because of max_version.
2024-02-13T19:32:52     INFO    The Lizmap configuration file has been written to "/***/teste2.qgs.cfg"
2024-02-13T19:32:52     INFO    Getting min/max scales

@Gustry Gustry transferred this issue from 3liz/lizmap-web-client Feb 14, 2024
@Gustry
Copy link
Member

Gustry commented Feb 14, 2024

That "critical" log message is curious

indeed, but this code is dodgy, I was trying to use the QGIS plugin manager, do not worry about this log. Let's see next release what is the output, CF 1aa2fbe

@Gustry just 1. I'm using a clean QGIS profile with LM plugin which points to just one fresh LMWC 3.7.3 installation.

Ok I got it. It's because the signel lwc_version_changed wasn't triggered in your case.
I will see what I can do. In the meantime, I made a quick fix : 9bcfeb1

@Gustry
Copy link
Member

Gustry commented Jul 1, 2024

Do you still replicate this issue @gioman ?
I made a few bugfix about this signal to be emitted when the server has changed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants