Bug #78301

Use context specific table in PageRepository

Added by Oliver Hader almost 3 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Database API (Doctrine DBAL)
Target version:
Start date:
2016-10-14
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
Stabilization Sprint

Description

By invoking the ConnectionPool instance, it's possible to use different database connections and different database platforms for a particular table. If for instance the system is running on MySQL in general, but one single table (not `pages`) uses SQLite, PageRepository::enableFields() is using quoting characters for MySQL in the SQLite context - which fails.

Associated revisions

Revision f7a30114 (diff)
Added by Oliver Hader almost 3 years ago

[BUGFIX] Use context specific table in PageRepository

By invoking the ConnectionPool instance, it's possible to use
different database connections and different database platforms
for a particular table. If for instance the system is running on
MySQL in general, but one single table (not pages) uses SQLite,
PageRepository::enableFields() is using quoting characters for
MySQL in the SQLite context - which fails.

Resolves: #78301
Releases: master
Change-Id: I8317a22126b1725a1730ebc66042dbdbfc18fd7c
Reviewed-on: https://review.typo3.org/50231
Tested-by: TYPO3com <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>
Reviewed-by: Oliver Hader <>
Tested-by: Oliver Hader <>

History

#1 Updated by Gerrit Code Review almost 3 years ago

  • Status changed from New 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/50231

#2 Updated by Benni Mack almost 3 years ago

  • Sprint Focus set to Stabilization Sprint

#3 Updated by Oliver Hader almost 3 years ago

  • Description updated (diff)

#4 Updated by Gerrit Code Review almost 3 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/50231

#5 Updated by Gerrit Code Review almost 3 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50231

#6 Updated by Oliver Hader almost 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#7 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF