TYPO3 Core

Peter Kraume/The localization team asked for feedback:

TBH I really don't care much for this. To avoid issues I run some regex replacements from time to time on my projects.
If anyone is interested:

<trans-unit([^>]+)id="([^"]+)"([^>]+|)>
<trans-unit id="$2" resname="$2" approved="yes">

I do not know how much Weblate (which seems to need it according to the issues description) is used, but it seems actively maintained.
As I do not see much effort to fix all XLF files with the replacement shown above I would tend to say: Let's keep resname to stay compatible.

Google "weblate resname attribute":

The resname attribute is supposed to be human friendly identifier of the unit making it more suitable for Weblate to display instead of id . The resname has to be unique in the whole XLIFF file.

Our current habit of having identical "id" and "resname" is pointless. Even in weblate, "resname" is an optional additional information, potentially useful when it contains different content than "id".

As far as I can see, "resname" can and should be removed throughout the core. Did I miss something?

We discussed the idea in the Localization Team and we also don't see any issues which would block us from removing the resname attribute.

Suggestion:

find . -name \*\.xlf -exec sed -I "" -E 's/<trans-unit([^>]+)id="([^"]+)"([^>]+)>/<trans-unit id="\2">/' {} \;

Can we find in the history why we introduced it in the first place?
I remember we did not add them for fun, there was some good reason.

Markus Klein wrote in #note-6:

Can we find in the history why we introduced it in the first place?
I remember we did not add them for fun, there was some good reason.

It was needed for crowdin back then, obviously

When we do the change, we also need to update the documentation:
https://docs.typo3.org/m/typo3/reference-coreapi/main/en-us/ApiOverview/Localization/XliffFormat.html

Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87096

Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87096

Patch set 4 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87096

Patch set 5 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87096

Patch set 6 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87096

Patch set 7 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87096