Task #84627

Simplify security viewhelpers

Added by Mathias Brodala over 1 year ago. Updated 11 months ago.

Status:
Closed
Priority:
Should have
Category:
Fluid
Target version:
Start date:
2018-04-05
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

The viewhelpers IfAuthenticatedViewHelper and IfHasRoleViewHelper can be simplified by dropping their render() method. The same logic is already implemented in AbstractConditionViewHelper::renderStatic().

Associated revisions

Revision c2f3876b (diff)
Added by Mathias Brodala over 1 year ago

[TASK] Simplify frontend/backend security viewhelpers

This logic is already implemented in
AbstractConditionViewHelper::renderStatic().

Resolves: #84627
Releases: master
Change-Id: Iaf1fa4630fcc5df0ef22bec55d4fa15df1aa0986
Reviewed-on: https://review.typo3.org/56561
Tested-by: TYPO3com <>
Reviewed-by: Stefan Neufeind <>
Tested-by: Stefan Neufeind <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

History

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

#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/56561

#3 Updated by Mathias Brodala over 1 year ago

  • Subject changed from Simplify security viewhelpers of EXT:backend to Simplify security viewhelpers

The same goes for frontend viewhelpers.

#4 Updated by Mathias Brodala over 1 year ago

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

#5 Updated by Benni Mack 11 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF