Project

General

Profile

Actions

Bug #64066

closed

Testingframework record collector does not reset internal variables

Added by Bernhard Kraft almost 10 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Tests
Target version:
Start date:
2014-12-31
Due date:
% Done:

100%

Estimated time:
3.00 h
TYPO3 Version:
7
PHP Version:
5.3
Tags:
Complexity:
medium
Is Regression:
No
Sprint Focus:

Description

When using the Unit-Test framework frontend record collector [1] the resulting response is invalid.

Instead of adding records only to the specific section (configured via TypoScript) the records pile up in $this->records and get added to every section. The reason this only true for $this->records of the Collector is, that for the other class variables (like $this->structure) they are using appropriate array keys anyways.

My proposed solution is to reset the internal member variables: records, structure and structurePath each time after attaching a section. See patch on gerrit.

[1] https://git.typo3.org/Packages/TYPO3.CMS.git/blob/HEAD:/typo3/sysext/core/Tests/Functional/Framework/Frontend/Collector.php

Actions #1

Updated by Gerrit Code Review almost 10 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 http://review.typo3.org/35692

Actions #2

Updated by Gerrit Code Review almost 10 years ago

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

Actions #3

Updated by Anja Leichsenring almost 10 years ago

  • Sprint Focus set to On Location Sprint
Actions #4

Updated by Anonymous almost 10 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 50 to 100
Actions #5

Updated by Anja Leichsenring almost 9 years ago

  • Sprint Focus deleted (On Location Sprint)
Actions #6

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF