Task #9136
Clean up View implementations
| Status: | Resolved | Start date: | 2010-08-04 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Bastian Waidelich | % Done: | 100% |
|
| Category: | MVC | |||
| Target version: | TYPO3 Flow Base Distribution - 1.0 alpha 11 | |||
| Sprint: | Has patch: | |||
| PHP Version: | Complexity: |
Description
Currently there are four classes implementing F3\FLOW3\MVC\View\ViewInterface in the FLOW3 package: AbstractView, EmptyView, JsonView and NotFoundView.
Some of them don't implement the interface correctly (assign() should return an instance of $this to allow chaining).
We decided to move the assignment functionality back to the AbstractView.
Obviously the code should still only reference the interfaces.
Besides, the AbstractView needs a cleaned up as it contains a lot of unneeded methods and objects.
Related issues
Associated revisions
[+TASK] FLOW3 (MVC): Cleaned up View implementations. They no correctly implement F3\FLOW3\MVC\View\ViewInterface. This resolves #9136
[+BUGFIX] FLOW3 (MVC): F3\FLOW3\MVC\View\JsonView no longer throws an exception if more than one variable is assigned. This relates to #9136
[+FEATURE] FLOW3 (MVC): F3\FLOW3\MVC\View\JsonView is now able to render multiple variables. By default only variables with the name "value" are rendered. You can change that behavior through the setVariablesToRender() setter. This relates to #9136
Change-Id: Ie7589504416eda13a3d57fe5ee411e5ea0cce0e4
History
#1 Updated by Bastian Waidelich almost 6 years ago
- Status changed from Accepted to Resolved
- % Done changed from 0 to 100
Applied in changeset r4931.
#2 Updated by Karsten Dambekalns almost 5 years ago
- Target version set to 1.0 alpha 11