[TYPO3-core] RFC: Fix #7657: Add hook to $TSFE->contentStrReplace()

Michael Stucki michael at typo3.org
Sun Mar 2 00:52:15 CET 2008


REMINDER (I know it's too early, but beta3 is getting closer...)

After all this is a pretty simple hook.

- michael

Michael Stucki wrote:

> Oliver Hader wrote:
> 
>> Just two nits ;-)
> 
> Must be nits, yes :-)
> (I just copied the hook from a different place and modified it as
> needed...)
> 
>> * Please use
>> isset($this->TYPO3_CONF_VARS['SC_OPTIONS']['tslib/class.tslib_fe.php'
> ['tslib_fe-contentStrReplace'])
>> before the is_array() check of the same variable, or put it to a
>> reference and make the term a bit shorter, e.g.
>>    $contentStrReplaceHooks =&
>> $this->TYPO3_CONF_VARS['SC_OPTIONS']['tslib/class.tslib_fe.php'
> ['tslib_fe-contentStrReplace'];
> 
> Added isset() and reference.
> 
>> * Is the key 'pObj' in the parameters array required
>> (t3lib_div::callUserFunction() has '$this' too)?
> 
> Right, removed it. (Again, this was just copied without thinking :-))
> 
>> +1 on reading
> 
> For the rest, I have additionally added a test-hook to the indexed_search
> extension. It will simply replace all occurences of "Search" with "Find".
> Again: No caching is applied, and it only works if cacheHeaders are
> disabled!
> 
> - michael

-- 
Use a newsreader! Check out
http://typo3.org/community/mailing-lists/use-a-news-reader/


More information about the TYPO3-team-core mailing list