Skip to content
This repository has been archived by the owner on Feb 5, 2024. It is now read-only.

NotFoundException should have a exception message #75

Open
enumag opened this issue Jan 4, 2016 · 0 comments
Open

NotFoundException should have a exception message #75

enumag opened this issue Jan 4, 2016 · 0 comments

Comments

@enumag
Copy link

enumag commented 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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants