Feature #33143

Allow Format.Currency ViewHelper regard Locale

Added by Adrian Föder over 3 years ago. Updated almost 3 years ago.

Status:Resolved Start date:2012-01-12
Priority:Could have Due date:
Assigned To:Adrian Föder % Done:

0%

Category:ViewHelpers
Target version:TYPO3 Flow Base Distribution - 2.0 beta 1
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 currencies New
duplicated by TYPO3.Fluid - Feature #36014: Prepend currency sign Resolved

Associated revisions

Revision afdbc774
Added by Adrian Föder almost 3 years ago

[FEATURE] Allow using Locale in Currency and Date ViewHelper

This enables switching the use of the current active Locale
in Format.Currency and Format.Date ViewHelper.

Change-Id: Id6de71d15d5e1cf08fea1dc4e6533d8c2e39927b
Resolves: #33143
Releases: 1.2

History

#1 Updated by Gerrit Code Review over 3 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 3 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 3 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 3 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 3 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 about 3 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 about 3 years ago

  • Target version changed from 1.1 to 1.1 RC1

#8 Updated by Karsten Dambekalns about 3 years ago

  • Target version changed from 1.1 RC1 to 2.0 beta 1

#9 Updated by Gerrit Code Review about 3 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 3 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 almost 3 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 almost 3 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 almost 3 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 almost 3 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 almost 3 years ago

  • Status changed from Under Review to Resolved

Related patch set has been merged.

Also available in: Atom PDF