[TYPO3-core] RFC #12429: Method "t3lib_tstemplate::procesIncludes" has misspelled "proces"
Xavier Perseguers
typo3 at perseguers.ch
Mon Nov 2 09:09:57 CET 2009
Hi Bernard,
> Solution:
> Rename the method. Create a wrapper function having the old name and mark it
> as deprecated and do a call to t3lib_div::logDeprecatedFunction()
As with all patches I sent regarding deprecation of methods, I
personally would like to read an explicit text telling the developer
that the method will be removed in TYPO3 4.5, just to be as explicit as
possible and to be able to easily remove those calls in 2 versions.
In addition, the second of the phpDoc is "Modifies" in original method
and in your wrapper. I guess the phrase was never written :-)
Thanks.
Besides that, +1 by reading.
--
Xavier Perseguers
MVC ExtJS Leader
http://forge.typo3.org/projects/show/extension-mvc_extjs
More information about the TYPO3-team-core
mailing list