Changeset 10956

Timestamp:

05/06/2021 05:55:52 AM (3 years ago)

Author:

dd32

Message:

Translate: When removing spaced placeholders, only do it if it ends on a word boundary.

This fixes it attempting to correct 100% over to 100%over, while still allowing it to convert <a href='% 1 $ s'> to <a href='%1$s'>.

Follow up to r10683.
See #5563, #5154.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-translation-fixer/wporg-gp-translation-fixer.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-translation-fixer/wporg-gp-translation-fixer.php

@@ -69 +69 @@
                         // Try replacing spaced translation type with no spaces `% 1 $ s` (Machine translated text)
                         $translation = preg_replace_callback(
-                            '!%\s?(\d+\s?\$(?:\d+)?)?\s?[bcdefgosux]!i',
+                            '!%\s?(\d+\s?\$(?:\d+)?)?\s?[bcdefgosux]\b!i',
                             function( $m ) {
                                 return str_replace( ' ', '', $m[0] );