I18nTextDomainFixer: remove unnecessary variable initialization

[rodrigoprimo]

While working on #2512, I noticed an unnecessary variable initialization, so I'm creating this PR to suggest that it be removed. Its value is immediately overridden in an if/else below.

https://github.com/rodrigoprimo/WordPress-Coding-Standards/blob/690f05a51d190586e0bb0426a0f04242224d0723/WordPress/Sniffs/Utils/I18nTextDomainFixerSniff.php#L727

https://github.com/rodrigoprimo/WordPress-Coding-Standards/blob/690f05a51d190586e0bb0426a0f04242224d0723/WordPress/Sniffs/Utils/I18nTextDomainFixerSniff.php#L749

[jrfnl]

LGTM.

[dingo-d]

We could probably run 'inspect code' function from PhpStorm and quickly see similar unused variables in the codebase (and potential other fixes from the suggestion)

[rodrigoprimo]

We could probably run 'inspect code' function from PhpStorm and quickly see similar unused variables in the codebase (and potential other fixes from the suggestion)

That is a good idea, @dingo-d. If you are not planning to do it yourself, I can try that. Just let me know what you prefer.

[dingo-d]

I mean if you're already working on it you can take over, but it's not a problem for me to do as well 😄

[jrfnl]

Sounds like a plan gentlemen ;-) Please do check history though when doing this as some of these things may have been put in place to satisfy a higher PHPStan level than we're currently running at and a commit message may hint at that, if that's the case.

[rodrigoprimo]

Just documenting here that I created the PR to remove unused variables detected by PHPStorm: #2514