NotFoundException should have a exception message #27

New Issue

admin · 2026-01-23T11:34:52+01:00

admin commented

2026-01-23 11:34:52 +01:00

Originally created by @enumag on GitHub (Jan 4, 2016).

Many storages (if not all) throw the NotFoundException without any message. It would be better to have a exception message (containing the requested class and key).

Maybe it is not necessary for every storage to throw this exception - when the storage returns an empty value UnitOfWork::reconstititute() will throw the exception anyway and it would be nicer to have it at one place. This change would be a BC Break though.

Originally created by @enumag on GitHub (Jan 4, 2016). Many storages (if not all) throw the NotFoundException without any message. It would be better to have a exception message (containing the requested class and key). Maybe it is not necessary for every storage to throw this exception - when the storage returns an empty value `UnitOfWork::reconstititute()` will throw the exception anyway and it would be nicer to have it at one place. This change would be a BC Break though.

admin added the enhancement label 2026-01-23 11:34:52 +01:00

admin referenced this issue

2026-01-23 11:35:05 +01:00

[PR #32] [MERGED] Added common LICENSE #67

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/KeyValueStore#27