Project

General

Profile

Actions

Bug #77989

closed

PSR-7 ServerRequest array properties must be initialized as arrays

Added by Patrik Karisch over 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
Start date:
2016-09-19
Due date:
% Done:

100%

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

Description

All the PSR-7 Request/Response object properties which are expected to be arrays when accessed must be initialized as arrays from the beginning. Otherwise this raises errors, when e.g. queryParams or attributes are accessed, which may not be initialized from outside with the correspoding `with*()` method.

Since the PSR-7 implementation is inspired by `phly/http/` which is superseded by `zend-diactoros`, it's perfectly to fix this, as this is fixed in diactoros too (Comparing the abandoned phly implementation against current diactoros).

Actions #1

Updated by Patrik Karisch over 7 years ago

  • Subject changed from PSR-7 Request/Response array properties must be initialized as arrays to PSR-7 ServerRequest array properties must be initialized as arrays
Actions #2

Updated by Gerrit Code Review over 7 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/49969

Actions #3

Updated by Patrik Karisch over 7 years ago

  • % Done changed from 0 to 100
Actions #4

Updated by Gerrit Code Review over 7 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/49969

Actions #5

Updated by Gerrit Code Review over 7 years ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49970

Actions #6

Updated by Patrik Karisch over 7 years ago

  • Status changed from Under Review to Resolved
Actions #7

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF