[TASK] Cleanup Extbase for 6.1 release
authorAlexander Schnitzler <alex.schnitzler@typovision.de>
Sun, 14 Apr 2013 13:24:25 +0000 (15:24 +0200)
committerAlexander Schnitzler <typo3@alexanderschnitzler.de>
Tue, 23 Apr 2013 20:44:19 +0000 (22:44 +0200)
* Remove whitespace after closing php tag
* Fix PhpDocComments

Releases: 6.1
Resolves: #47216
Change-Id: I8f35f9adb65c75cb11e1045bbade3d0b6708db86
Reviewed-on: https://review.typo3.org/19976
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Alexander Schnitzler
Tested-by: Alexander Schnitzler
43 files changed:
typo3/sysext/extbase/Classes/Configuration/BackendConfigurationManager.php
typo3/sysext/extbase/Classes/Configuration/ConfigurationManagerInterface.php
typo3/sysext/extbase/Classes/Domain/Model/AbstractFileFolder.php
typo3/sysext/extbase/Classes/Domain/Model/BackendUserGroup.php
typo3/sysext/extbase/Classes/Error/Result.php
typo3/sysext/extbase/Classes/Mvc/Controller/ActionController.php
typo3/sysext/extbase/Classes/Mvc/Controller/Argument.php
typo3/sysext/extbase/Classes/Mvc/Controller/Arguments.php
typo3/sysext/extbase/Classes/Mvc/Controller/CommandController.php
typo3/sysext/extbase/Classes/Mvc/Request.php
typo3/sysext/extbase/Classes/Mvc/Web/Routing/UriBuilder.php
typo3/sysext/extbase/Classes/Object/Container/ClassInfoCache.php
typo3/sysext/extbase/Classes/Persistence/Generic/Backend.php
typo3/sysext/extbase/Classes/Persistence/Generic/LazyLoadingProxy.php
typo3/sysext/extbase/Classes/Persistence/Generic/LazyObjectStorage.php
typo3/sysext/extbase/Classes/Persistence/Generic/Mapper/DataMapFactory.php
typo3/sysext/extbase/Classes/Persistence/Generic/Mapper/DataMapper.php
typo3/sysext/extbase/Classes/Persistence/Generic/PersistenceManager.php
typo3/sysext/extbase/Classes/Persistence/Generic/Query.php
typo3/sysext/extbase/Classes/Persistence/Generic/Storage/Typo3DbBackend.php
typo3/sysext/extbase/Classes/Persistence/Generic/Typo3QuerySettings.php
typo3/sysext/extbase/Classes/Persistence/ObjectStorage.php
typo3/sysext/extbase/Classes/Persistence/QueryInterface.php
typo3/sysext/extbase/Classes/Persistence/Repository.php
typo3/sysext/extbase/Classes/Property/Mapper.php
typo3/sysext/extbase/Classes/Property/PropertyMapper.php
typo3/sysext/extbase/Classes/Property/PropertyMappingConfiguration.php
typo3/sysext/extbase/Classes/Property/TypeConverter/AbstractFileFolderConverter.php
typo3/sysext/extbase/Classes/Property/TypeConverter/AbstractTypeConverter.php
typo3/sysext/extbase/Classes/Property/TypeConverter/DateTimeConverter.php
typo3/sysext/extbase/Classes/Property/TypeConverter/FileConverter.php
typo3/sysext/extbase/Classes/Property/TypeConverter/FileReferenceConverter.php
typo3/sysext/extbase/Classes/Property/TypeConverter/FolderConverter.php
typo3/sysext/extbase/Classes/Property/TypeConverter/PersistentObjectConverter.php
typo3/sysext/extbase/Classes/Reflection/ClassReflection.php
typo3/sysext/extbase/Classes/Reflection/ObjectAccess.php
typo3/sysext/extbase/Classes/Scheduler/FieldProvider.php
typo3/sysext/extbase/Classes/SignalSlot/Dispatcher.php
typo3/sysext/extbase/Classes/Utility/ArrayUtility.php
typo3/sysext/extbase/Classes/Utility/DebuggerUtility.php
typo3/sysext/extbase/Classes/Validation/Validator/BooleanValidator.php
typo3/sysext/extbase/Classes/Validation/Validator/NotEmptyValidator.php
typo3/sysext/extbase/Classes/Validation/ValidatorResolver.php

index 8d7607b..1876812 100644 (file)
@@ -181,7 +181,6 @@ class BackendConfigurationManager extends \TYPO3\CMS\Extbase\Configuration\Abstr
        /**
         * Returns a comma separated list of storagePid that are below a certain storage pid.
         *
-        *
         * @param string $storagePid Storage PID to start at; multiple PIDs possible as comma-separated list
         * @param integer $recursionDepth Maximum number of levels to search, 0 to disable recursive lookup
         * @return string storage PIDs
index 4e4852a..ca0d0fd 100644 (file)
@@ -27,6 +27,10 @@ namespace TYPO3\CMS\Extbase\Configuration;
  *
  *  This copyright notice MUST APPEAR in all copies of the script!
  ***************************************************************/
