Bug #87149

Cannot debug Generator output

Added by Mathias Brodala 12 months ago. Updated 7 months ago.

Status:
Closed
Priority:
Should have
Category:
Extbase
Target version:
Start date:
2018-12-13
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

If DebuggerUtility::var_dump() is used on a \Generator, an error occurs:

Cannot rewind a generator that was already run

This is due to the implicit rewind() call within DebuggerUtility::renderCollection() which should be avoided for generators.

Associated revisions

Revision 78c0d489 (diff)
Added by Mathias Brodala 12 months ago

[BUGFIX] Do not rewind Generator in DebuggerUtility::var_dump()

Resolves: #87149
Releases: master, 8.7
Change-Id: I8cf2cfab94634276a67fd3d0dd36215c12ae5490
Reviewed-on: https://review.typo3.org/59132
Tested-by: TYPO3com <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>
Reviewed-by: Riny van Tiggelen <>
Reviewed-by: Ralf Merz <>
Tested-by: Ralf Merz <>
Reviewed-by: Mathias Brodala <>
Tested-by: Mathias Brodala <>

Revision abd75646 (diff)
Added by Mathias Brodala 12 months ago

[BUGFIX] Do not rewind Generator in DebuggerUtility::var_dump()

Resolves: #87149
Releases: master, 8.7
Change-Id: I8cf2cfab94634276a67fd3d0dd36215c12ae5490
Reviewed-on: https://review.typo3.org/59134
Tested-by: TYPO3com <>
Reviewed-by: Mathias Brodala <>
Tested-by: Mathias Brodala <>

History

#1 Updated by Gerrit Code Review 12 months 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/59132

#2 Updated by Gerrit Code Review 12 months 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/59132

#3 Updated by Gerrit Code Review 12 months 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/59132

#4 Updated by Gerrit Code Review 12 months ago

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

#5 Updated by Mathias Brodala 12 months ago

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

#6 Updated by Benni Mack 7 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF