Task #84345

Epic #84280: Make unit tests notice free

Make CObjectViewHelperTest notice free

Added by Jan Helke over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
Start date:
2018-03-16
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
medium
Sprint Focus:
On Location Sprint

Description

typo3/sysext/fluid/Tests/Unit/ViewHelpers/CObjectViewHelperTest.php


Related issues

Related to TYPO3 Core - Bug #86729: CObjectVIewHelper does not render top level object path Closed 2018-10-24

Associated revisions

Revision a55c2549 (diff)
Added by Jan Helke over 1 year ago

[TASK] Make CObjectViewHelperTest notice free

Releases: master
Resolves: #84345
Change-Id: I845ff3db453eee41fbcaf46f76efdeb406c51c53
Reviewed-on: https://review.typo3.org/56225
Tested-by: TYPO3com <>
Reviewed-by: Anja Leichsenring <>
Tested-by: Anja Leichsenring <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

Revision 1aed1588 (diff)
Added by Helmut Hummel about 1 year ago

[BUGFIX] Let CObjectViewHelper render top level objects again

Making the CObjectViewHelper had the unfortunate side effect,
that top level object paths are not working any more.

Adapt the tests to show the mistake and make the rendering
work in this case.

Additionally now an exception is also thrown, if the given
object path does not resolve to a TypoScript object.

Releases: master
Resolves: #86729
Related: #84345
Change-Id: I686119b480c6fae88be7d704b82094e35789c0d7
Reviewed-on: https://review.typo3.org/58686
Tested-by: TYPO3com <>
Reviewed-by: Anja Leichsenring <>
Tested-by: Anja Leichsenring <>
Reviewed-by: Andreas Fernandez <>
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

History

#1 Updated by Gerrit Code Review over 1 year ago

  • Status changed from In Progress 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/56225

#2 Updated by Gerrit Code Review over 1 year 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/56225

#3 Updated by Gerrit Code Review over 1 year 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/56225

#4 Updated by Gerrit Code Review over 1 year 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/56225

#5 Updated by Jan Helke over 1 year ago

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

#6 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

#7 Updated by Helmut Hummel about 1 year ago

  • Related to Bug #86729: CObjectVIewHelper does not render top level object path added

Also available in: Atom PDF