Task #25099

simulate static methods of tslib_fe are deprecated but still used in the core

Added by Christian Kuhn over 4 years ago. Updated over 3 years ago.

Status:Resolved Start date:2011-02-19
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:- Spent time: -
Target version:-
TYPO3 Version:4.7 Complexity:
PHP Version:5.3 Sprint Focus:

Description

Method makeSimulFileName() and friends are deprecated and scheduled for removal in 4.6 in class tslib_fe but still used in the core.

Related to this: makeSimulatedFileNameCompat() of tx_simulatestatic

(issue imported from #M17666)


Related issues

related to Core - Task #39889: Add upgrade wizard to install extensions from TER Resolved 2012-08-17
related to Core - Task #31461: The Big Cleanup 4.7 Resolved 2012-02-19 2012-02-20
related to Core - Bug #43324: Add simulatestatic to upgrade wizard Resolved 2012-11-27
related to Core - Task #36025: Make simulatestatic TER ready Resolved 2012-04-14
precedes TypoScript Reference (TSref) - Task #42420: Add a note that the sysext simulatestatic has been removed Closed 2012-10-26

Associated revisions

Revision f078009b
Added by Markus Klein over 3 years ago

[CLEANUP] Remove simulate static from Core

This patch removes the old simulate_static system extension.
Since tslib_fe still contains idPartsAnalyze(), which is a relict of
simulate static methods, this method also removed.

Change-Id: I3631909fe6a77fd0861e7be5acdb6f3bf82fdb42
Resolves: #25099
Releases: 6.0
Reviewed-on: http://review.typo3.org/9132
Reviewed-by: Stefan Galinski
Tested-by: Stefan Galinski
Reviewed-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

Revision 3c0be8bf
Added by Christian Kuhn over 3 years ago

[TASK] Add note on EXT:simulatestatic to NEWS.txt

Change-Id: I13ef4c0a0a8d306ab83c2760628b3a6dc8040c16
Related: #25099
Releases: 6.0
Reviewed-on: http://review.typo3.org/10451
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

History

#1 Updated by Dmitry Dulepov over 4 years ago

I could find that this function is used only inside the other deprecated function.

#2 Updated by Markus Klein over 3 years ago

  • Target version deleted (0)

As of 4.7 the above mentioned functions are not present in Core anymore.
Only idPartsAnalyze() is still used in Core and marked for deletion in 4.5.

An exact copy of idPartsAnalyze() is already available in tx_simulatestatic, but the call to idPartsAnalyze() is not so easy to replace.
I'd suggest to introduce a hook, adding the possibility for a general id modification by an extension.

#3 Updated by Thorsten Kahler over 3 years ago

  • Category deleted (Communication)
  • Status changed from New to Accepted
  • Target version set to 4.7.0-beta1
  • TYPO3 Version changed from 4.5 to 4.7

#4 Updated by Gerrit Code Review over 3 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9132

#5 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9132

#6 Updated by Gerrit Code Review over 3 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9132

#7 Updated by Steffen Ritter over 3 years ago

  • Target version changed from 4.7.0-beta1 to 4.7.0-beta2

#8 Updated by Steffen Ritter over 3 years ago

  • Tracker changed from Bug to Task

#9 Updated by Steffen Ritter over 3 years ago

  • Target version deleted (4.7.0-beta2)

#10 Updated by Gerrit Code Review over 3 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9132

#11 Updated by Gerrit Code Review over 3 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9132

#12 Updated by Gerrit Code Review over 3 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9132

#13 Updated by Gerrit Code Review over 3 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9132

#14 Updated by Gerrit Code Review over 3 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9132

#15 Updated by Gerrit Code Review over 3 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9132

#16 Updated by Markus Klein over 3 years ago

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

Also available in: Atom PDF