Rebind PDO statement parameters on every execute

[simon-mundy]

Problem

Pdo\Statement::bindParametersFromContainer() returned early once $parametersBound was set, so re-executing a prepared statement with new values silently ran with the first values:

$statement->setSql('SELECT name FROM t WHERE id = :id');
$statement->execute(['id' => 1]); // 'one'
$statement->execute(['id' => 2]); // 'one' — expected 'two'

The same happened when passing a replacement ParameterContainer or mutating the existing container between executes. PDO's bindParam() binds by reference to a local copy of the container's values, so later container changes never reached the statement.

Fix

Remove the early return so parameters are bound from the current container on every execute(). The $parametersBound property is left in place for subclass compatibility.

Tests

• Three regression tests (array params, replacement container, mutated container) against an in-memory SQLite table; all failed before the fix.
• Removed two tests whose names described the old skip-when-bound behaviour; the new tests cover that path with real assertions.
• Pdo\Statement stays at 100% line coverage. mago format --check, mago lint and mago analyze are clean; one lint-baseline entry for Statement.php (kan-defect) no longer applies and has been dropped.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!