clean up the code base

+++ b/tests/src/Functional/GoogleTranslatorTest.php
@@ -17,6 +18,7 @@ use Drupal\Core\Url;
 
+  use StringTranslationTrait;
   /**
    * A tmgmt_translator with a server mock.
    *
@@ -71,7 +73,7 @@ class GoogleTranslatorTest extends TMGMTTestBase {

@@ -71,7 +73,7 @@ class GoogleTranslatorTest extends TMGMTTestBase {
     $this->translator->save();
 
     $languages = $this->translator->getSupportedTargetLanguages('en');
-    $this->assertTrue(empty($languages), t('We can not get the languages using wrong api parameters.'));
+    $this->assertTrue(empty($languages), $this->t('We can not get the languages using wrong api parameters.'));
 

Let's just remove the t() on tests.

This is also going to conflict with #3025137: Convert request method to POST to get past 5000 character limit, maybe just leave out the message that is being removed there otherwise I'll wait with committing this until that is fixed.

Hi @Berdir
Removed t() function from the same line.
Please review.

The patch not working. Needs re-rolled.

I've cleaned some part of the code according to drupal standard.

I have removed most of the warnings and errors according to drupal standard and corrected the file paths in patch. This will work now.

Corrected path for files in patch. Please verify

Due to some confusion I was working on wrong version. I have update the code. Please review.

I will work on this.

I'll review it

Patch on #18 fixes all listed issues, is clean and passing reviews. Nicely done, @GersonJL!
Only partial tests were performed on the module at a live site, as it needs a Google API Key billable for translations. I can at least confirm the module is operational after patch, and this should suffice.