search

zendframework / zend-mvc

Mvc component from Zend Framework
BSD 3-Clause "New" or "Revised" License
105 stars 90 forks source link

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

Open GeeH opened 8 years ago

GeeH commented 8 years ago

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 commented 4 years ago

This repository has been closed and moved to laminas/laminas-mvc; a new issue has been opened at https://github.com/laminas/laminas-mvc/issues/28.