Escape regex dots when stripping .min suffix

[mrfade]

Summary

Fixes an incorrect regex replacement that caused generated translation file hashes not to match WordPress’ expectations.

Fixes #456

Problem

The code used the following expression when stripping .min from file names:

preg_replace( "/.min.{$extension}$/", ".{$extension}", $file );

Because the . characters were not escaped, the regex matched any character, not a literal dot.
Example:

• Input: my-plugin-admin.js
• Regex consumed "dmin." (because . matched d)
• Output: my-plugin-a.js

This resulted in WordPress computing the wrong hash for the script source, so it tried to load a non-existent JSON translation file. As a result, JavaScript translations were never found.

Fix

Escape the dot characters in the regex so they are treated literally:

preg_replace( "/\.min\.{$extension}$/", ".{$extension}", $file );

Now only .min.js (or the correct extension) is matched, and file names remain intact (my-plugin-admin.js). The translation JSON filenames are generated with the correct hash and translations load as expected.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[swissspidy]

Good catch! Thanks for the PR. At first glsnce a separate test class just for this one line change is a bit overkill. Could we just add another Behag test case instead?

[mrfade]

You're absolutely right! A separate test class for testing one regex line is indeed overkill. I've removed the PHPUnit test and added a simple Behat test case instead.