Project

General

Profile

Actions

Bug #34327

closed

exec_INSERTmultipleRows doesn't check for alternative handlers

Added by Leon Dietsch about 12 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Database API (Doctrine DBAL)
Target version:
-
Start date:
2012-02-27
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

I spotted this issue while using the DbBackend of the CachingFramework, wich uses exec_INSERTmultipleRows to add tags to the tags table. When routing the tags table to another handler wich is also "native" it won't use the alternative handler for the tags table.

This happens because the function does not fetch the handler key for the current table and just calls the parent function if the type is "native".

A patch is attached to this issue. I will try to commit a patch in gerrit for review.


Files

patch.diff (803 Bytes) patch.diff Leon Dietsch, 2012-02-27 16:11

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #48220: exec_INSERTmultipleRows insert double RowsClosed2013-05-15

Actions
Actions #1

Updated by Leon Dietsch about 12 years ago

I forgot to add the patch. To resolve the issue I fetch the handler for the current table and check if the handlerKey is _DEFAULT, additionally to the check for handler type being 'native'.

Actions #2

Updated by Gerrit Code Review about 12 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9272

Actions #3

Updated by Gerrit Code Review about 11 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9272

Actions #4

Updated by Gerrit Code Review about 11 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9272

Actions #5

Updated by Gerrit Code Review about 11 years ago

Patch set 1 for branch dbal_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/19758

Actions #6

Updated by Gerrit Code Review about 11 years ago

Patch set 1 for branch dbal_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19759

Actions #7

Updated by Gerrit Code Review about 11 years ago

Patch set 1 for branch DBAL_1-2 has been pushed to the review server.
It is available at https://review.typo3.org/19760

Actions #8

Updated by Christian Kuhn about 11 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset commit:6494493f00e5b5388b00fe52946b235bd4934099.

Actions #9

Updated by Michael Stucki over 10 years ago

  • Project changed from 329 to TYPO3 Core
Actions #10

Updated by Michael Stucki over 10 years ago

  • Category set to 999
Actions #11

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF