GeeH / advent

Advent of Code
0 stars 0 forks source link

[ZF3][RFC] View refactoring #5

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/3935 User: @MatyCZ Created On: 2013-03-02T15:28:49Z Updated At: 2014-06-12T10:51:43Z Body

User: @prolic Created On: 2013-03-02T19:33:37Z Updated At: 2013-03-02T19:33:37Z Body

looks good on first sight

Comment

User: @weierophinney Created On: 2013-03-11T21:39:20Z Updated At: 2013-03-11T21:39:20Z Body I think there are some great ideas here. However, you've removed methods that existed on the view models, and also removed methods from interfaces -- which means that functionality that was consumed before no longer will work. In other words, it's not backwards compatible.

That said, I still think they're good ideas; I've simply changed the milestone to 3.0.0, as that will be the earliest we can make these changes.

Comment

User: @MatyCZ Created On: 2013-06-12T10:09:52Z Updated At: 2013-06-12T10:09:52Z Body There is one Test which fails but I don't understand why:

Zend\View\ViewTest::testModelFromEventIsUsedByRenderer