[PR #427] [MERGED] Fix ORMInvalidArgumentException factory methods with return instead of throw #8171

New Issue

admin · 2026-01-22T15:58:44+01:00

admin commented

2026-01-22 15:58:44 +01:00

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/427
Author: @chEbba
Created: 8/19/2012
Status: ✅ Merged
Merged: 8/29/2012
Merged by: @beberlei

Base: 2.3 ← Head: return-exception

📝 Commits (1)

3aaa90e Fix ORMInvalidArgumentException factory methods with return instead of throw

📊 Changes

1 file changed (+6 additions, -6 deletions)

View changed files

📝 lib/Doctrine/ORM/ORMInvalidArgumentException.php (+6 -6)

📄 Description

Exception factories should not throw exceptions but just return them

_{🔄 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/427 **Author:** [@chEbba](https://github.com/chEbba) **Created:** 8/19/2012 **Status:** ✅ Merged **Merged:** 8/29/2012 **Merged by:** [@beberlei](https://github.com/beberlei) **Base:** `2.3` ← **Head:** `return-exception` --- ### 📝 Commits (1) - [`3aaa90e`](https://github.com/doctrine/orm/commit/3aaa90e1a8043cdfd2a4e4faaf14b0b5b02588e9) Fix ORMInvalidArgumentException factory methods with return instead of throw ### 📊 Changes **1 file changed** (+6 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/ORMInvalidArgumentException.php` (+6 -6) </details> ### 📄 Description Exception factories should not throw exceptions but just return them --- <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:58:44 +01:00

admin closed this issue

2026-01-22 15:58:45 +01:00

admin referenced this issue

2026-01-22 16:08:52 +01:00

[PR #8171] Use correct target branch info in PR templates #10842

Sign in to join this conversation.