Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistency with tl_iso_product.price.0 translation #2136

Closed
fritzmg opened this issue May 3, 2020 · 5 comments
Closed

Inconsistency with tl_iso_product.price.0 translation #2136

fritzmg opened this issue May 3, 2020 · 5 comments
Labels
Milestone

Comments

@fritzmg
Copy link
Contributor

fritzmg commented May 3, 2020

The original English translation string for tl_iso_product.price.0 is Price & tax class (Price & tax class). However, in many of the other translations, it's just Price.

The following languages use Price:

  • da
  • de
  • fr
  • pl
  • ru

The following languages use Price & tax class:

  • nl
  • sl
  • tr

Since this translation is also used in the front end, e.g. for the product sorting options of the product filter module, it probably makes sense if the translation is just Price for all languages (including English), as "sort by price and tax class" does not make sense for front end users, since it actually only sorts by price and the tax class isn't typically visibly anyway.

As a side note - the following languages do not have a translation for tl_iso_product.price.0 yet:

  • cs
  • es
  • fa
  • fi
  • pt_BR
@aschempp
Copy link
Member

aschempp commented May 6, 2020

I wonder why the same label is used in the front end though? That does not really make sense to me, or does it to you?

@fritzmg
Copy link
Contributor Author

fritzmg commented May 6, 2020

I assume it automatically uses the translation from tl_iso_product.*, since you can add any custom field as a sorting or filter parameter?

@fritzmg
Copy link
Contributor Author

fritzmg commented May 8, 2020

See ProductFilter implementation:

$arrOptions[] = [
'label' => Format::dcaLabel('tl_iso_product', $field) . ', ' . $asc,
'value' => $field . ':ASC',
'default' => ($isDefault && $objSorting->isAscending()) ? '1' : '',
];
$arrOptions[] = [
'label' => Format::dcaLabel('tl_iso_product', $field) . ', ' . $desc,
'value' => $field . ':DESC',
'default' => ($isDefault && $objSorting->isDescending()) ? '1' : '',
];

@aschempp aschempp added the bug label May 8, 2020
@aschempp aschempp added this to the 2.6.11 milestone May 8, 2020
@fritzmg
Copy link
Contributor Author

fritzmg commented May 11, 2020

Shouldn't the labels for nl, sl and tr also be adjusted?

@aschempp
Copy link
Member

Thats up to the respective translation team (on Transifex).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants