Project

General

Profile

Actions

Task #69261

closed

Reduce PageLayoutController and PageLayoutView dependencies

Added by Oliver Hader almost 9 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Code Cleanup
Target version:
Start date:
2015-08-22
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
5.5
Tags:
Complexity:
Sprint Focus:

Description

This is a follow-up to #68395 to reduce complexity and exposed public API of that change.

Even if class components have the names "controller" and "view", which implies an underlying MVC-stack, the PageLayout components are far from using the pattern. That's why it's quite fine to put processing logic to the place where it makes most sense and where dependencies and cross-calls to other objects are kept low.

This change is only about refactoring and reducing the public API in terms of information hiding and encapsulation.
Besides that the $languagesInColumnCache class variable was defined but never use - which is changed now as well.
Further flaws concerning proper record selection are part of another change.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #68395: Make CE copy action more flexibleClosedAndreas Kienast2015-07-20

Actions
Actions #1

Updated by Gerrit Code Review almost 9 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42838

Actions #2

Updated by Oliver Hader over 8 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #3

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF