MySQL, очередная проблема?

Question

[The-TS]

Возможно, я повторил свой вопрос. Простите
Мой вопрос:

Что я, в очередной, раз делаю неправильно?
Почему SQL-запрос не выполняется?

Да, в консоли PHPMyAdmin запрос выполняется правильно
-------------------------------------------------------------------------
Вот код:

<?php
  //Require-скрипты
  require 'Authorize.php';
  require 'connect.php';

  //ID сессии, старт сессии
  session_id($_COOKIE['PHPSESSID']);
  session_start();

  $ID = (integer) GetID();
  var_dump($_COOKIE['PHPSESSID']);

  $sql = "SELECT `provider` FROM `Users` WHERE 'ID'=:ID";
  $query = $pdo->prepare($sql);
  $query->execute([
    "ID" => $ID
]);
  $row = $query->fetch(PDO::FETCH_OBJ);

  if($row->provider = "Google") {
    $Authorized = Authorize_Google();

    if($Authorized == true) {
      if(!empty($_COOKIE['session_id'])) {
        $VerificationCode = $_GET['Code'];

        if($VerificationCode == $_SESSION['vereficationCode']) {
          $sql = 'UPDATE `Users` SET `email_verified`=1 WHERE `ID` = :ID'; // <= Проблема в этом запросе
          $query = $pdo->prepare($sql);                                                       
          $query->execute([
            "ID" => $ID
          ]);
          echo "Ваша почта подтверждена!";
        }
      }
    } else {
      echo "<h1 align='center'>Вы не авторизованы, <a href='google.php'>Войдите через Google</a></h1>";
    }
  } else {
    $Authorized = Authorized($_COOKIE['session_id']);
    if($Authorized === true) {
      $VerificationCode = $_GET['Code'];

      if($VerificationCode == $_SESSION['vereficationCode']) {
        $sql = 'UPDATE `Users` SET `email_verified`=1 WHERE `ID` = :ID';
        $query = $pdo->prepare($sql);
        $query->execute([
          "ID" => $ID
        ]);
        $row = $query->fetch(PDO::FETCH_OBJ);
        var_dump($ID);
        echo "Ваша почта подтверждена!";
      }
    } else {
      echo "<h1 align='center'>Вы не авторизированы!</h1>";
      http_response_code(401);
    }
  }

Спасибо

Comments

[maxim]

Ну что я помог ?

Answer 1

[FanatPHP]

проблема не во втором запросе, а в первом. Имя поля взято в одинарные кавычки.
Судя по всему, в коде нет никаких настроек которые бы сообщали об ошибках.

Соединяться надо как написано здесь
и плюс добавить error_reporting(E_ALL);

Comments

[The-TS]

Спасибо за помощь, возможно, вот проблема:

PHP Fatal error:  Uncaught PDOException: SQLSTATE[28000] [1045] Access denied for user 'tsecret1'@'localhost' (using password: NO) in /home/tsecret1/Test-Server/AUTH-2/connect.php:13

Как исправить? Уч. Данные верны

[necrodeflorator]

secret secretom, вам по английски написано что не так. Доступ пользователя запрещён в бд.
Намного быстрее эту ошибку загуглить чем сюда писать. Пхп вообще в этом плане очень дружелюбный язык, всегда подскажет где вы ошиблись, не надо только лишать его такой возможности. И читайте внимательно что он вам пишет

[FanatPHP]

Учётные данные неверны. Пароль не задан

[The-TS]

FanatPHP, Пароль задан

[FanatPHP]

пароль не задан
запомни простое правило: выбирая между своими фантазиями и тем что тебе говорит РНР - всегда выбирай второе.
у тебя в сообщении об ошибке черным по белому написано, using password: NO
то есть ты только думаешь что пароль задан, но на самом деле он пустой

[The-TS]

FanatPHP, Хорошо:
Вот код:

<?php
  $dsn = 'mysql:host=localhost;dbname=tsecret1_DataBase;';
  $opt = [
    PDO::ATTR_ERRMODE            => PDO::ERRMODE_EXCEPTION,
    PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
    PDO::ATTR_EMULATE_PREPARES   => false,
  ];
  $pdo = new PDO($dsn, 'tsecret1_TEST', 'r7Pz****e6mx', $opt);

  try {
       $pdo = new \PDO($dsn, $user, $pass, $options);
  } catch (PDOException $e) {
       throw new PDOException($e->getMessage(), (int)$e->getCode());
  }
?>

Что я делаю не так?

[FanatPHP]

