Feature #12502

translate viewhelper should have optional parameter extensionName

Added by Georg Ringer over 4 years ago. Updated about 3 years ago.

Status:Resolved Start date:2011-01-24
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:ViewHelpers
Target version:-
Has patch:Yes

Description

The translate view helper works great bug would be even better if it would be possible to add the extensionName as argument.

examples:
1) Using a standalone view with Tx_Fluid_View_StandaloneView makes it impossible to work with translationViewHelper without duplicating translations

2) using example #1 can sometimes lead to have no extensionName available, therefore no translation can ever happen

3) having many extension in one project but very often the same translations leads to the problem having duplicate keys.

[pending in v4mvc list]

translate-extkey.patch Magnifier (1.5 kB) Georg Ringer, 2011-01-24 15:18

Associated revisions

Revision cf05b1fa
Added by Zach Davis over 3 years ago

[FEATURE] TranslateViewHelper accepts package argument

This change allows the TranslateViewHelper to accept an optional
package argument, which is passed to the Translator class for lookups.
If the package argument is NULL, the view helper will default to the
current controller's package key.

Change-Id: Ic2b9fe9e2ce20d37ec422c15acc76500e86063cf
Resolves: #12502
Related: #14037
Releases: 1.1

History

#1 Updated by Sebastian Kurfuerst about 4 years ago

  • Has patch set to Yes

#2 Updated by Mr. Jenkins over 3 years ago

Patch set 5 of change Ic2b9fe9e2ce20d37ec422c15acc76500e86063cf has been pushed to the review server.
It is available at http://review.typo3.org/4431

#3 Updated by Gerrit Code Review over 3 years ago

  • Status changed from New to Under Review

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

#4 Updated by Zach Davis over 3 years ago

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

Also available in: Atom PDF