Fix the issues reported by PHPCS

-  if (empty($_POST['g-recaptcha-response']) || empty($recaptcha_secret_key)) {
+  if (empty($this->requestStack->getCurrentRequest()->request->get('g-recaptcha-response')) || empty($recaptcha_secret_key)) {

Outside of a class method, $this is not available.

   $resp = $recaptcha->verify(
-    $_POST['g-recaptcha-response'],
+    $request->get('g-recaptcha-response'),
     \Drupal::request()->getClientIp()

Given that the very next line is using \Drupal::request(), I take there is not a $request variable.

All that is left for this is replacing the use of $_POST.

MR updated with relevant changes. Ready for review now.

Just a thought on #8 comment: Noyiced the lines $_POST['g-recaptcha-response'] and \Drupal::request()->getClientIp() were added in different commits, may be that's why there is a difference? Am not 100% sure here, maintiner can confirm.

Thanks!

Anybody → closed merge request !13

Anybody → closed merge request !12