Bug report #13856: DB Manager: SET DEFAULT syntax wrong - QGIS Application - QGIS Issue Tracking

Bug report #13856

DB Manager: SET DEFAULT syntax wrong

Added by Paolo Cavallini about 9 years ago. Updated over 6 years ago.

Status:	Closed
Priority:	Low
Assignee:	-
Category:	DB Manager
Affected QGIS version:	3.0.0	Regression?:	No
Operating System:		Easy fix?:	No
Pull Request or Patch supplied:	No	Resolution:	fixed/implemented
Crashes QGIS or corrupts data:	No	Copied to github as #:	21879

Description

When setting the default for a column value, DB Manager fails to quote the value, e.g.

ALTER TABLE "pippo"."comuni" ALTER "nomemin" SET DEFAULT test123

instead of

ALTER TABLE "pippo"."comuni" ALTER "nomemin" SET DEFAULT 'test123'

therefore the command fails.

Associated revisions

Revision 333f2c3a
Added by Jürgen Fischer over 6 years ago

db_manager: add tooltip to default value field when editing table field properties (fixes #13856)

History

#1 Updated by Sebastian Dietrich about 9 years ago

I would say this is intended behaviour. You are not entering a value, but an expression.

ALTER TABLE "table" ALTER "column" SET DEFAULT current_user

works, because current_user is a (PostgreSQL-)function and therefore a valid expression.

#2 Updated by Paolo Cavallini about 9 years ago

If the user is required to add '', this should be made clear, it is not obvious. Perhaps adding a note to the error message would suffice.

#3 Updated by Sebastian Dietrich about 9 years ago

Adding quotes always works, but is not always the correct solution. E.g. if I enter currentuser the solution would be to correct the function name, not to add quotes. So the errormessage could be misleading.

Would a hint to the user when entering the default (e.g. a tooltip or a label) also suffice?
Something like:
Note: This is an expression, string literals must be properly quoted.