issue #1 - Colocado teste do empty no lugar mais adequado.

carloshenrq · Aug 2, 2016 · 46b16ac · 46b16ac
1 parent e05b82f
commit 46b16ac
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/Controller/Account.php b/app/Controller/Account.php
@@ -1188,7 +1188,7 @@ public static function login(ServerRequestInterface $request, ResponseInterface
         $return = ['stage' => 0, 'loginSuccess' => false, 'loginError' => false];
 
         // Adicionado teste para recaptcha para segurança das requisições enviadas ao forms.
-        if(BRACP_RECAPTCHA_ENABLED && (empty($data['recaptcha']) || !self::getApp()->checkReCaptcha($data['recaptcha'])))
+        if(BRACP_RECAPTCHA_ENABLED && !self::getApp()->checkReCaptcha($data['recaptcha']))
         {
             $return['stage'] = 0;
             $return['loginError'] = true;

diff --git a/app/brACPApp.php b/app/brACPApp.php
@@ -138,7 +138,7 @@ public function checkReCaptcha($response)
     {
         // Se a configuração estiver desabilitada para o captcha,
         //  não enviar requisição, apenas retornar false.
-        if(!BRACP_RECAPTCHA_ENABLED)
+        if(!BRACP_RECAPTCHA_ENABLED || empty($response))
             return false;
 
         // Realiza a requisição da key no servido reCaptcha do google