Annoying warning "...no such table: lizmap_search..."

3liz / lizmap-web-client

Transfer a QGIS project on a server, Lizmap is providing the web interface to browse it

https://www.lizmap.com

Mozilla Public License 2.0

254 stars 143 forks source link

Annoying warning "...no such table: lizmap_search..." #1285

Open nboisteault opened 5 years ago

nboisteault commented 5 years ago

What is the bug?

When loading LWC errors.log shows : 2019-07-17 10:07:27 10.0.2.2 warning 2019-07-17 10:07:27 [2] SQLite3::query(): Unable to prepare statement: 1, no such table: lizmap_search /lizmap_web_client/lib/jelix/plugins/db/sqlite3/sqlite3.dbconnection.php 112

Steps to reproduce the issue

Load LWC
Look last line ine /var/log/errors.log
There is a warning like above

Environment

Lizmap version: 3.2 and master
Lizmap admin information (users, logs…) stored into Sqlite (default)

infoble commented 4 years ago

Idem for me with lizmap 3.3. This error does not seem to affect the application. Me my only problem is the lack of qgis server log but I do not think there is a link.

nboisteault commented 4 years ago

@mdouchin @laurentj We should replace this query with this one ('Slow and sure' part). But it only works with PostgreSQL. Should this search functionality work with SQLite ?

laurentj commented 4 years ago

There is already all we need into Jelix to see if the table exists, with jDbSchema. It does the query you indicate for pgsql, and similar queries for other type of databases.

I have a patch. But it annoys me, because doing such query at each requests is stupid.

I don't understand why this "search engine" is always enabled. It should be deactivated by default, and we should have a checkbox into the configuration panel to enable/disable it (so we should have a configuration parameter into lizmapConfig.ini.php). And of course, when the administrator wants to activate it, the table should be checked, and if it not there, it should be kept disabled (or we could disable the checkbox).

laurentj commented 4 years ago

We should have also :

an update script that adds this new parameter, with true if the table already exists
the installer script should create this table if the parameter is set to true, or, in the configuration panel, it should create the table if the checkbox is checked.

nboisteault commented 3 years ago

@laurentj @rldhont @mdouchin do we postpone the fix?

rldhont commented 3 years ago

yes

rldhont commented 3 years ago

link to https://github.com/3liz/lizmap-web-client/issues/1968

rldhont commented 3 years ago

It's more than a bug, it's a feature.

laurentj commented 3 years ago

We should have a "checkup" when displaying the dashboard of lizmap : it could tests this feature, (and spatialite), and display a message in the dashboard if it fails.

Gustry commented 3 years ago

This has been fixed, no ?

Gustry commented 2 years ago

@nboisteault This ticket is only moving from milestone to milestone these last 3 years. Removing the milestone.