Feature #33143

Allow Format.Currency ViewHelper regard Locale

Added by Adrian Föder over 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
Could have
Assignee:
Category:
ViewHelpers
Start date:
2012-01-12
Due date:
% Done:

0%

Estimated time:
Has patch:
No

Description

The Format.Currency ViewHelper currently only renders its formatting "hard-coded". Allow usage of the Locale and Cldr functionality.


Related issues

Related to TYPO3.Flow - Feature #9313: Support for currenciesNew

Actions
Has duplicate TYPO3.Fluid - Feature #36014: Prepend currency signResolved

Actions
#1

Updated by Gerrit Code Review over 9 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#2

Updated by Adrian Föder over 9 years ago

What do you think about the idea and its implementation at all?

What I already thought to the end is that the currency itself has to be set explicitly always, a currency is always bound to the value to describe. It does not make sense to give a value and assign "any" currency wildly.
What may be considered is the position of the currency sign (leading or trailing); on one hand it is set at the CLDR information (see Framework\TYPO3.FLOW3\Resources\Private\Locale\CLDR\Sources\main\de_CH.xml in contrast to ...de.xml, in first case the currency symbol is leading, in latter case it's trailing.

So please share your thoughts, if we come to an end I will ad the date ViewHelper in a similar style to the look we've been discussing here than.

#3

Updated by Gerrit Code Review over 9 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#4

Updated by Gerrit Code Review over 9 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#5

Updated by Gerrit Code Review over 9 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#6

Updated by Gerrit Code Review over 9 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#7

Updated by Karsten Dambekalns over 9 years ago

  • Target version changed from 1.1 to 1.1 RC1
#8

Updated by Karsten Dambekalns over 9 years ago

  • Target version changed from 1.1 RC1 to 2.0 beta 1
#9

Updated by Gerrit Code Review about 9 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#10

Updated by Gerrit Code Review about 9 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#11

Updated by Gerrit Code Review about 9 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#12

Updated by Gerrit Code Review about 9 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#13

Updated by Gerrit Code Review about 9 years ago

Patch set 10 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#14

Updated by Gerrit Code Review about 9 years ago

Patch set 11 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7783

#15

Updated by Adrian Föder about 9 years ago

  • Status changed from Under Review to Resolved

Related patch set has been merged.

Also available in: Atom PDF