Bug #89538: Wrong comment for bulkInsert in Connection class - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #89538

closed

Wrong comment for bulkInsert in Connection class

Added by Stefan Froemken about 5 years ago. Updated almost 5 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Database API (Doctrine DBAL)

Target version:

next-patchlevel

Start date:

2019-10-30

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

7.2

Tags:

Complexity:

no-brainer

Is Regression:

Sprint Focus:

Description

Hello,

in annotation of method bulkInsert there is a wrong description of third parameter:

@param array $columns An array containing associative arrays of column-value pairs.

It looks like you have to set column names as key into the array, but that is wrong, they have to be the value of the array.

Stefan

https://docs.typo3.org/m/typo3/reference-coreapi/master/en-us/ApiOverview/Database/Connection/Index.html#bulkinsert

Actions

Copy link

Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/62177

Actions

Copy link

Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/62177

Actions

Copy link

Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/62177

Actions

Copy link

Updated by Gerrit Code Review about 5 years ago

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62141

Actions

Copy link