t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead

Kindly review patch.

Hi I reviewed patch #2, applied it against Omnisend 1.1.0, and confirmed the issue is fixed.

I added screenshot for reference.
Thank You.

class OmnisendApi
-{
+{   

That is an off-topic change for an issue that aims to remove the calls to t() from classes code. It is also wrong, since the curly parenthesis is placed on the same line containing the class name, in Drupal.

I am also curious on why the issue decided to fix a single line, out of all the possible errors/warnings given by PHP_CodeSniffer.

Furthermore, the report given in 🐛 Fix the issues reported by phpcs Needs review already contains warnings/errors about using t() in classes.