Task #71236

Provide correct dummy data for DatabaseLanguageRowsTest

Added by Oliver Eglseder about 5 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
Start date:
2015-11-01
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
5.6
Tags:
Complexity:
Sprint Focus:

Description

parent: #70584

The dummy data provided in DatabaseLanguageRowsTest leads to some E_NOTICEs.

Since PIDs are indispensable when it comes to data in TYPO3 it's not a bug in the core.
The mocking of BE_USER prevents access to "workspace" on null. The result of accessing unset variables/members is always null, therefor the set value is null, too.


Related issues

Related to TYPO3 Core - Task #70584: Reduce thrown E_NOTICEsClosed2018-03-07

Actions
#1

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/44470

#2

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/44470

#3

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/44470

#4

Updated by Gerrit Code Review about 5 years ago

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

#5

Updated by Gerrit Code Review about 5 years ago

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

#6

Updated by Oliver Eglseder about 5 years ago

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

Updated by Riccardo De Contardi over 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF