Task #42461
closed
Properly deprecate parameters which are not used anymore
0%
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.
Updated by Stefan Neufeind about 12 years ago
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
Updated by Thorsten Kahler about 12 years ago
- Status changed from New to Accepted
Updated by Mathias Schreiber almost 10 years ago
- 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
Updated by Alexander Opitz about 9 years ago
- 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.