id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
24585	Remove repetitive guarding against double quoting database identifiers	Jon Dufresne	nobody	"All built-in database backends define a `quote_name()` function. In all cases, the function first checks if the string argument is already quoted.

IMO, this looks a bit like ""just in case"" style programming and a bit of a code smell. Ideally, the calling code should not be passing quoted names in the first place. The Django logic should be such that names are quoted exactly once and not multiple time. I believe if names are quoted twice, it should be treated as a bug in the calling code, not a bug in the `quote_name()` function.

After testing, these guards are unnecessary and all tests pass without it across multiple database backends.

Removing the guard means it won't need to be checked every time SQL names are quoted.

PR to follow."	Cleanup/optimization	closed	Database layer (models, ORM)	dev	Normal	wontfix			Unreviewed	1	0	0	0	0	0