Closed (fixed)
Project:
Webform
Version:
6.1.3
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
31 Aug 2022 at 15:17 UTC
Updated:
17 Nov 2025 at 17:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
joseph.olstadComment #3
cilefen commentedIs this a bug?
Comment #4
joseph.olstadyes this is what I would call a 'bug'
Comment #5
joseph.olstadComment #6
joseph.olstadmight be related somehowComment #7
joseph.olstadComment #8
sfcamil commentedHy,
I have the same issue with webform 6.1.3 , core 9.5 with postgres
Drupal\Core\Database\DatabaseExceptionWrapper: SQLSTATE[42601]: Syntax error: 7 ERROR: syntax error at or near "." LINE 6: ...T OUTER JOIN "users_field_data" "user" ON log.uid = user.uid ^: SELECT COUNT(*) AS "expression" FROM (SELECT 1 AS "expression" FROM "webform_submission_log" "log" LEFT OUTER JOIN "users_field_data" "user" ON log.uid = user.uid LEFT OUTER JOIN "webform_submission" "submission" ON log.sid = submission.sid) "subquery"; Array ( ) in Drupal\webform_submission_log\WebformSubmissionLogManager->loadByEntities() (line 133 of modules/contrib/webform/modules/webform_submission_log/src/WebformSubmissionLogManager.php).Comment #9
jrockowitz commentedPrepending [postgres] to the title to see if someone using postgres can fix it.
Comment #10
pirvudoru commentedHad a different issue, but similar with a Postgres db, while using webform autosave, that was calling into WebformSubmissionLogManager
This change fixed it for me.
Would like to set up automated tests for postgres, but don't know yet how, yet.
PS: First time contributor to a drupal project, tried to follow the contribution guide. Let me know if anything needs to change.
Comment #11
jrockowitz commentedYour patch makes sense because USER is a reserved word in Postgres
https://www.postgresql.org/docs/current/sql-keywords-appendix.html
If the test even partially passes for Postgres and MySQL tests are fine, we can probably safely commit the patch.
Comment #12
jrockowitz commentedComment #13
jrockowitz commentedComment #14
joseph.olstadlooks good, should hold up nicely for everyone pgsql/mariadb/mysql/sqlite/mssql/others
Comment #17
jrockowitz commentedComment #20
cmanalansan commentedCan a maintainer reopen this issue? It got reintroduced when the fix for https://www.drupal.org/project/webform/issues/3508995 get applied