Possible [Bug] when handling transactions

[atlasphp]

Possible [Bug] when handling transactions

[pmjones]

@dijitaltrix I have merged the fix from @sunkan into the 2.x branch, with a refactoring of the logic therein. Try it out and let me know if it still works for you; if it does, I'll make a release.

Thanks, everyone, for your work here.

[pmjones]

Hi @dijitaltrix --

I am unable to reproduce the error you describe. I tried doing so in the issue-86 branch, via this commit: 0c7cd7e

Take a look at that test, and let me know how to make it match your case more closely.

[dijitaltrix]

Hi @pmjones,

Thanks for taking a look, it may be the issue is with MariaDB or the PDO driver.

I've run the tests from Issue-86 and can confirm they test ok against an sqlite memory database, I've not modified the test you supplied as the issue only seems to occur using MySQL and though I modified the phpunit tests in orm/tests they threw a few errors due to backticks and I wasn't confident of the results.

I've created a repository here https://github.com/dijitaltrix/Issue-86 with the minimum code required to show the issue. It follows the examples in the docs and the code in Issue86 very closely.

You'll see two files mysql_create.php and sqlite_create.php they are identical apart from the new AtlasContainer() statement.

The Sqlite code runs as excepted, it creates the parent and relations as it should, and when an Exception is thrown no records are created.

With MySQL the same code gives this error:

The Transaction failed:
persist Test\Parent\ParentRecord via Test\Parent\ParentMapper threw exception Expected 1 row affected, actual 0.

[sunkan]

Hi @dijitaltrix

I looked at your tests and the error is because you are trying to use time() on an mysql date type.

And that don't work so I change to date('Y-m-d') in you test data after that I get the correct exception.

A better question might be why PDO don't report the error from mysql

[dijitaltrix]

Hi @sunkan
Thanks for taking a look, I should have spotted that! As you say the example does run fine when given the correct data! I have updated the example as I'm still seeing the issue, please would you check again.

[dijitaltrix]

Hi @pmjones
I've updated my example, now it's looping through a data array and creating records off that. The issue only occurs if a subsequent transaction fails.

There's four test files now.. I assumed the issue was only occurring in MySQL but it also occurs in Sqlite, they're almost identical as before so take your pick!

I think the issue may be to do with the response from an Aura SQL call $this->pdo->inTransaction() it seems to return 1 on the first transaction but not on subsequent transactions.

[sunkan]

The problem is in Atlas\Orm\Table\ConnectionManager::getConnection

It checks if the connection already exists and just returns it.

It should also check inTransaction is true like this

if (isset($this->conn[$type][$tableClass])) {
    if ($this->inTransaction() && ! $this->conn[$type][$tableClass]->inTransaction()) {
        $this->conn[$type][$tableClass]->beginTransaction();
    }
    return $this->conn[$type][$tableClass];
}

I will put together a pull request in a an hour need to do some actual work now :)

@pmjones

[pmjones]

@sunkan I look forward to it, and thanks!

[dijitaltrix]

@sunkan commit #87 has fixed the issue for me, thank you!

[dijitaltrix]

@pmjones, @sunkan Thanks, I've tested the fix with my example code and it's working as expected, I assumed it was an issue with MySQL hence I created the example and didn't modify your unit tests. I'll write a test and create a pull request.

[pmjones]

Closing this as "bug has been fixed" and will wait for separate PR from @dijitaltrix for a test. Thanks everyone!