Feature #81330: Provide a path to migrate internal class properties to protected - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Feature #81330

closed

Provide a path to migrate internal class properties to protected

Added by Elmar Hinz over 7 years ago. Updated almost 6 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Elmar Hinz

Category:

Code Cleanup

Target version:

9.0

Start date:

2017-05-24

Due date:

% Done:

100%

Estimated time:

PHP Version:

Tags:

Complexity:

Sprint Focus:

Description

A lot of properties of core classes are public since years, making it hard to refactor the classes. Every change is threatening to break extensions. This slows down the modernisation of TYPO3 a lot.

A path is needed to be able to set public properties to protected without immediately breaking extensions. This is generally reached by deprecation and period of transition to adjust extensions.

Deprecation works well for methods, as they can trigger the deprecation log directly. Provide this feature for properties, by triggering the depreciation log by using magic accessor methods to trigger the log.

Suggestion is to use a trait to temporarily include the feature into classes for periods of transition.

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

Status changed from New to Under Review

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

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#10

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#11

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#12

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#13

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#14

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#15

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#16

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#17

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#18

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#19

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#20

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#21

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#22

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#23

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#24

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#25

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

#26

Updated by Gerrit Code Review almost 7 years ago

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

Actions

Copy link

#27

Updated by Gerrit Code Review almost 7 years ago

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

Actions

Copy link

#28

Updated by Gerrit Code Review almost 7 years ago

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

Actions

Copy link

#29

Updated by Gerrit Code Review almost 7 years ago

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

Actions

Copy link

#30