[TYPO3-core] RFC #11358: Bug: typoLink() perfomance

[Rupert Germann]

hi Vladimir,

+1 by reading and testing

(the new vars $this->cache_getPage_noCheck and
$this->cache_getPageIdFromAlias should of course ne initialized)

I made a small test by adding a counter to function getPage_noCheck()
as you already sayd: the new cache saves 50 or more DB calls per click when
the page is not in cache.

really great improvement !

greets
rupert


Vladimir Podkovanov wrote:
> Hi all,
> 
> This is an SVN patch request.
> 
> Type: Bugfix
> 
> Bugtracker references:
> http://bugs.typo3.org/view.php?id=11358
> 
> Branches: TYPO3_4-1, TYPO3_4-2, trunk
> 
> 1)
> Problem:
> typolink uses ->getPage_noCheck() to check out if page's id exists.
> ->getPage_noCheck() do then DB request but result is not stored, so if
> some page contains 50 links to some other page (that is pretty common if
> the links goes to some plugin) then DB gets 50 same requests to check
> out if the id exists.
> 
> Solution:
> to cache results inside $TSFE->sys_page in internal var
> cache_getPage_noCheck
> 
> 2)
> Problem:
> typolink uses ->getPageIdFromAlias to get id from an alias. Same story
> here - result is not cached.
> 
> Solution:
> to cache results inside $TSFE->sys_page in internal var
> cache_getPageIdFromAlias
>