[PR #572] [CLOSED] Fixed SQLServer ORDER BY problem in paginator CountOutputWalker #8387

New Issue

admin · 2026-01-22T15:59:41+01:00

admin commented

2026-01-22 15:59:41 +01:00

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/572
Author: @norberttech
Created: 2/7/2013
Status: ❌ Closed

Base: master ← Head: paginator

📝 Commits (3)

2842c25 Fixed SQLServer ORDER BY problem in paginator CountOutputWalker
6a7a625 Added test to check query with ORDER BY and SQLServerPlatform
3d8e114 ORDER BY removed from all count queries regardless of the platform

📊 Changes

2 files changed (+14 additions, -0 deletions)

View changed files

📝 lib/Doctrine/ORM/Tools/Pagination/CountOutputWalker.php (+1 -0)
📝 tests/Doctrine/Tests/ORM/Tools/Pagination/CountOutputWalkerTest.php (+13 -0)

📄 Description

Code explains everything :)

_{🔄 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/572 **Author:** [@norberttech](https://github.com/norberttech) **Created:** 2/7/2013 **Status:** ❌ Closed **Base:** `master` ← **Head:** `paginator` --- ### 📝 Commits (3) - [`2842c25`](https://github.com/doctrine/orm/commit/2842c254d26e105f1921b84f487430d6d4e8bae5) Fixed SQLServer ORDER BY problem in paginator CountOutputWalker - [`6a7a625`](https://github.com/doctrine/orm/commit/6a7a625af91e8cf642cd8df96f5b538402a14488) Added test to check query with ORDER BY and SQLServerPlatform - [`3d8e114`](https://github.com/doctrine/orm/commit/3d8e1149c65c22643a81ed967019a655cf2ffde5) ORDER BY removed from all count queries regardless of the platform ### 📊 Changes **2 files changed** (+14 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/Tools/Pagination/CountOutputWalker.php` (+1 -0) 📝 `tests/Doctrine/Tests/ORM/Tools/Pagination/CountOutputWalkerTest.php` (+13 -0) </details> ### 📄 Description Code explains everything :) --- <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 15:59:41 +01:00

admin closed this issue

2026-01-22 15:59:42 +01:00

admin referenced this issue

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

[PR #8467] [GH-8410] Fix memory leak in new toIterable and state bug. #11004

Sign in to join this conversation.