Почему транзакция проскакивает и ничего не меняет?

Question

[Оптимус Пьян]

Код не замысловатый, смена пароля на новый по хешу с почты.
Проблема код проходит до конца выводя что $e = NULL при этом в БД нет никаких изменений, переменная $Changed которая как раз считает количество изменённых строк имеет значение 0.

try {
    $this->Connect->beginTransaction(); // начало транзакции

    $db = $this->Connect->prepare(" UPDATE `users` SET `pwd`=:pwd, `mailh`='' WHERE `mailh`=:hash ");
    $db->bindParam(":pwd",$new_password);
    $db->bindParam(":hash",$hash);
    $db->execute();
    $Changed = $db->rowCount();

    $this->Connect->commit();  // конец транзакции

} catch (\PDOException $e) {
    $this->Connect->rollBack();  // откат транзакции
}
echo $e; // NULL
echo $Changed;  // 0

Answer 1

[Александр Аксентьев]

} catch (\PDOException $e) {
    $this->Connect->rollBack();  // откат транзакции
}
echo $e; // NULL

это пять просто.
php.net/manual/en/language.exceptions.php

Запомните самую главную и важную команду: var_dump
Научитесь пользоваться, будете править миром.

Comments

[Оптимус Пьян]

var_dump умею полльзоваться, но что тут вардампить?

Прочитал, понял что мне надо где-то сделать throw new Exception('...'); но где и на какое событие его ловить?

[Александр Аксентьев]

Оптимус Пьян,

var_dump умею полльзоваться, но что тут вардампить?

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

Прочитал, понял что мне надо где-то сделать throw new Exception('...'); но где и на какое событие его ловить?

нет, надо где-то(в catch блоке) сделать $e->getMessage() и дампнуть его, и узнать в чем ошибка, если она есть.

$e нигде кроме catch() не видно т.к. это другая область видимости и оно передаётся туда.

[FanatPHP]

Мужчина. Сколько раз говорить, что бредовые идеи надо писать в комменты.

[FanatPHP]

> $e нигде кроме catch() не видно т.к. это другая область видимости и оно передаётся туда.

впрочем нет. тебе писать не надо вообще ничего. нужно молчать изо всех сил.

[Александр Аксентьев]

FanatPHP,
> молчать изо всех сил.
похоже что не только у меня это не получается.

[FanatPHP]

А мне и не нужно молчать. Я не делаю идиотских заявлений. За которые умному человеку было бы стыдно, а дураку-то конечно все равно.

[Александр Аксентьев]

FanatPHP, молчать можно по разным причинам.
Если например желчь из всех щелей тачёт и адекватно написать поправку/уточнение не получается.

[FanatPHP]

Нельзя написать поправку к идиотскому потоку мыслей.

Answer 2

[FanatPHP]

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

1. PDO error reporting надо установить на PDO::ERRMODE_EXCEPTION иначе ловить будет нечего
   
2. ловить надо Exception, а не PDOException
   
3. после отката исключение надо перевыбросить
   
4. убедиться что движок БД поддерживает транзакции (например InnoDB)
   
5. среди запросов нет такого, который модифицирует стркутуру БД
   

Ну и убедись, что в БД у тебя не лежат уже обновленные данные. Потому что в этом случае $Changed законно будет нулем.

Comments

[Оптимус Пьян]

Я помню ваш ответ про throw но что-то не догнал просто как его тут применить
4,5 да InnoDB, не модифицируется
1,2, думаю где-то тут косяк проверю эти пункты

P.S. Я на самом деле ждал именно когда вы придёте, т.к. знаю что по PDO всегда отвечаете ;)

[FanatPHP]

Оптимус Пьян, 2 не для этого случая, а вот 1 скорее всего.