скорее всего ты вызываешь не этот код, а какой-то другой.
для проверки убери try catch, оставь только
$pdo = new \PDO($dsn, $user, $pass, $options);
и напиши сюда что тебе вывело

[The-TS]

FanatPHP, Понял ошибку - простите мою тупость.
Но 1 все еще не записываеться

[FanatPHP]

кавычки в первом запросе исправлены?
error_reporting(E_ALL); стоит?

[The-TS]

FanatPHP,
- "error_reporting(E_ALL); стоит?" Да
- "кавычки в первом запросе исправлены?" Так?:

$sql ="UPDATE `Users` SET `email_verified`=1 WHERE `ID`=:ID";
          $query = $pdo->prepare($sql);
          $query->execute([
            "ID" => $ID
          ]);

[FanatPHP]

Так

[The-TS]

FanatPHP, Все еще не работает...
:(

[FanatPHP]

разбирайся
перед выполнением запроса выведи id
навернякак такой же глупуй косяк где-то как с соединением было
я ставлю на то что запрос вообще не вызывается из-за условия выше

[The-TS]

FanatPHP,
Прости, проблема была в самой переменной $ID.
Спасибо за по траченое время, Спасибо за ответ

Answer 2

[maxim]

Попробуй завернуть ID в переменную, а sql так:

$sql = “UPDATE `Users` SET `email_verified`=1 WHERE `ID` =  ‘{$id}’ “;

Ручками там если что поправь, писал не в редакторе.

Comments

[FanatPHP]

Пожалуйста, сотрите свой ответ.
Вы не понимаете как работают подготовленные выражения, которые используются в вопросе.
"Заворачивать" переменные в запрос категорически не рекомендуется

[maxim]

FanatPHP, и зачем стирать ?)

[maxim]

FanatPHP, я сказал просто попробовать

[FanatPHP]

Потому что он неграмотный и вредный.
Не надо ничего пробовать.
Весь мир давно использует подготовленные выражения. И автор пусть продолжает их использовать

[maxim]

FanatPHP, ну ладно ) а ты что предложишь ?

[FanatPHP]

я уже предложил

[FanatPHP]

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

[maxim]

FanatPHP, а не заметил

Answer 3

[John Didact]

У тебя ID в одинарных кавычках и

$query->execute([
 ":ID" => $ID
]);

вроде бы, точно не помню))

UPD. Проверил в мануале (скрин прикрепил) скорее всего, я был прав. Ваш код просмотрел бегло, хорошо было бы ещё написать сюда номер и сообщение ошибки, версию PHP/PDO

Comments

[FanatPHP]

стирай

[John Didact]

FanatPHP, зачем?

[FanatPHP]

Ну понятно зачем
Потому что бесполезный
Никогда не пиши ответов про технологии, с которыми не работал
В этом случае даже мануал не поможет, потому что ты поймёшь его неправильно

[John Didact]

FanatPHP, не сотру

[FanatPHP]

Не стирать можно
Тупить не стоит

[John Didact]

FanatPHP, ну дак не тупи, мне то об этом зачем знать?

[FanatPHP]

Какая лапочка :)
Обгадился, а по виду и не скажешь )))

[John Didact]

FanatPHP, считаешь, что обладаешь экстрасенсорными навыками?) Тебя разочаровать?

[FanatPHP]

завязывай в общем с этим
ты когда хочешь учиться то нормальный чувак
но когда ты тупишь, то становишься каким-то вурдалаком.

нет ничего постыдного в том чтобы ступить. это нормально. все тупят, пишут ерунду, ошибаются.
нет ничего постыдного в том чтобы признать свою неправоту

а вот продолжать тупить когда тебе сказали что ты неправи, да ещё начинать лаять в ответ - это ты уже начинаешь позорить себя перед окружающими
подумай над своим поведением

[John Didact]

FanatPHP, с чем завязывать?

Давай посмотрим, до чего вы дойдёте с автором вопроса, потом, если окажется, что я реально ступил, то я это признаю…

З. Ы. Я не утверждаю, что те ошибки, на которые указал ты - не ошибки. Перепрочти мой ответ и пойми его смысл.

З. З. Ы. Своё утверждение я, к сожалению, не проверял (жду, когда автор дойдет до этой ошибки), так как не до этого. Если ты помнишь меня и мои слова, то, скорее всего, помнишь и то, что этому всему я практически время не уделяю, поэтому, собственно, и не проверял.

[FanatPHP]

Своё утверждение я, к сожалению, не проверял

Вот это твоя главная проблема.
Не проверял, но споришь с тем кто проверял.