[PR #787] [MERGED] Update DDC719Test.php to be compatible with MsSQL #8708

New Issue

admin · 2026-01-22T16:01:11+01:00

admin commented

2026-01-22 16:01:11 +01:00

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/787
Author: @flip111
Created: 9/13/2013
Status: ✅ Merged
Merged: 10/7/2013
Merged by: @guilhermeblanco

Base: master ← Head: patch-1

📝 Commits (3)

a190dad Update DDC719Test.php to be compatible with MsSQL
67dd32d A nicer way of detecting the platform
4282fa4 Used ternary operator to reduce LOC

📊 Changes

1 file changed (+4 additions, -2 deletions)

View changed files

📝 tests/Doctrine/Tests/ORM/Functional/Ticket/DDC719Test.php (+4 -2)

📄 Description

Apparently Doctrine adds "with (nolock)" now for the mssql platform, but the test has not been updated yet.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/doctrine/orm/pull/787 **Author:** [@flip111](https://github.com/flip111) **Created:** 9/13/2013 **Status:** ✅ Merged **Merged:** 10/7/2013 **Merged by:** [@guilhermeblanco](https://github.com/guilhermeblanco) **Base:** `master` ← **Head:** `patch-1` --- ### 📝 Commits (3) - [`a190dad`](https://github.com/doctrine/orm/commit/a190dad0b136bf2efc7ba3f860014731e6416845) Update DDC719Test.php to be compatible with MsSQL - [`67dd32d`](https://github.com/doctrine/orm/commit/67dd32d9fb9a7e1176a3add2719dd3040df97cf8) A nicer way of detecting the platform - [`4282fa4`](https://github.com/doctrine/orm/commit/4282fa478744e346f50c734a76b58f037c30d79c) Used ternary operator to reduce LOC ### 📊 Changes **1 file changed** (+4 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `tests/Doctrine/Tests/ORM/Functional/Ticket/DDC719Test.php` (+4 -2) </details> ### 📄 Description Apparently Doctrine adds "with (nolock)" now for the mssql platform, but the test has not been updated yet. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

admin added the pull-request label 2026-01-22 16:01:11 +01:00

admin closed this issue

2026-01-22 16:01:11 +01:00

admin referenced this issue

2026-01-22 16:09:50 +01:00

[PR #8708] Fix ClassMetadaInfo template inference #11136

Sign in to join this conversation.