Проверка электронной почты из ссылки php

Question

Я пытаюсь создать проверку электронной почты. Отправка электронной почты со ссылкой на пользователя работает. Теперь я хочу установить active = 1, когда пользователь нажимает на ссылку, которую он получил. Я проверил переменные $ email и ключ $, они получают правильную информацию из URL-адреса. Когда для активного установлено значение 1, я хочу, чтобы echo ahref в login.php. Я думаю, что в моем SQL-запросе есть что-то неправильно, может кто-нибудь помочь?

<?php 

if (isset($_GET['email'])) { 
$email = $_GET['email']; 
} 
if (isset($_GET['hash'])){ 
$key = $_GET['hash']; 
} 

$query = $mysqli->query("UPDATE `users` SET active=1 WHERE `email` = '". $email ."' AND `mailcheck` ='". $key ."' "); 

$result = $query->fetch_row(); 

if($result == 1){ 

    echo "Your account is now active. You may now <a href="login.php">Log in</a>"; 

} 
else { 
echo "Your account could not be activated. Please recheck the link or contact the system administrator. test"; 
} 


} 
?> 

Answer 1

Держитесь здесь. fetch_row()http://php.net/manual/en/mysqli-result.fetch-row.php для SELECT, а не UPDATE.

Что вы хотите использовать это mysqli_affected_rows()

• http://php.net/manual/en/mysqli.affected-rows.php

на UPDATE, чтобы проверить, если обновление прошло успешно.

Если вы хотите сделать SELECT здесь (что имеет больше смысла), тогда вам нужно использовать mysqli_num_rows(), и если оба существуют, то выполните UPDATE.

• http://php.net/manual/en/mysqli-result.num-rows.php

Вы должны также проверить на наличие ошибок в отношении вашего запроса:

• http://php.net/manual/en/mysqli.error.php

---

Если строка/пользователь существует:

Проконсультируйтесь с моим ответом https://stackoverflow.com/a/22253579/1415724, чтобы узнать, существует ли пользователь, на котором вы можете основываться на нем.

---

Плюс, предложение. Используйте !empty() вместо isset(). Обычно лучше всего проверять ценности.

Что также лучше работает, это проверить, являются ли какие-либо пустые, а не 2 условными операторами.

Если один из них оставлен пустым, ваш код продолжит выполнение и, в свою очередь, ваш запрос не будет выполнен.

Если вы хотите сохранить свой нынешний метод, вы должны exit; после каждого GET, но я бы не рекомендовал его.

Больше как:

if (!empty($_GET['email']) && !empty($_GET['hash'])) { 
    $email = $_GET['email']; 
    $key = $_GET['hash']; 
} 

else{ exit; } 

---

Ваш Настоящий Кодекс открыт для SQL injection. Использовать mysqli_* with prepared statements, или PDO с prepared statements.

Answer 2

Проблема в том, что в следующей строке,

$result = $query->fetch_row(); 

Вы пытаетесь сделать UPDATE операцию, но вы на самом деле извлечения результата строку, используя ->fetch_row() заявление, которое, кстати, не существует, потому что UPDATE Операция не возвращает какой-либо результирующий набор.

Используйте ->affected_rows свойство, чтобы получить число изменяемых строк из UPDATE операции, например:

$mysqli->query("UPDATE `users` SET active=1 WHERE `email` = '". $email ."' AND `mailcheck` ='". $key ."'"); 

if($mysqli->affected_rows == 1){ 
    echo "Your account is now active. You may now <a href=\"login.php\">Log in</a>"; 
}else{ 
    echo "Your account could not be activated. Please recheck the link or contact the system administrator."; 
} 

Вот ссылка:

• mysqli::$affected_rows

Отредактировано:

Ваш код на странице проверки должны быть такими:

if(isset($_GET['email']) && isset($_GET['hash'])){ 
    $email = htmlentities($_GET['email']); 
    $key = htmlentities($_GET['hash']); 

    $mysqli->query("UPDATE `users` SET active=1 WHERE `email` = '". $email ."' AND `mailcheck` ='". $key ."'"); 

    if($mysqli->affected_rows){ 
     echo "Your account is now active. You may now <a href=\"login.php\">Log in</a>"; 
    }else{ 
     echo "Your account could not be activated. Please recheck the link or contact the system administrator."; 
    } 
}else{ 
    echo "wrong parameters."; 
} 

отредактировал Re-:

После обширной отладки с помощью OP вопрос решается сейчас, и это последний рабочий код,

if (isset($_GET['email']) && isset($_GET['hash'])) { 
    $email = $_GET['email']; 
    $key = $_GET['hash']; 

    $mysqli->query("UPDATE `users` SET active=1 WHERE `email` = '". $email ."' AND `mailcheck` ='". $key ."' "); 


    if($mysqli->affected_rows) { 

     echo "Your account is now active"; 

    }else { 
     echo "Failed"; 
    } 
}