administration page failed loading when piwigo is installad without any table prefix #872

New Issue

2026-10-05T00:22:08+02:00

adam commented

2026-10-05 00:22:08 +02:00

Versions
Release : head of master branch
MySQL: 8.0.11
PHP: 7.2.4

Step to reproduce
1 Install piwigo on a newly created database without any prefix for table name
2 Add a picture
3 With menu on left got to administration page

Results
:heavy_check_mark:Step 1: OK
:heavy_check_mark:Step 2: OK
:x:Step 3: KO. Following error message appeared in the browser

Piwigo a rencontré une erreur non récupérable
[mysql error 1064] You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'groups' at line 2

SELECT COUNT(*)
FROM groups
;

#1 my_error D:\www_root\piwigo_src\include\dblayer\functions_mysqli.inc.php(147)
#2 pwg_query D:\www_root\piwigo_src\admin\intro.php(190)
#3 include D:\www_root\piwigo_src\admin.php(324)

**Versions** Release : head of master branch MySQL: 8.0.11 PHP: 7.2.4 **Step to reproduce** 1 Install piwigo on a newly created database without any prefix for table name 2 Add a picture 3 With menu on left got to administration page **Results** :heavy_check_mark:Step 1: OK :heavy_check_mark:Step 2: OK :x:Step 3: KO. Following error message appeared in the browser Piwigo a rencontré une erreur non récupérable [mysql error 1064] You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'groups' at line 2 SELECT COUNT(*) FROM groups ; #1 my_error D:\www_root\piwigo_src\include\dblayer\functions_mysqli.inc.php(147) #2 pwg_query D:\www_root\piwigo_src\admin\intro.php(190) #3 include D:\www_root\piwigo_src\admin.php(324)

adam closed this issue

2026-10-05 00:22:08 +02:00

adam commented

2026-10-05 00:22:09 +02:00

OK, so "groups" is now a reserved keyword in MySQL 8 :-/ gonna have to escape it with backtits!

OK, so "groups" is now a reserved keyword in MySQL 8 :-/ gonna have to escape it with backtits!

adam commented

2026-10-05 00:22:10 +02:00

PR #922 is related (for the rank keyword).

PR #922 is related (for the `rank` keyword).

adam commented

2026-10-05 00:22:11 +02:00

@samwilson @pstimpel my opinion right now, is that Piwigo needs to escape column names with backtits everywhere, which has less "risky" impact than renaming all problematic columns.

Considering MySQL 8 is not yet very common, I think we can plan this important change for Piwigo 2.10 and not for 2.9. Do you both agree with that?

@samwilson @pstimpel my opinion right now, is that Piwigo needs to escape column names with backtits everywhere, which has less "risky" impact than renaming all problematic columns. Considering MySQL 8 is not yet very common, I think we can plan this important change for Piwigo 2.10 and not for 2.9. Do you both agree with that?

adam commented

2026-10-05 00:22:12 +02:00

Yep most definitely. :-)

I don't even really think there's much wrong with having column names the same as keywords. Better to not to, certainly, but no need to be a high priority to fix.

(And by the way, it's backticks, with a ck...)

Yep most definitely. :-) I don't even really think there's much wrong with having column names the same as keywords. Better to not to, certainly, but no need to be a high priority to fix. (And by the way, it's `backticks`, with a `ck`...)

adam commented

2026-10-05 00:22:12 +02:00

I agree with the backticks, and with your timeline as well. Keep in mind we do not know about Oracles plans for Mysql. Maybe you will create kind of a sleeping nightmare. But yes, for th moment this should be OK

I agree with the backticks, and with your timeline as well. Keep in mind we do not know about Oracles plans for Mysql. Maybe you will create kind of a sleeping nightmare. But yes, for th moment this should be OK

adam commented

2026-10-05 00:22:13 +02:00

(And by the way, it's backticks, with a ck...)

LOL, I don't want to know what "backtit" might mean ;-) thank you for telling me it's backtick :-D

> (And by the way, it's `backticks`, with a `ck`...) LOL, I don't want to know what "backtit" might mean ;-) thank you for telling me it's backtick :-D

adam commented

2026-10-05 00:22:14 +02:00

@samwilson @pstimpel my opinion right now, is that Piwigo needs to escape column names with backtits everywhere, which has less "risky" impact than renaming all problematic columns.

Considering MySQL 8 is not yet very common, I think we can plan this important change for Piwigo 2.10 and not for 2.9. Do you both agree with that?

Yes sure.
It's important to plan it for easier troubleshooting and maintenance.

> @samwilson @pstimpel my opinion right now, is that Piwigo needs to escape column names with backtits everywhere, which has less "risky" impact than renaming all problematic columns. > > Considering MySQL 8 is not yet very common, I think we can plan this important change for Piwigo 2.10 and not for 2.9. Do you both agree with that? Yes sure. It's important to plan it for easier troubleshooting and maintenance.

Sign in to join this conversation.

master

issue-2560-people

translation

16.x

feat/embedding-based-contextual-tagging

issue-2576-search-include-ai-in-search-results

feat/search-ocr-ai

issue-2391-update-menu-design

15.x

detect-container

issue-2386-update-filters-users-activity

issue-2362-add-filters-options-in-admin

issue-2367-imagemagick7

issue-2225-quicksearch-help

14.x

issue-2164-BatchManagerUnitMode-Redesign

13.x

12.x

issue-1657-SVG_Support

issue-1703-historySearchMethod

11.x

2.10

2.9

2.8

2.7

2.5

2.6

2.4

2.3

2.2

2.1

2.0

1.7

1.3

1.6

1.5

1.4

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: adam/Piwigo#872