Fix the issues reported by phpcs

Above error/warnings has been fixed.

The patch fixes all phpcs issues. Marking RTBC!

- $message = $this->t('You have Rejected the invitation.');
+ // $message = $this->t('You have Rejected the invitation.');

Why is that line commented out? That is not change required to fix the issues reported by phpcs.

+  /**
+   * The entity type manager.
+   *
+   * @var \Drupal\Core\Entity\EntityTypeManagerInterface
+   */
+  protected $entityTypeManager;
+
+  /**
+   * The route match service.
+   *
+   * @var \Drupal\Core\Routing\RouteMatchInterface
+   */
+  protected $routeMatch;
+
+  /**
+   * The current user.
+   *
+   * @var \Drupal\Core\Session\AccountProxyInterface
+   */
+  protected $currentUser;
+
+  /**
+   * The email validator.
+   *
+   * @var \Drupal\Component\Utility\EmailValidatorInterface
+   */
+  protected $emailValidator;
+

Some of those properties are already defined from the parent class.

+  /**
+   * PrivateMessageIniviteForm form construct.
+   *

The documentation comment for a class is different and it should include the class namespace.

I will try fixing the issues. Thanks,

I checked that, $message is an unused variable as it was not being used anywhere in the function, hence, the line is commented.
Fixed the mentioned issues -
- corrected the misspelled class name
- added the namespace to class documentation comment
- reused the parent properties

-        $messenger = \Drupal::messenger();
+        $messenger = $this->messenger;

Since there is a property for the messenger, the code needs to be changed to use that property.

-    $config = \Drupal::service('config.factory')->getEditable('private_message_invite.settings');
+    $config = $this->configFactory->getEditable('private_message_invite.settings');

A class that extends ConfigFormBase uses the config() method, not that code. That is the same error done by the current code, but since that line is changed, it needs to be changed correctly.

+ *
+ * @package Drupal\private_message_invite\Form

@package is not used for classes, in Drupal.

+  /**
+   * Constructs PrivateMessageInviteForm.

The comment is still not using the class namespace. Then, it is not a class to be constructed, but an instance of that class (or object).

-    $values = $form_state->getValues();
+    // $values = $form_state->getValues();

If a variable is only initialized, but not used, the lines initializing (or changing the value of) that variable should be removed, not commented out.

Working on this.

Patch updated.
Please review

+  /**
+   * Constructs PrivateMessageInviteForm object.
+   *

The namespace is still not used, and there is an article missing in that phrase. (Then, the usual comment says it is a new object.)

Updated patch.
Please review.

Please ignore the last patch and interdiff.
Here is the updated paatch.

+    if ($form_state->hasValue(['custom', 'modules']) && is_array(
+      $form_state->getValue(['custom', 'modules']))) {

-          foreach ([MailsystemManager::MAILSYSTEM_TYPE_FORMATTING, MailsystemManager::MAILSYSTEM_TYPE_SENDING] as $type) {
+          foreach ([MailsystemManager::MAILSYSTEM_TYPE_FORMATTING, MailsystemManager::MAILSYSTEM_TYPE_SENDING,
+          ] as $type) {

It is still better to leave the control statement on a single line.

Hi @apaderno I test and review this patch #13, and it is applied very cleanly. But there has still one error remained. In PrivateMessageIniviteForm.php file. Adding error message below.:

FILE: ...Source/drupal-10-1/modules/contrib/private_message_invite/src/Form/PrivateMessageIniviteForm.php
------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------
36 | WARNING | Line exceeds 80 characters; contains 86 characters
------------------------------------------------------------------------------------------------------

added updated patch now it's remove remaining issues