Task #42461
closed
Properly deprecate parameters which are not used anymore
Added by Stefan Neufeind about 12 years ago.
Updated about 9 years ago.
Description
We should check argument-counts and properly mark parameters as deprecated (with log-entries) so we can remove them in the future. Currently some parameters just have a phpdoc-comment.
A quick search on "not used" and something like "anymore" or "anywhere" gave me:
typo3/sysext/wizard_sortpages/Classes/View/SortPagesWizardModuleFunction.php: * Adds menu items... but I think this is not used at all. Looks very much like some testing code. If anyone cares to check it we can remove it some day...
typo3/sysext/rtehtmlarea/Classes/RteHtmlAreaBase.php: * @param boolean $concatenate Not used anymore
typo3/sysext/frontend/Classes/Page/PageRepository.php: * @param integer The expiration time (not used anymore)
typo3/sysext/core/Classes/TypoScript/ExtendedTemplateService.php: * @param array (not used anymore)
typo3/sysext/core/Classes/Utility/GeneralUtility.php: * @param string $checkPrefix Not used anymore since 6.0
typo3/sysext/core/Classes/Utility/GeneralUtility.php: * @param string $checkPrefix Not used anymore since 6.0
typo3/sysext/core/Classes/Utility/GeneralUtility.php: * @param boolean $silent Not used anymore since 6.0
typo3/sysext/wizard_crpages/Classes/Controller/CreatePagesWizardModuleFunctionController.php: * Adds menu items... but I think this is not used at all. Looks very much like some testing code. If anyone cares to check it we can remove it some day...
typo3/sysext/indexed_search/Classes/Indexer.php: * NOT USED anywhere inside this class!
typo3/sysext/backend/Classes/RecordList/AbstractRecordList.php: // OBSOLETE - NOT USED ANYMORE. leftMargin
typo3/sysext/backend/Classes/RecordList/AbstractRecordList.php: * @param integer OBSOLETE - NOT USED ANYMORE. $lMargin is the leftMargin (integer)
typo3/sysext/backend/Classes/Form/FormEngine.php: * (Not used anywhere...?)
typo3/sysext/backend/Classes/Form/FormEngine.php: * (Not used anywhere...?)
typo3/sysext/backend/Classes/Form/FormEngine.php: * Gets default record. Maybe not used anymore. FE-editor?
typo3/sysext/backend/Classes/Utility/BackendUtility.php: * @param string $thumbScript Optional: $thumbScript - not used anymore since FAL
- Status changed from New to Accepted
- Status changed from Accepted to Needs Feedback
- Assignee set to Mathias Schreiber
Hey Stefan, I think parts of this is done so we could at least shorten the functions that need to be marked as deprecated
What's the state of this issue?
- Status changed from Needs Feedback to Closed
- Assignee deleted (
Mathias Schreiber)
No feedback within the last 90 days => closing this issue.
If you think that this is the wrong decision or experience this issue again, then please write to the mailing list typo3.teams.bugs with issue number and an explanation or open a new ticket and add a relation to this ticket number.
Also available in: Atom
PDF