Bug #81610
Install Tool > Database Analyzer: Migration failed for fields containing a space
100%
Description
In one of my installations, the install tool database analyzer shows the following query to execute:
ALTER TABLE tx_foo_bar CHANGE l18n_diffsourc e `zzz_deleted_l18n_diffsourc e` MEDIUMBLOB DEFAULT NULL
The existing field contains really a space, which is fine for some databases. (sure in is this case it is a mistake)
After executing the query, this query fails:
Error: 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 '`zzz_deleted_l18n_diffsourc e` MEDIUMBLOB DEFAULT NULL' at line 1
I guess the problem is, that the field name is not quoted.
Associated revisions
[BUGFIX] Always quote SQL identifiers in schema migrations
Doctrine doesn't always return quoted identifiers when reading the
schema information from the database. This patch works around that
by properly quoting the identifiers when determining the required
changes to the database.
Resolves: #81610
Releases: master, 8.7
Change-Id: I746a8a023cf494050cd83c089e0d2bca98c046f1
Reviewed-on: https://review.typo3.org/53373
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
[BUGFIX] Always quote SQL identifiers in schema migrations
Doctrine doesn't always return quoted identifiers when reading the
schema information from the database. This patch works around that
by properly quoting the identifiers when determining the required
changes to the database.
Resolves: #81610
Releases: master, 8.7
Change-Id: I746a8a023cf494050cd83c089e0d2bca98c046f1
Reviewed-on: https://review.typo3.org/53403
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
History
#1
Updated by Morton Jonuschat over 2 years ago
- Status changed from New to Accepted
- Assignee set to Morton Jonuschat
#2
Updated by Gerrit Code Review over 2 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53373
#3
Updated by Morton Jonuschat over 2 years ago
- File Screen Shot 2017-07-01 at 10.56.39.png View added
#4
Updated by Gerrit Code Review over 2 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53373
#5
Updated by Gerrit Code Review over 2 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53403
#6
Updated by Morton Jonuschat over 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 1dd2254c112b3191cb3a86b964ec221e67637f52.
#7
Updated by Benni Mack about 1 year ago
- Status changed from Resolved to Closed