Bug #27611

Rename and combine TypeConverters

Added by Bastian Waidelich about 4 years ago. Updated about 4 years ago.

Status:Resolved Start date:2011-06-21
Priority:Should have Due date:
Assigned To:Bastian Waidelich % Done:

100%

Category:Property
Target version:TYPO3 Flow Base Distribution - 1.0 beta 1
PHP Version: Complexity:
Has patch: Affected Flow version:

Description

The TypeConverters should be renamed so that each converter belongs to one target type:

ArrayCollectionConverter -> ArrayCollectionConverter
ArrayToArrayConverter -> ArrayConverter
ArrayToObjectConverter -> ObjectConverter
ArrayToSplObjectStorageConverter -> SplObjectStorageConverter
BooleanToBooleanConverter -> BooleanConverter
IntegerToIntegerConverter -> IntegerConverter
PersistentObjectConverter -> PersistentObjectConverter
StringToBooleanConverter -> (not needed anymore)
StringToFloatConverter -> FloatConverter
StringToIntegerConverter -> (not needed anymore)
StringToSplObjectStorageConverter -> (not needed anymore)
StringToStringConverter -> StringConverter

Related issues

related to TYPO3.Flow - Feature #27499: Refactor TypeConverter API Resolved 2011-06-16
related to TYPO3.Flow - Feature #26132: FloatToFloatConverter missing in F3\Property\TypeConverter Closed 2011-04-19

Associated revisions

Revision acd95018
Added by Bastian Waidelich about 4 years ago

[+BUGFIX] Rename and combine TypeConverters

This change renames provided TypeConverters and combines converters
with the same target type.
Besides the ArrayToSplObjectConverter was removed, as it did not work as intended.

Change-Id: I56d54eefcaf4b2c2b57578503141e543efd12f68
Resolves: #27611

History

#1 Updated by Mr. Hudson about 4 years ago

Patch set 1 of change I56d54eefcaf4b2c2b57578503141e543efd12f68 has been pushed to the review server.
It is available at http://review.typo3.org/2852

#2 Updated by Bastian Waidelich about 4 years ago

  • Status changed from Accepted to Under Review

#3 Updated by Mr. Hudson about 4 years ago

Patch set 2 of change I56d54eefcaf4b2c2b57578503141e543efd12f68 has been pushed to the review server.
It is available at http://review.typo3.org/2852

#4 Updated by Mr. Hudson about 4 years ago

Patch set 3 of change I56d54eefcaf4b2c2b57578503141e543efd12f68 has been pushed to the review server.
It is available at http://review.typo3.org/2852

#5 Updated by Mr. Hudson about 4 years ago

Patch set 4 of change I56d54eefcaf4b2c2b57578503141e543efd12f68 has been pushed to the review server.
It is available at http://review.typo3.org/2852

#6 Updated by Bastian Waidelich about 4 years ago

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

Also available in: Atom PDF