Fix the issues reported by phpcs

Kindly review patch.

-- Externally authenticated users (via `externalauth` module) are excluded from validation and time-based expiration
+- Externally authenticated users (via `externalauth` module) are excluded
+from validation and time-based expiration

Since that text is part of a list, the last line must be indented.

 function _password_policy_history_insert_password_hash(AccountInterface $account) {
-  if (empty($_POST['pass']['pass1']) || empty($_POST['pass']['pass2'])) {
+  $request = \Drupal::service('request_stack')->getCurrentRequest();
+  $pass = $request->get('pass');
+  if (empty($pass['pass1']) || empty($pass['pass2'])) {
     return;
   }

_password_policy_history_insert_password_hash() is called by password_policy_history_user_update(), which is a hook_ENTITY_TYPE_update() implementation. It should not assume the password is contained in $_POST['pass']['pass1'] nor $request->get('pass'), as it is not necessarily called after a user form is submitted.

No patch has been provided after my review.

Addressed #5.1, #5.2 still needs to addressed.

Duplicate issue found. I'll need to close one of these.

📌 Fix the issues reported by phpcs Needs review

Closing this as duplicate of another issue that has had more work. I'll copy issue credits over.

📌 Fix the issues reported by phpcs Needs review