TYPO3 Core

On searching the Core for t3lib_timeTrack I found only one strange place where this is used (except the front-end rendering):
typo3/sysext/rtehtmlarea/htmlarea/plugins/SpellChecker/spell-check-logic.php

However, I don't know what effect the time tracker has in RTEhtmlarea...

Null object should have the same methods other PHP error will occur.

of course, we'll just introduce a new interface that both the TT that we have right now and the new null object do implement (with the same methods that TT has right now)

here's a version "0". This thing works but there's a significant problem with the whole time tracking:

The point in EXT:cms/tslib/index_ts.php where TT is instantiated is very early in the script. The only point where the information collected by TT is used is the adminpanel. Now whether the admin panel gets shown is decided way further down and you can't use the same condition used there to decide whether to use the real TT or the null TT at the point where TT is instantiated...

Any suggestions?

So here's v1 which decides which time tracker to load by checking for a cookie. That cookie gets set when the Typoscript section in the adminpanel gets opened.

For now the cookie setting/unsetting is done in PHP, but could also be moved to JS as soon as we have a dedicated admin panel JS class

The patch has also non–functional changes (like added "public" definitions) that should be left out. In addition it should be fixed to follow CGL ("{}" is not allowed).

Having an extra cookie is probably not good. I suppose we might look how TT is used and get rid of early messages. I doubt they are really useful. They were useful before real profilers appeared but now TT is more use for content rendering tracking than for profiling. So we can get rid of early TT usage and avoid cookies too.

I'm personally fine with another cookie that will only be set if admin panel is enabled. Usual FE users won't ever get this cookie set.

Another way to enable TT early in the script is to do the same check for _COOKIE('be_typo_user) like it's done for the BE_USER instantiation later in the script. This however might lead to false positives (cookie set, but BE user not "really" logged in). I could live with that as well.

A non representative, quick test showed me a positive performance impact of up to 8% for cached pages, I think that's worth the hassle.

actually the check for the BE user should be enough. I can't remember exactly why I added a new cookie back then

@Ingo: With "check for the BE user" you mean: "Check if the be_typo_user cookie is set"?

I'm preparing a patch with a simplified BE_USER check.
the current v1 patch also causes a problem with realurl because function debug_typo3PrintError() is called which is now protected.

added v2 of the patch.
changes compared to v1:
- the check for a logged in be_user is done by checking the be_user cookie
- I droppend the interface class. Benchmarks showed that it is faster without (little bit)

benchmark results:

cached pages:
trunk: 293.83 trans/sec 100%
patched: 308.95 trans/sec 105%

uncached pages:
trunk 34.47 trans/sec 100 %
patched 37.03 trans/sec 107 %

Details:

cached pages:

trunk r5856
---------------------------------------------------------------------
Transactions: 17762 hits
Availability: 100.00 %
Elapsed time: 60.45 secs
Data transferred: 439.07 MB
Response time: 0.05 secs
Transaction rate: 293.83 trans/sec
Throughput: 7.26 MB/sec
Concurrency: 14.96
Successful transactions: 17762
Failed transactions: 0
Longest transaction: 0.43
Shortest transaction: 0.00

patched 9657v2
---------------------------------------------------------------------
Transactions: 18534 hits
Availability: 100.00 %
Elapsed time: 59.99 secs
Data transferred: 458.64 MB
Response time: 0.05 secs
Transaction rate: 308.95 trans/sec
Throughput: 7.65 MB/sec
Concurrency: 14.97
Successful transactions: 18534
Failed transactions: 0
Longest transaction: 0.46
Shortest transaction: 0.00

uncached pages:

trunk r5856
-------------------------------------------------------------
Transactions: 2086 hits
Availability: 100.00 %
Elapsed time: 60.52 secs
Data transferred: 53.89 MB
Response time: 0.43 secs
Transaction rate: 34.47 trans/sec
Throughput: 0.89 MB/sec
Concurrency: 14.93
Successful transactions: 2086
Failed transactions: 0
Longest transaction: 1.49
Shortest transaction: 0.07

patched 9657v2
-------------------------------------------------------------
Transactions: 2227 hits
Availability: 100.00 %
Elapsed time: 60.14 secs
Data transferred: 57.53 MB
Response time: 0.40 secs
Transaction rate: 37.03 trans/sec
Throughput: 0.96 MB/sec
Concurrency: 14.92
Successful transactions: 2227
Failed transactions: 0
Longest transaction: 1.08
Shortest transaction: 0.07

committed v3 patch to trunk rev 5881