rector: `RemoveUnusedVariableInCatchRector`

[sreichel]

See https://getrector.com/rule-detail/remove-unused-variable-in-catch-rector

[sreichel]

Pls ignore Sonar.

[Hanmac]

i'm more curious why the Exceptions aren't used, not even logged with Mage::logException? And in most of the cases where i would throw a different exception, i would use the old exception as previous one.

Also since most of Mage core stuff does throw Throwable instead of Exception now, wouldn't it be better to catch Throwable instead, or is that too dangerous?

[sreichel]

I think there is some space for improvements. Want to oen a new issue or discussion for that?

[sonarqubecloud[bot]]

Quality Gate failed

Failed conditions
8.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

[sreichel]

Pls ignnore sonar.

(in free version it is not possible to adjust that limit)

[sonarqubecloud[bot]]

Quality Gate failed

Failed conditions
5.6% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

[sreichel]

Merged with one green.