I disagree.

Start with the case where transLevel = 0 (e.g. no calls to the transaction functions.)

beginTransaction() increments transLevel to 1.

Calling commit() or rollBack() with your code will try to ROLLBACK or RELEASE the save points where none have been created yet.

However with transLevel—; at the start of the function it will decrement it first then correctly try to call the parent::commit() or parent::rollBack() functions as appropriate.

public function rollBack() {

if(!$this->nestable() || $this->transLevel == 0) {
parent::rollBack();
} else {
$this->exec(“ROLLBACK TO SAVEPOINT LEVEL{$this->transLevel}”);
}

$this->transLevel–;
}

and :

public function commit() {
if(!$this->nestable() || $this->transLevel == 0) {
parent::commit();
} else {
$this->exec(“RELEASE SAVEPOINT LEVEL{$this->transLevel}”);
}
$this->transLevel–;
}

in the metods commit and rollback, it could be at the end of the function NOT in front.

Nice script indeed