Feature #14037

Translation files (XLIFF) must be possible per package

Added by Karsten Dambekalns over 4 years ago. Updated over 3 years ago.

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

100%

Category:I18n
Target version:TYPO3 Flow Base Distribution - 1.1 beta 1
PHP Version: Complexity:
Has patch:

Description

Currently the XliffTranslationProvider uses a hardcoded base path to look for XLIFF files of resource://FLOW3/Private/Locale/Translations/.

But it is necessary to have translations in each package, as it does not make sense to have labels for the Blog package in the FLOW3 package.

Also the TranslateViewHelper needs to offer the use of labels from different packages - e.g. to use labels from the Blog by default but fetch labels for the login screen from a shared file in the FLOW3 package.


Related issues

related to TYPO3.Flow - Task #11954: Check the current status of i18n / L10n Resolved

Associated revisions

Revision f5afd4fc
Added by Zach Davis over 3 years ago

[FEATURE] Allow translation files (XLIFF) on a per package basis

Fixes issue in which XliffTranslationProvider prefaces a hardcoded
base path of resource://FLOW3/Private/Locale/Translations before
the sourceName argument passed to getTranslationByOriginalLabel and
getTranslationById methods.

This fix modifies the translateById and translateByOriginalLabel
methods in the TYPO3\FLOW3\I18n\Translator to use packageKey
as an argument. The XliffTranslationProvider will construct a
base path of "<packageName>/Private/Locale/Translations". This base
path is prepended to the value of the sourceName argument to create
the translation file resource string.

Unit tests have been adjusted as well to accommodate these changes
to the XliffTranslationProvider and Translator classes. The
TranslationProviderInterface has been adjusted as well.

Change-Id: Iff62c0dc4cf47f8b2d5a9d143132ab64e2012891
Resolves: #14037
Releases: 1.1

History

#1 Updated by Karsten Dambekalns over 4 years ago

  • Status changed from New to Accepted
  • Assigned To set to Karsten Dambekalns

#2 Updated by Karsten Dambekalns about 4 years ago

  • Target version changed from 1.0 beta 1 to 1.1

#3 Updated by Mr. Hudson almost 4 years ago

  • Status changed from Accepted to Under Review

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

#4 Updated by Mr. Hudson almost 4 years ago

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

#5 Updated by Mr. Hudson almost 4 years ago

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

#6 Updated by Mr. Hudson almost 4 years ago

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

#7 Updated by Karsten Dambekalns almost 4 years ago

  • Affected Flow version set to FLOW3 1.0.0

#8 Updated by Karsten Dambekalns over 3 years ago

  • Tracker changed from Bug to Feature

#9 Updated by Mr. Hudson over 3 years ago

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

#10 Updated by Mr. Hudson over 3 years ago

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

#11 Updated by Mr. Jenkins over 3 years ago

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

#12 Updated by Mr. Jenkins over 3 years ago

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

#13 Updated by Gerrit Code Review over 3 years ago

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

#14 Updated by Karsten Dambekalns over 3 years ago

  • % Done changed from 0 to 100

#15 Updated by Karsten Dambekalns over 3 years ago

  • Status changed from Under Review to Resolved

#16 Updated by Karsten Dambekalns over 3 years ago

  • Target version changed from 1.1 to 1.1 beta 1

Also available in: Atom PDF