+
+/**
+ * Class ConfigurationManagerInterface
+ */
 interface ConfigurationManagerInterface extends \TYPO3\CMS\Core\SingletonInterface {
 
        const CONFIGURATION_TYPE_FRAMEWORK = 'Framework';
index bf1bf13..c9e4229 100644 (file)
@@ -54,4 +54,4 @@ abstract class AbstractFileFolder extends \TYPO3\CMS\Extbase\DomainObject\Abstra
        }
 }
 
-?>
+?>
\ No newline at end of file
index b296a05..1041c08 100644 (file)
@@ -418,7 +418,7 @@ class BackendUserGroup extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
        /**
         * Set the the bit for file operations are allowed.
         *
-        * @param boolean
+        * @param boolean $value
         * @return void
         */
        public function setFileOperationAllowed($value) {
@@ -437,7 +437,7 @@ class BackendUserGroup extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
        /**
         * Set the the bit for unzip files are allowed.
         *
-        * @param boolean
+        * @param boolean $value
         * @return void
         */
        public function setFileUnzipAllowed($value) {
@@ -456,7 +456,7 @@ class BackendUserGroup extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
        /**
         * Set the the bit for directory operations are allowed.
         *
-        * @param boolean
+        * @param boolean $value
         * @return void
         */
        public function setDirectoryOperationAllowed($value) {
@@ -475,7 +475,7 @@ class BackendUserGroup extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
        /**
         * Set the the bit for copy directories.
         *
-        * @param boolean
+        * @param boolean $value
         * @return void
         */
        public function setDirectoryCopyAllowed($value) {
@@ -494,7 +494,7 @@ class BackendUserGroup extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
        /**
         * Set the the bit for remove directories recursively.
         *
-        * @param boolean
+        * @param boolean $value
         * @return void
         */
        public function setDirectoryRemoveRecursivelyAllowed($value) {
index db1d450..b03a735 100644 (file)
@@ -278,7 +278,7 @@ class Result {
         * Flatten a tree of Result objects, based on a certain property.
         *
         * @param string $propertyName
-        * @param array $result
+        * @param array &$result
         * @param array $level
         * @return void
         */
index 4a09e84..3b7f528 100644 (file)
@@ -490,7 +490,7 @@ class ActionController extends \TYPO3\CMS\Extbase\Mvc\Controller\AbstractControl
         * display no flash message at all on errors. Override this to customize
         * the flash message in your action controller.
         *
-        * @return string|boolean The flash message or FALSE if no flash message should be set
+        * @return string The flash message or FALSE if no flash message should be set
         * @api
         */
        protected function getErrorFlashMessage() {
index 283c809..3d1925c 100644 (file)
@@ -225,7 +225,7 @@ class Argument {
        /**
         * Injects the Persistence Manager
         *
-        * @param \TYPO3\CMS\Extbase\Persistence\PersistenceManagerInterface
+        * @param \TYPO3\CMS\Extbase\Persistence\PersistenceManagerInterface $persistenceManager
         * @return void
         */
        public function injectPersistenceManager(\TYPO3\CMS\Extbase\Persistence\PersistenceManagerInterface $persistenceManager) {
@@ -385,7 +385,7 @@ class Argument {
        /**
         * Create and set a validator chain
         *
-        * @param array Object names of the validators
+        * @param array $objectNames Object names of the validators
         * @return \TYPO3\CMS\Extbase\Mvc\Controller\Argument Returns $this (used for fluent interface)
         * @api
         * @deprecated since Extbase 1.4.0, will be removed in Extbase 6.1
@@ -426,8 +426,8 @@ class Argument {
        /**
         * Sets the value of this argument.
         *
-        * @param mixed $rawValue: The value of this argument
-        * @return \TYPO3\CMS\Extbase\Mvc\Controller\Argument $this
+        * @param mixed $rawValue The value of this argument
+        * @return \TYPO3\CMS\Extbase\Mvc\Controller\Argument
         * @throws \TYPO3\CMS\Extbase\Mvc\Exception\InvalidArgumentValueException if the argument is not a valid object of type $dataType
         */
        public function setValue($rawValue) {
index 89ffcea..ca5a4de 100644 (file)
@@ -249,7 +249,7 @@ class Arguments extends \ArrayObject {
         * If an argument with the specified name or short name does not exist, an empty
         * string is returned.
         *
-        * @param string argument name
+        * @param string $argumentName argument name
         * @return string long argument name or empty string
         */
        protected function translateToLongArgumentName($argumentName) {
index 140251c..b553d58 100644 (file)
@@ -224,7 +224,7 @@ class CommandController implements \TYPO3\CMS\Extbase\Mvc\Controller\CommandCont
         * @see http://www.php.net/sprintf
         * @param string $text Text to output
         * @param array $arguments Optional arguments to use for sprintf
-        * @return string
+        * @return void
         */
        protected function output($text, array $arguments = array()) {
                if ($arguments !== array()) {
index 4e7bf33..12f02fa 100644 (file)
@@ -316,7 +316,7 @@ class Request implements \TYPO3\CMS\Extbase\Mvc\RequestInterface {
         *
         * Note that the action name must start with a lower case letter and is case sensitive.
         *
-        * @param string $actionName: Name of the action to execute by the controller
+        * @param string $actionName Name of the action to execute by the controller
         *
         * @return void
         * @throws \TYPO3\CMS\Extbase\Mvc\Exception\InvalidActionNameException if the action name is not valid
index e81c396..928e39c 100644 (file)
@@ -688,8 +688,8 @@ class UriBuilder {
         *
         * @param \TYPO3\CMS\Extbase\DomainObject\AbstractDomainObject $object
         * @return array
+        * @todo Refactore this into convertDomainObjectsToIdentityArrays()
         */
-       // TODO Refactore this into convertDomainObjectsToIdentityArrays()
        public function convertTransientObjectToArray(\TYPO3\CMS\Extbase\DomainObject\AbstractDomainObject $object) {
                $result = array();
                foreach ($object->_getProperties() as $propertyName => $propertyValue) {
index 9416165..1a4ea7a 100644 (file)
@@ -65,6 +65,7 @@ class ClassInfoCache {
         * Gets the cache for the id
         *
         * @param string $id
+        * @return mixed
         */
        public function get($id) {
                if (!isset($this->level1Cache[$id])) {
@@ -76,8 +77,8 @@ class ClassInfoCache {
        /**
         * sets the cache for the id
         *
-        * @param $id
-        * @param $value
+        * @param string $id
+        * @param mixed $value
         */
        public function set($id, $value) {
                $this->level1Cache[$id] = $value;
index 78c29dd..a1cf9f2 100644 (file)
@@ -143,7 +143,7 @@ class Backend implements \TYPO3\CMS\Extbase\Persistence\Generic\BackendInterface
        /**
         * Injects the Reflection Service
         *
-        * @param \TYPO3\CMS\Extbase\Reflection\ReflectionService
+        * @param \TYPO3\CMS\Extbase\Reflection\ReflectionService $reflectionService
         * @return void
         */
        public function injectReflectionService(\TYPO3\CMS\Extbase\Reflection\ReflectionService $reflectionService) {
@@ -449,7 +449,7 @@ class Backend implements \TYPO3\CMS\Extbase\Persistence\Generic\BackendInterface
         * @param \TYPO3\CMS\Extbase\Persistence\ObjectStorage $objectStorage The object storage to be persisted.
         * @param \TYPO3\CMS\Extbase\DomainObject\DomainObjectInterface $parentObject The parent object. One of the properties holds the object storage.
         * @param string $propertyName The name of the property holding the object storage.
-        * @param array $row The row array of the parent object to be persisted. It's passed by reference and gets filled with either a comma separated list of uids (csv) or the number of contained objects.
+        * @param array &$row The row array of the parent object to be persisted. It's passed by reference and gets filled with either a comma separated list of uids (csv) or the number of contained objects.
         * @return void
         */
        protected function persistObjectStorage(\TYPO3\CMS\Extbase\Persistence\ObjectStorage $objectStorage, \TYPO3\CMS\Extbase\DomainObject\DomainObjectInterface $parentObject, $propertyName, array &$row) {
@@ -579,8 +579,8 @@ class Backend implements \TYPO3\CMS\Extbase\Persistence\Generic\BackendInterface
                $parentColumnMap = $parentDataMap->getColumnMap($parentPropertyName);
                if ($parentColumnMap->getTypeOfRelation() !== \TYPO3\CMS\Extbase\Persistence\Generic\Mapper\ColumnMap::RELATION_HAS_MANY) {
                        throw new \TYPO3\CMS\Extbase\Persistence\Exception\IllegalRelationTypeException(
-                               'Parent column relation type is ' . $parentColumnMap->getTypeOfRelation()
-                               ' but should be ' . \TYPO3\CMS\Extbase\Persistence\Generic\Mapper\ColumnMap::RELATION_HAS_MANY,
+                               'Parent column relation type is ' . $parentColumnMap->getTypeOfRelation() .
+                               ' but should be ' . \TYPO3\CMS\Extbase\Persistence\Generic\Mapper\ColumnMap::RELATION_HAS_MANY,
                                1345368105
                        );
                }
@@ -880,7 +880,7 @@ class Backend implements \TYPO3\CMS\Extbase\Persistence\Generic\BackendInterface
         * Adds common databse fields to a row
         *
         * @param \TYPO3\CMS\Extbase\DomainObject\DomainObjectInterface $object
-        * @param array $row
+        * @param array &$row
         * @return void
         */
        protected function addCommonFieldsToRow(\TYPO3\CMS\Extbase\DomainObject\DomainObjectInterface $object, array &$row) {
@@ -898,7 +898,7 @@ class Backend implements \TYPO3\CMS\Extbase\Persistence\Generic\BackendInterface
         * Adjustes the common date fields of the given row to the current time
         *
         * @param \TYPO3\CMS\Extbase\DomainObject\DomainObjectInterface $object
-        * @param array $row The row to be updated
+        * @param array &$row The row to be updated
         * @return void
         */
        protected function addCommonDateFieldsToRow(\TYPO3\CMS\Extbase\DomainObject\DomainObjectInterface $object, array &$row) {
@@ -931,7 +931,7 @@ class Backend implements \TYPO3\CMS\Extbase\Persistence\Generic\BackendInterface
         * Deletes an object
         *
         * @param \TYPO3\CMS\Extbase\DomainObject\DomainObjectInterface $object The object to be removed from the storage
-        * @param bool $markAsDeleted Wether to just flag the row deleted (default) or really delete it
+        * @param boolean $markAsDeleted Wether to just flag the row deleted (default) or really delete it
         * @return void
         */
        protected function removeEntity(\TYPO3\CMS\Extbase\DomainObject\DomainObjectInterface $object, $markAsDeleted = TRUE) {
index 60b9a3b..7a496d6 100644 (file)
@@ -195,7 +195,7 @@ class LazyLoadingProxy implements \Iterator, \TYPO3\CMS\Extbase\Persistence\Gene
        /**
         * Returns the next position of the storage array
         *
-        * @return mixed
+        * @return void
         */
        public function next() {
                $realInstance = $this->_loadRealInstance();
index 48e7948..7aed39a 100644 (file)
@@ -126,6 +126,8 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
 
        // Delegation to the ObjectStorage methods below
        /**
+        * @param \TYPO3\CMS\Extbase\Persistence\ObjectStorage $storage
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::addAll
         */
        public function addAll($storage) {
@@ -134,6 +136,9 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @param object $object The object to add.
+        * @param mixed $data The data to associate with the object.
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::attach
         */
        public function attach($object, $data = NULL) {
@@ -142,6 +147,9 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @param object $object The object to look for.
+        * @return boolean
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::contains
         */
        public function contains($object) {
@@ -171,6 +179,8 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @return object The object at the current iterator position.
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::current
         */
        public function current() {
@@ -179,6 +189,8 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @param object $object The object to remove.
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::detach
         */
        public function detach($object) {
@@ -187,6 +199,8 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @return string The index corresponding to the position of the iterator.
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::key
         */
        public function key() {
@@ -203,6 +217,9 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @param object $object The object to look for.
+        * @return boolean
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::offsetExists
         */
        public function offsetExists($object) {
@@ -211,6 +228,9 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @param object $object The object to look for.
+        * @return mixed
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::offsetGet
         */
        public function offsetGet($object) {
@@ -219,6 +239,9 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @param object $object The object to add.
+        * @param mixed $info The data to associate with the object.
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::offsetSet
         */
        public function offsetSet($object, $info) {
@@ -227,6 +250,9 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @param object $object The object to remove.
+        * @return void
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::offsetUnset
         */
        public function offsetUnset($object) {
@@ -235,6 +261,9 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @param \TYPO3\CMS\Extbase\Persistence\ObjectStorage $storage The storage containing the elements to remove.
+        * @return void
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::removeAll
         */
        public function removeAll($storage) {
@@ -251,6 +280,8 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @return boolean
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::valid
         */
        public function valid() {
@@ -259,6 +290,8 @@ class LazyObjectStorage extends \TYPO3\CMS\Extbase\Persistence\ObjectStorage imp
        }
 
        /**
+        * @return array
+        *
         * @see \TYPO3\CMS\Extbase\Persistence\ObjectStorage::toArray
         */
        public function toArray() {
index b1ec3a3..d052ae8 100644 (file)
@@ -179,7 +179,6 @@ class DataMapFactory implements \TYPO3\CMS\Core\SingletonInterface {
                        $columnMap = $this->setFieldEvaluations($columnMap, $columnDefinition['config']);
                        $dataMap->addColumnMap($columnMap);
                }
-               // debug($dataMap);
                return $dataMap;
        }
 
index 66a80bb..a354429 100644 (file)
@@ -111,7 +111,7 @@ class DataMapper implements \TYPO3\CMS\Core\SingletonInterface {
        /**
         * Injects the Reflection Service
         *
-        * @param \TYPO3\CMS\Extbase\Reflection\ReflectionService
+        * @param \TYPO3\CMS\Extbase\Reflection\ReflectionService $reflectionService
         * @return void
         */
        public function injectReflectionService(\TYPO3\CMS\Extbase\Reflection\ReflectionService $reflectionService) {
@@ -121,7 +121,7 @@ class DataMapper implements \TYPO3\CMS\Core\SingletonInterface {
        /**
         * Injects the DataMap Factory
         *
-        * @param \TYPO3\CMS\Extbase\Persistence\Generic\Mapper\DataMapFactory
+        * @param \TYPO3\CMS\Extbase\Persistence\Generic\Mapper\DataMapFactory $dataMapFactory
         * @return void
         */
        public function injectDataMapFactory(\TYPO3\CMS\Extbase\Persistence\Generic\Mapper\DataMapFactory $dataMapFactory) {
@@ -323,7 +323,7 @@ class DataMapper implements \TYPO3\CMS\Core\SingletonInterface {
         * @param \TYPO3\CMS\Extbase\DomainObject\DomainObjectInterface $parentObject The object instance this proxy is part of
         * @param string $propertyName The name of the proxied property in it's parent
         * @param mixed $fieldValue The raw field value.
-        * @param bool $enableLazyLoading A flag indication if the related objects should be lazy loaded
+        * @param boolean $enableLazyLoading A flag indication if the related objects should be lazy loaded
         * @return \TYPO3\CMS\Extbase\Persistence\Generic\LazyObjectStorage|\TYPO3\CMS\Extbase\Persistence\QueryResultInterface The result
         */
        public function fetchRelated(\TYPO3\CMS\Extbase\DomainObject\DomainObjectInterface $parentObject, $propertyName, $fieldValue = '', $enableLazyLoading = TRUE) {
@@ -552,8 +552,8 @@ class DataMapper implements \TYPO3\CMS\Core\SingletonInterface {
        /**
         * Returns the column name for a given property name of the specified class.
         *
-        * @param string $className
         * @param string $propertyName
+        * @param string $className
         * @return string The column name
         */
        public function convertPropertyNameToColumnName($propertyName, $className = NULL) {
index 9606334..30617a9 100644 (file)
@@ -320,10 +320,8 @@ class PersistenceManager implements \TYPO3\CMS\Extbase\Persistence\PersistenceMa
        /**
         * Converts the given object into an array containing the identity of the domain object.
         *
-        * @param object $object The object to be converted
-        * @return array The identity array in the format array('__identity' => '...')
         * @throws \TYPO3\CMS\Extbase\Persistence\Generic\Exception\NotImplementedException
-        * @throws \TYPO3\CMS\Extbase\Persistence\Exception\UnknownObjectException if the given object is not known to the Persistence Manager
+        * @param object $object The object to be converted
         * @api
         */
        public function convertObjectToIdentityArray($object) {
@@ -334,10 +332,8 @@ class PersistenceManager implements \TYPO3\CMS\Extbase\Persistence\PersistenceMa
         * Recursively iterates through the given array and turns objects
         * into arrays containing the identity of the domain object.
         *
-        * @param array $array The array to be iterated over
-        * @return array The modified array without objects
         * @throws \TYPO3\CMS\Extbase\Persistence\Generic\Exception\NotImplementedException
-        * @throws \TYPO3\CMS\Extbase\Persistence\Exception\UnknownObjectException if array contains objects that are not known to the Persistence Manager
+        * @param array $array The array to be iterated over
         * @api
         * @see convertObjectToIdentityArray()
         */
index 60e960a..45bb74c 100644 (file)
@@ -593,7 +593,6 @@ class Query implements \TYPO3\CMS\Extbase\Persistence\QueryInterface {
         * It matches if the multivalued property contains no values or is NULL.
         *
         * @param string $propertyName The name of the multivalued property to compare against
-        * @return boolean
         * @throws \TYPO3\CMS\Extbase\Persistence\Generic\Exception\NotImplementedException
         * @throws \TYPO3\CMS\Extbase\Persistence\Exception\InvalidQueryException if used on a single-valued property
         * @api
index f2a3874..daa8e41 100644 (file)
@@ -395,7 +395,7 @@ class Typo3DbBackend implements \TYPO3\CMS\Extbase\Persistence\Generic\Storage\B
         * Parses the query and returns the SQL statement parts.
         *
         * @param \TYPO3\CMS\Extbase\Persistence\QueryInterface $query The query
-        * @param array $parameters
+        * @param array &$parameters
         * @return array The SQL statement parts
         */
        public function parseQuery(\TYPO3\CMS\Extbase\Persistence\QueryInterface $query, array &$parameters) {
@@ -757,10 +757,10 @@ class Typo3DbBackend implements \TYPO3\CMS\Extbase\Persistence\Generic\Storage\B
        }
 
        /**
-        * @param $className
-        * @param $tableName
-        * @param $propertyPath
-        * @param array $sql
+        * @param string &$className
+        * @param string &$tableName
+        * @param array &$propertyPath
+        * @param array &$sql
         * @throws \TYPO3\CMS\Extbase\Persistence\Generic\Exception
         * @throws \TYPO3\CMS\Extbase\Persistence\Generic\Exception\InvalidRelationConfigurationException
         * @throws \TYPO3\CMS\Extbase\Persistence\Generic\Exception\MissingColumnMapException
@@ -861,12 +861,11 @@ class Typo3DbBackend implements \TYPO3\CMS\Extbase\Persistence\Generic\Storage\B
         * Replace query placeholders in a query part by the given
         * parameters.
         *
-        * @param string $sqlString The query part with placeholders
+        * @param string &$sqlString The query part with placeholders
         * @param array $parameters The parameters
         * @param string $tableName
         *
         * @throws \TYPO3\CMS\Extbase\Persistence\Generic\Exception
-        * @return string The query part with replaced placeholders
         */
        protected function replacePlaceholders(&$sqlString, array $parameters, $tableName = 'foo') {
                // TODO profile this method again
@@ -899,7 +898,7 @@ class Typo3DbBackend implements \TYPO3\CMS\Extbase\Persistence\Generic\Storage\B
         *
         * @param \TYPO3\CMS\Extbase\Persistence\Generic\QuerySettingsInterface $querySettings The TYPO3 CMS specific query settings
         * @param string $tableName The table name to add the additional where clause for
-        * @param string $sql
+        * @param string &$sql
         * @return void
         */
        protected function addAdditionalWhereClause(\TYPO3\CMS\Extbase\Persistence\Generic\QuerySettingsInterface $querySettings, $tableName, &$sql) {
@@ -1028,8 +1027,8 @@ class Typo3DbBackend implements \TYPO3\CMS\Extbase\Persistence\Generic\Storage\B
                                if (isset($GLOBALS['TCA'][$tableName]['ctrl']['transOrigPointerField'])
                                        && $querySettings->getSysLanguageUid() > 0
                                ) {
-                                       $additionalWhereClause .= ' OR (' . $tableName . '.' . $GLOBALS['TCA'][$tableName]['ctrl']['languageField'] . '=0' .
-                                               ' AND ' . $tableName . '.uid NOT IN (' . 'SELECT ' . $tableName . '.' . $GLOBALS['TCA'][$tableName]['ctrl']['transOrigPointerField'] .
+                                       $additionalWhereClause .= ' OR (' . $tableName . '.' . $GLOBALS['TCA'][$tableName]['ctrl']['languageField'] .
+                                               ' =0 AND ' . $tableName . '.uid NOT IN (SELECT ' . $tableName . '.' . $GLOBALS['TCA'][$tableName]['ctrl']['transOrigPointerField'] .
                                                ' FROM ' . $tableName .
                                                ' WHERE ' . $tableName . '.' . $GLOBALS['TCA'][$tableName]['ctrl']['transOrigPointerField'] . '>0' .
                                                ' AND ' . $tableName . '.' . $GLOBALS['TCA'][$tableName]['ctrl']['languageField'] . '>0';
@@ -1309,4 +1308,4 @@ class Typo3DbBackend implements \TYPO3\CMS\Extbase\Persistence\Generic\Storage\B
        }
 }
 
-?>
+?>
\ No newline at end of file
index ce0d771..5a7ab62 100644 (file)
@@ -141,7 +141,7 @@ class Typo3QuerySettings implements \TYPO3\CMS\Extbase\Persistence\Generic\Query
         * Sets the pid(s) of the storage page(s) that should be respected for the query.
         *
         * @param array $storagePageIds If TRUE the storage page ID will be determined and the statement will be extended accordingly.
-        * @return \TYPO3\CMS\Extbase\Persistence\Generic\QuerySettingsInterface instance of $this to allow method chaining
+        * @return void
         * @api
         */
        public function setStoragePageIds(array $storagePageIds) {
@@ -182,7 +182,7 @@ class Typo3QuerySettings implements \TYPO3\CMS\Extbase\Persistence\Generic\Query
         * Sets the language uid for the language overlay.
         *
         * @param integer $sysLanguageUid language uid for the language overlay
-        * @return \TYPO3\CMS\Extbase\Persistence\Generic\QuerySettingsInterface instance of $this to allow method chaining
+        * @return void
         * @api
         */
        public function setSysLanguageUid($sysLanguageUid) {
index 0b0460a..2d8861f 100644 (file)
@@ -306,7 +306,7 @@ class ObjectStorage implements \Countable, \Iterator, \ArrayAccess, \TYPO3\CMS\E
        /**
         * Dummy method to avoid unserialization.
         *
-        * @param $serialized
+        * @param string $serialized
         * @throws \RuntimeException
         * @return void
         */
index ac33e2c..7b8e3ed 100644 (file)
@@ -358,7 +358,7 @@ interface QueryInterface {
        /**
         * Returns the maximum size of the result set to limit.
         *
-        * @param integer
+        * @return integer
         * @api
         */
        public function getLimit();
index ba74fb2..aeddf95 100644 (file)
@@ -123,7 +123,6 @@ class Repository implements \TYPO3\CMS\Extbase\Persistence\RepositoryInterface,
        }
 
        /**
-        *
         * Injects the Persistence Session
         *
         * @param \TYPO3\CMS\Extbase\Persistence\Generic\Session $session The persistence session
index aea0e8a..bbfffcd 100644 (file)
@@ -135,8 +135,9 @@ class Mapper implements \TYPO3\CMS\Core\SingletonInterface {
         * @param array $propertyNames Names of the properties to map.
         * @param mixed $source Source containing the properties to map to the target object. Must either be an array, ArrayObject or any other object.
         * @param object $target The target object
-        * @param \TYPO3\CMS\Extbase\Validation\Validator\ObjectValidatorInterface $targetObjectValidator A validator used for validating the target object
         * @param array $optionalPropertyNames Names of optional properties. If a property is specified here and it doesn't exist in the source, no error is issued.
+        * @param \TYPO3\CMS\Extbase\Validation\Validator\ObjectValidatorInterface $targetObjectValidator A validator used for validating the target object
+        *
         * @return boolean TRUE if the mapped properties are valid, otherwise FALSE
         * @see getMappingResults()
         * @see map()
@@ -164,7 +165,7 @@ class Mapper implements \TYPO3\CMS\Core\SingletonInterface {
        /**
         * Add errors to the mapping result from an object validator (property errors).
         *
-        * @param array Array of \TYPO3\CMS\Extbase\Validation\PropertyError
+        * @param array $errors Array of \TYPO3\CMS\Extbase\Validation\PropertyError
         * @return void
         */
        protected function addErrorsFromObjectValidator($errors) {
@@ -353,4 +354,4 @@ class Mapper implements \TYPO3\CMS\Core\SingletonInterface {
        }
 }
 
-?>
+?>
\ No newline at end of file
index 22745b2..64d514e 100644 (file)
@@ -137,7 +137,7 @@ class PropertyMapper implements \TYPO3\CMS\Core\SingletonInterface {
         * @param mixed $source the source data to map. MUST be a simple type, NO object allowed!
         * @param string $targetType The type of the target; can be either a class name or a simple type.
         * @param \TYPO3\CMS\Extbase\Property\PropertyMappingConfigurationInterface $configuration Configuration for the property mapping.
-        * @param array $currentPropertyPath The property path currently being mapped; used for knowing the context in case an exception is thrown.
+        * @param array &$currentPropertyPath The property path currently being mapped; used for knowing the context in case an exception is thrown.
         * @throws Exception\TypeConverterException
         * @return mixed an instance of $targetType
         */
index 32965e0..f47a060 100644 (file)
@@ -155,7 +155,7 @@ class PropertyMappingConfiguration implements \TYPO3\CMS\Extbase\Property\Proper
         *
         * Example: allowProperties('title', 'content', 'author')
         *
-        * @return void
+        * @return \TYPO3\CMS\Extbase\Property\PropertyMappingConfiguration
         * @api
         */
        public function allowProperties() {
@@ -393,4 +393,4 @@ class PropertyMappingConfiguration implements \TYPO3\CMS\Extbase\Property\Proper
        }
 }
 
-?>
+?>
\ No newline at end of file
index 2f66a2e..bd73bfa 100644 (file)
@@ -86,4 +86,4 @@ abstract class AbstractFileFolderConverter extends \TYPO3\CMS\Extbase\Property\T
        abstract protected function getOriginalResource($source);
 }
 
-?>
+?>
\ No newline at end of file
index 94feed6..4ca8bd9 100644 (file)
@@ -131,7 +131,7 @@ abstract class AbstractTypeConverter implements \TYPO3\CMS\Extbase\Property\Type
         * Returns an empty list of sub property names
         *
         * @param mixed $source
-        * @return array<string>
+        * @return array
         * @api
         */
        public function getSourceChildPropertiesToBeConverted($source) {
@@ -144,10 +144,10 @@ abstract class AbstractTypeConverter implements \TYPO3\CMS\Extbase\Property\Type
         * @param string $targetType
         * @param string $propertyName
         * @param \TYPO3\CMS\Extbase\Property\PropertyMappingConfigurationInterface $configuration
-        * @return string
         * @api
         */
        public function getTypeOfChildProperty($targetType, $propertyName, \TYPO3\CMS\Extbase\Property\PropertyMappingConfigurationInterface $configuration) {
        }
 }
+
 ?>
\ No newline at end of file
index dd25cc4..869ef92 100644 (file)
@@ -170,7 +170,8 @@ class DateTimeConverter extends \TYPO3\CMS\Extbase\Property\TypeConverter\Abstra
 
        /**
         * Returns whether date information (day, month, year) are present as keys in $source.
-        * @param $source
+        *
+        * @param array $source
         * @return bool
         */
        protected function isDatePartKeysProvided(array $source) {
index f212821..0d35f50 100644 (file)
@@ -58,4 +58,4 @@ class FileConverter extends \TYPO3\CMS\Extbase\Property\TypeConverter\AbstractFi
        }
 }
 
-?>
+?>
\ No newline at end of file
index f8d3803..be07a26 100644 (file)
@@ -58,4 +58,4 @@ class FolderConverter extends \TYPO3\CMS\Extbase\Property\TypeConverter\Abstract
        }
 }
 
-?>
+?>
\ No newline at end of file
index c14ec2f..5604ba7 100644 (file)
@@ -209,7 +209,7 @@ class PersistentObjectConverter extends \TYPO3\CMS\Extbase\Property\TypeConverte
         *
         * @param array $source
         * @param string $targetType
-        * @param array $convertedChildProperties
+        * @param array &$convertedChildProperties
         * @param \TYPO3\CMS\Extbase\Property\PropertyMappingConfigurationInterface $configuration
         * @throws \TYPO3\CMS\Extbase\Property\Exception\InvalidPropertyMappingConfigurationException
         * @return object
index 83f6978..c3656b4 100644 (file)
@@ -116,7 +116,7 @@ class ClassReflection extends \ReflectionClass {
         * that a \TYPO3\CMS\Extbase\Reflection\PropertyReflection object is returned instead of the
         * orginal ReflectionProperty instance.
         *
-        * @param string $name: Name of the property
+        * @param string $name Name of the property
         * @return \TYPO3\CMS\Extbase\Reflection\PropertyReflection Property reflection object of the specified property in this class
         */
        public function getProperty($name) {
@@ -155,7 +155,7 @@ class ClassReflection extends \ReflectionClass {
         * Checks if the doc comment of this method is tagged with
         * the specified tag
         *
-        * @param string $tag: Tag name to check for
+        * @param string $tag Tag name to check for
         * @return boolean TRUE if such a tag has been defined, otherwise FALSE
         */
        public function isTaggedWith($tag) {
index 75e1ce0..b1feef3 100644 (file)
@@ -87,7 +87,7 @@ class ObjectAccess {
         * @param mixed $subject Object or array to get the property from
         * @param string $propertyName name of the property to retrieve
         * @param boolean $forceDirectAccess directly access property using reflection(!)
-        * @param boolean $propertyExists (by reference) will be set to TRUE if the specified property exists and is gettable
+        * @param boolean &$propertyExists (by reference) will be set to TRUE if the specified property exists and is gettable
         *
         * @throws Exception\PropertyNotAccessibleException
         * @return mixed Value of the property
@@ -184,7 +184,7 @@ class ObjectAccess {
         * on it without checking if it existed.
         * - else, return FALSE
         *
-        * @param mixed $subject The target object or array
+        * @param mixed &$subject The target object or array
         * @param string $propertyName Name of the property to set
         * @param mixed $propertyValue Value of the property
         * @param boolean $forceDirectAccess directly access property using reflection(!)
index 1323d47..b788bdf 100644 (file)
@@ -68,7 +68,7 @@ class FieldProvider implements \TYPO3\CMS\Scheduler\AdditionalFieldProviderInter
        /**
         * Render additional information fields within the scheduler backend.
         *
-        * @param array $taskInfo Array information of task to return
+        * @param array &$taskInfo Array information of task to return
         * @param mixed $task \TYPO3\CMS\Scheduler\Task\AbstractTask or tx_scheduler_Execution instance
         * @param \TYPO3\CMS\Scheduler\Controller\SchedulerModuleController $schedulerModule Reference to the calling object (BE module of the Scheduler)
         * @return array Additional fields
@@ -94,7 +94,7 @@ class FieldProvider implements \TYPO3\CMS\Scheduler\AdditionalFieldProviderInter
        /**
         * Validates additional selected fields
         *
-        * @param array $submittedData
+        * @param array &$submittedData
         * @param \TYPO3\CMS\Scheduler\Controller\SchedulerModuleController $schedulerModule
         * @return boolean
         */
index 8a2076f..d6d79dc 100644 (file)
@@ -62,13 +62,14 @@ class Dispatcher implements \TYPO3\CMS\Core\SingletonInterface {
         * Since this dispatches is used very early when the ObjectManager
         * is not fully initialized (especially concerning caching framework),
         * this is the only way.
+        *
+        * @return void
         */
        public function initializeObject() {
-               if ($this->isInitialized) {
-                       return;
+               if (!$this->isInitialized) {
+                       $this->objectManager = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Extbase\\Object\\ObjectManager');
+                       $this->isInitialized = TRUE;
                }
-               $this->objectManager = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Extbase\\Object\\ObjectManager');
-               $this->isInitialized = TRUE;
        }
 
        /**
index cd7fdf6..bf8aa13 100644 (file)
@@ -253,7 +253,7 @@ class ArrayUtility {
        /**
         * Sorts multidimensional arrays by recursively calling ksort on its elements.
         *
-        * @param array $array the array to sort
+        * @param array &$array the array to sort
         * @param integer $sortFlags may be used to modify the sorting behavior using these values (see http://www.php.net/manual/en/function.sort.php)
         * @return boolean TRUE on success, FALSE on failure
         * @see asort()
index 489a4d8..e0d0e68 100644 (file)
@@ -202,7 +202,7 @@ class DebuggerUtility {
        /**
         * Checks if a given object was already rendered.
         *
-        * @param $object
+        * @param object $object
         * @return bool TRUE if the given object was already rendered
         */
        static protected function isAlreadyRendered($object) {
@@ -212,10 +212,10 @@ class DebuggerUtility {
        /**
         * Renders the header of a given object/collection. It is usually the class name along with some flags.
         *
-        * @param $object
-        * @param $level
-        * @param $plainText
-        * @param $ansiColors
+        * @param object $object
+        * @param integer $level
+        * @param boolean $plainText
+        * @param boolean $ansiColors
         * @return string The rendered header with tags
         */
        static protected function renderHeader($object, $level, $plainText, $ansiColors) {
@@ -296,10 +296,10 @@ class DebuggerUtility {
        }
 
        /**
-        * @param $object
-        * @param $level
-        * @param $plainText
-        * @param $ansiColors
+        * @param object $object
+        * @param integer $level
+        * @param boolean $plainText
+        * @param boolean $ansiColors
         * @return string The rendered body content of the Object(Storage)
         */
        static protected function renderContent($object, $level, $plainText, $ansiColors) {
index 22b3070..4759a28 100644 (file)
@@ -78,4 +78,4 @@ class NotEmptyValidator extends \TYPO3\CMS\Extbase\Validation\Validator\Abstract
        }
 }
 
-?>
+?>
\ No newline at end of file
index b03eb9d..3f0ce95 100644 (file)
@@ -286,7 +286,7 @@ class ValidatorResolver implements \TYPO3\CMS\Core\SingletonInterface {
         * Parses $rawValidatorOptions not containing quoted option values.
         * $rawValidatorOptions will be an empty string afterwards (pass by ref!).
         *
-        * @param string &$rawValidatorOptions
+        * @param string $rawValidatorOptions
         * @return array An array of optionName/optionValue pairs
         */
        protected function parseValidatorOptions($rawValidatorOptions) {