Automatically closed - issue fixed for 2 weeks with no activity.
If you set a table destination and use auto_increment: true
with batch_size: 2
the Table
destination plugin errors out on
$this->lastId = (int) $this->dbConnection->query("SELECT MAX($field) AS MaxId FROM {{$this->tableName}}")
->fetchField();
because fetchField()
returns a string and $this->lastId
is expected to be an integer.
The MigrateTableIncrementBatchTest
class is meant to test this but $this->batchSize
is never used by the parent class.
Cast the fetchField()
result to an int, remove the check for it not being set (it will be cast to 0 if there's no existing data) fix the test(s), and add a test case where the destination table already has at least one entry to check that the last id is fetched correctly.
None
None
None
None
Fixed
6.0
Plugins
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Automatically closed - issue fixed for 2 weeks with no activity.