Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

Suggestion: AbstractRestfulController:$identifierName should default to null #173

Open
GeeH opened this issue Jun 28, 2016 · 1 comment
Open

Comments

@GeeH
Copy link
Contributor

GeeH commented Jun 28, 2016

This issue has been moved from the zendframework repository as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


Original Issue: https://api.github.com/repos/zendframework/zendframework/issues/7395
User: @tylkomat
Created On: 2015-04-01T07:27:21Z
Updated At: 2015-11-06T21:05:28Z
Body
As pointed out on the mailing list strange things can happen when relying on the default value.

I propose to let it default to null and throw an exception if not set. That makes the usage for the developer much easier, as he will have to override it in his class.

Unfortunately this change is a BC-Break for the people that rely on the default value.


@weierophinney
Copy link
Member

This repository has been closed and moved to laminas/laminas-mvc; a new issue has been opened at laminas/laminas-mvc#28.

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

No branches or pull requests

2 participants