[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