View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0028015||mantisbt||db schema||public||2021-03-08 08:46||2022-01-11 02:44|
|Summary||0028015: Update ADOdb to 5.21.0|
New ADOdb release https://github.com/ADOdb/ADOdb/blob/v5.21.0/docs/changelog.md
This needs to be tested carefully, as there's been quite a lot of changes since 5.20.x, particularly on mssql and to a lesser extent pgsql drivers.
|related to||0026837||closed||dregad||Update ADOdb to 5.20.20|
|parent of||0028068||assigned||dregad||Impossible to insert child records with ADOdb 5.21.0 on mssql|
|parent of||0028069||assigned||dregad||PHP notices leading to unusable system with ADOdb 5.21.0 on pgsql|
|parent of||0026599||new||Behavioural changes for BLOBs in ADOdb 5.21|
|Not all the children of this issue are yet resolved or closed.|
Breaks PostgreSQL - there are many notices as shown below, which cause other, sometimes fatal errors down the line
The regression was introduced by https://github.com/ADOdb/ADOdb/pull/367
@dregad Did some Tests on 2.24.5 (with ADOdb 5.21) for mssqlnative (SQLServer 2016, driver 5.9.0)
1) All views are Ok.
3) Add Project Ok
5) Add Note to Issue
Might be more. I'll check, but I doubt this is just mssql. Anything special about mysql not running into this?
BTW: 2.24.5 (with ADOdb 5.20.20) for mssqlnative (SQLServer 2016, driver 5.9.0), no problems so far.
EDIT (dregad): added markdown
Thanks for testing and reporting these details @obmsch, I'll look into it.
I have not done much manual testing yet. I just let the phpunit tests run via CI  and focused on the PostgreSQL failure so it is quite possible there are issues with MySQL too, but keep in mind that the latter is usually more error-tolerant so it would not be the first time that something fails on MSSQL but works just fine on MySQL.
@dregad Looks like the Id (Identity) of an INSERT is not set/retrieved.
Have to check, what has changed in the driver.
As thought, its a problem with the mssqlnative driver and its handling of the last inserted Id (Identity).
EDIT (dregad) added link to upstream issue.
Thanks Martin. As mentioned upstream