- Issue created by @cristian100
- 🇲🇽Mexico cristian100 Tepic Nayarit
I've just added a patch, that adds all the necessary code for it to work along Drupal 11, the main thing was that the driver Connection.php, was extending DatabaseConnection class from core, back then, the core class removed the functionality from the functions upsert() and schema(), so I needed to add again that functionality but I added that in sqlsrv Connection class, also replaced some calling of classes that were updated in drupal 11, like StatementWrapperIterator.
Added a pull request to my fork, the changes are there now.
- 🇨🇦Canada dylan donkersgoed London, Ontario
Uploading a patch file with cristian100's changes above for use with composer.
- Status changed to Needs work
20 days ago 7:27pm 14 May 2025 Re-adding removed functionality should be in a different issue. Also, it wouldn't be good to merge a patch that contains commented out code, as that doesn't comply with Drupal coding standards.
The issue summary should be updated to reflect whether this is specifically for D11 compatibility (in which case this issue should be closed as a duplicate of 💬 Drupal 11 Compatibility Active ), or to include information about the return type functionality that was re-added.
Also, please use merge requests instead of patches. That way, the code review pipeline will run with the changes.