GlotPress is giving a warning when saving a record with unknown placeholder

[psmits1567]

WooCommerce uses placeholders that are not know by GlotPress
So if you save the record a warning is given, and a polyglots-warning is generated
This is a link to the offending translation
​https://translate.wordpress.org/projects/wp-plugins/yith-woocommerce-ajax-navigation/stable/ru/default/?filters[warnings]=yes&sort[by]=translation_date_added

[psmits1567]

Image showing the problem

[tobifjellner]

In order to spell it out for those who can't/won't see the attached image:
A printf() string like the following:

Data: %1$.2fMB | Index: %2$.2fMB | Free: %3$.2fMB | Engine: %4$s

Raises the warning:

Warning: The translation contains the following unexpected placeholders: %1$., %2$., %3$.

[dd32]

I've seen this message fly by in the warnings channel a few times, but I hadn't had a chance to look into adding full printf-syntax support.

I'm thinking one way around this is to leave the placeholder check as-is, but to also then do a if unknown placeholder exists at all within original, skip check.

[tobifjellner]

if unknown placeholder exists at all within original, skip check.

Yes, that's probably better than getting stuck in ever more complicated regex expressions.

[psmits1567]

I think that would be the best solution for the future. Unless the placeholder brakes functionality. Then it should still give a warning, but only if it is wrong formatted.

[Amieiro]

I have resolve this problem in GlotPress with this PR ​https://github.com/GlotPress/GlotPress-WP/pull/1271
Tomorrow I will review it with @dd32, and we will update it at ​https://translate.wordpress.org

[dd32]

In 11314:

Translate: Custom Warnings: Support precision within printf strings, and add an extra safety check to prevent warning about an incorrect placeholder if it existed within the original.

Fixes #5927.

[dd32]

In 11315:

Translate: Custom Warnings: Partially revert [11314] in favour of the upstream changes.

See ​https://github.com/GlotPress/GlotPress-WP/pull/1271.
See #5927.