Bug #33460

Missing function visiblity declarations

Added by Aske Ertmann almost 10 years ago. Updated almost 10 years ago.

Status:
Resolved
Priority:
Should have
Category:
-
Start date:
2012-01-26
Due date:
% Done:

100%

Estimated time:
PHP Version:
Has patch:
No
Complexity:
easy

Description

I found a missing visiblity declaration in \TYPO3\FLOW3\MVC\Web\Routing\Route, and thought I'd search to see if there were more.. With this "grep -rin $'\t'function *" in the FLOW3 package folder I found 6 occurrences..

TYPO3.FLOW3/Classes/Cache/Frontend/FrontendInterface.php:109:    function flush();
TYPO3.FLOW3/Classes/MVC/Web/Routing/Route.php:472:    function extractInternalArguments(array &$arguments) {
TYPO3.FLOW3/Classes/Persistence/Doctrine/Mapping/Driver/Flow3AnnotationDriver.php:673:            function ($className) {
TYPO3.FLOW3/Classes/Persistence/Doctrine/Service.php:164:            function ($message) use ($that) {
TYPO3.FLOW3/Classes/Utility/Environment.php:505:                    function(&$value, $key) {
TYPO3.FLOW3/Tests/Functional/AOP/Fixtures/Name.php:27:    function __construct($name) {

The Route function "extractInternalArguments" should be protected, as discussed with Bastian, since it's would require to construct a new route and it's not part of the api (@api) so I could likely change. It's stems from this commit: https://review.typo3.org/#change,5779

I haven't looked for further details about the other occurrences..

Also available in: Atom PDF