Add a panel with project misconfigurations & optimisation suggestions

[mdouchin]

We have a list of long well-known possible misconfigurations in a Lizmap project:

• PostgreSQL layers without a password (or better a service file)
• No integer primary keys
• int8 primary keys 39a6798
• Layer not published as WFS -> do it for the user
• Identifier field not published as WFS
• Local datasource (C:/ , etc.)
• ...

Or pieces of advice to improve performances or "user-friendlyship"

• Check the "Add the geometry to WMS GetFeatureInfo response"
• Use the "Trust" project option
• Use JPEG format for the base layers and for the layers with no need for transparency
• Use png; mode=16bit or png; mode=8bit when there is few colors in style
• Check Use estimated table metadata when creating connexion
• ...

It could be great to list these things and warn the user with a of changes to make.

[mdouchin]

cc @rldhont @nboisteault

[Gustry]

• According to the discussion today for LWC 3.6 : check the layer ID for as name
• Check fields used in all table manager if they are still in WMS/WFS
• Do not use bigint primary key : 39a6798
• geometry not included on the server side, while some expressions are using it, easiest is display name
• Any layer with a "qgis auth manager" set up (already done for vectors), IGN raster with basic auth
• A layer is not "queryable" in the project properties, "datasource tab" while the popup is enabled

[Gustry]

List of current QMessageBox used, because of a project check :

• Address search without a key : IGN&google. By the way, this really something to fix after bbd67e1
• "Use layer IDs as name" since LWC 3.6, blocking the saving
• SAAS
  • ECW
  • Auth DB for postgresql
  • Path higher than ../../
• Invalid field type tid, int8 ...
• Duplicated layer with filters
• Legend image 3.6
• Deprecated lizmap popup for all versions
• Legacy server without admin user, LWC 3.5 etc
• Legacy server not providing a correct metadata, the plugin on the server side was not required before 3.6
• Accent, space ... in project name (this has been fixed in latest LWC releases, need to check)
• Capitalize layer names
• Absolute path in project properties

[Gustry]

• In Add French GEOPF check #564 we were thinking to check rasters having a authcfg as well. It has been done only for French IGN provider for now to have an explicit error message. We can extend with generic layers as well.

[nboisteault]

• warn users when using jp2 files and propose a conversion to tiled GeoTiff, compressed to jpeg with pyramids
• warn users about heavy jpg, png, svg files in print composer

[nboisteault]

• suggest server cache for not editable layers and no specific rights. We could save a lot of GetMap requests.

[Gustry]

@nboisteault We are trying to make the list objective :)

warn users about heavy jpg, png, svg files in print composer

What is heavy ? :)

suggest server cache for not editable layers and no specific rights

This is IMHO very project specific. For now, we can't make "suggestion", as all "errors" are reported in the CFG file and then displayed in LWC when connected as an admin, so we must stay objective.

[Gustry]

suggest server cache for not editable layers and no specific rights. We could save a lot of GetMap requests.

See the issue 3liz/lizmap-web-client#4327 The cache can lead to other side effect and not necessary for all layers.

[nboisteault]

@nboisteault We are trying to make the list objective :)

warn users about heavy jpg, png, svg files in print composer

What is heavy ? :)

I don't know ;)

suggest server cache for not editable layers and no specific rights

This is IMHO very project specific. For now, we can't make "suggestion", as all "errors" are reported in the CFG file and then displayed in LWC when connected as an admin, so we must stay objective.

So let's not write it in cfg :)

[nboisteault]

suggest server cache for not editable layers and no specific rights. We could save a lot of GetMap requests.

See the issue 3liz/lizmap-web-client#4327 The cache can lead to other side effect and not necessary for all layers.

I'll fix this one.