tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 12:05:37 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #8874 from johannesx75/fix-pdoconnector-generatedid-type-2

Browse Source 
Fix PDOConnector GeneratedID return type
This commit is contained in:
Loz Calver 2019-03-22 09:09:23 +00:00 committed by GitHub
commit 8483a9644c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/ORM/Connect/PDOConnector.php
View File

@ -483,7 +483,7 @@ class PDOConnector extends DBConnector implements TransactionManager
public function getGeneratedID($table) public function getGeneratedID($table)
{ {
return $this->pdoConnection->lastInsertId(); return (int) $this->pdoConnection->lastInsertId();
} }
public function affectedRows() public function affectedRows()

11
tests/php/ORM/DataObjectTest.php
View File

@ -1925,6 +1925,17 @@ class DataObjectTest extends SapphireTest
$this->assertEquals('1988-03-04 06:30:00', $obj->DatetimeField); $this->assertEquals('1988-03-04 06:30:00', $obj->DatetimeField);
} }
/**
* Tests that the autogenerated ID is returned as int
*/
public function testIDFieldTypeAfterInsert()
{
$obj = new DataObjectTest\Fixture();
$obj->write();
$this->assertInternalType("int", $obj->ID);
}
public function testTwoSubclassesWithTheSameFieldNameWork() public function testTwoSubclassesWithTheSameFieldNameWork()
{ {
// Create two objects of different subclasses, setting the values of fields that are // Create two objects of different subclasses, setting the values of fields that are