Task #39347
closedRemove page hit statistics
Added by Mario Rimann over 12 years ago. Updated about 6 years ago.
100%
Description
For TYPO3 6.0, the included logging of each single page request should be removed in favor of using the well known tools like Google Analytics or Piwik.
Updated by Gerrit Code Review over 12 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229
Updated by Gerrit Code Review over 12 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229
Updated by Gerrit Code Review over 12 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229
Updated by Gerrit Code Review over 12 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229
Updated by Gerrit Code Review over 12 years ago
Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229
Updated by Gerrit Code Review over 12 years ago
Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229
Updated by Anonymous over 12 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 3136ffa942b6b4b6e2a4624722889c3fc9107991.
Updated by Bernhard Eckl over 11 years ago
So it is not possible to use statistic extension like ics_awstats anymore :-(
I think many people use this!
Updated by Bernhard Eckl over 11 years ago
In 4.7 config.stat_IP_anonymize and so on was added an in 6.0 everything removed?
Updated by Chris topher over 11 years ago
Bernhard Eckl wrote:
I think many people use this!
used this. ;-)
Updated by Clemens Riccabona about 11 years ago
Painful. Just painful.
I wonder sometimes, who makes decisions of such momentousness.
Ok, i read the core mailinglist, but not extra-carefully. Maybe I should change this behaviour. I did not recognize that there was made such a decision.
This now just means: now I have to configure statfile-writing via the webservers (apache in my case) vhost-files for each domain, configured in TYPO3, just to get awstats (e.g. ics_awstats) running.
before i could do it within TYPO3, where the statistics is presented to the author anyway.
Painful, really painful.
Updated by Christian Kuhn about 11 years ago
Extension like awstats could adapt and use some hook in the frontend to implement the logging on their own.
Updated by Clemens Riccabona about 11 years ago
Christian Kuhn wrote:
Extension like awstats could adapt and use some hook in the frontend to implement the logging on their own.
does it make sense, that functionality, which worked (at least for me) now about 10 (TEN) years, to remove from core and say: oh, if you want, you may hook.
nice to know that there is a hooking possibility for writing logfiles anyway.
but to me it does not make any sense.
was the code for writing logfiles in apache logformat so painful for the core developers, that it made sense for them to remove?
as i already stated, i did not recognize any discussion about that issue, i just stumbled upon luckily, today.
Therefore i am really interested in knowing why it is how it is now.
Updated by Christian Kuhn about 11 years ago
Yes, the code was so ugly, spaghetti, old, unmaintained, evil and cluttered that it actively denied refactoring of more important stuff.
Sometimes it is better to hand over some stuff to a dedicated extension that can handle this better and without a dependency to core code. We decided to do so in this very special case.
Updated by Clemens Riccabona about 11 years ago
Thanks Christian, for clarification of why this decision was made.
I am also interested in how I could prevent missing such important, feature-breaking decisions for the future, as I already told you, I stumbled upon this issue more or less by accident. (and i had great luck sticking strictly to LTS versions in the production. I don't want to think about what would had happened if I updated all my TYPO3 installations without knowing this fact!)
So my next feature-wish (as a paying assoc. member) is: please make a system extension which re-enables writing log-files in well documented and wide spread apache-log-format. ;) ;)
Maybe we could find some sponsors?
(yeah, i know, the official name of the log format is "NCSA Common log format", but most people I think know it by its "nickname").
Updated by Mario Rimann about 11 years ago
Hi Clemens
I agree that the removal of this old feature could be troublesome for a small percentage of users of TYPO3. When we discussed this issue, many votes were towards removing it, as most of the participants of that discussion mentioned that they anyway use a specialized tool like e.g. Piwik or Google Analytics (which provides way more information for the marketing team rather than pure hit counting that TYPO3 could log). So it seemed like the better way to remove it entirely - instead of carrying along old and hard-to-maintain code any further. And the 6.0 Release seemed to be the perfect moment for it - being a version that was clearly mentioned to contain breaking changes.
Regarding how to keep up to date, I agree that's not that easy - because there are just many changes between two versions - which is basically a good sign that there's something evolving.
If you look at the commit message of this change, you can clearly see it's marked with a bunch of exclamation marks and the "tag" "-FEATURE" - which should read: "Attention, breaking change of type feature removal". (https://review.typo3.org/#/c/13229/6//COMMIT_MSG)
I suggest you have a look at the change log which is part of each version's release notes. You don't need to read through all lines, but rather scan for lines with a "[!!!]" prefix - usually those are the things which will "break" something - or modify stuff in a way that needs you to adapt your previous work. Let's have a look at the Release Notes of 6.0.0 at http://wiki.typo3.org/TYPO3_6.0
This issue here was about removing the feature. If you'd like to get the feature back, please create a new issue and mention this one - and start a discussion on the dev- or core-mailinglist. This issue here is IMHO the wrong place for this discussion.
Best wishes,
Mario
Updated by Dan Kleine (Untenzu) almost 11 years ago
Clemens Riccabona wrote:
please make a system extension which re-enables writing log-files in well documented and wide spread apache-log-format
Hey Clemens, as Christian wrote, this should be part of a regular extension (and no system extension, since this is the same as having the code in the core again). Please create s Github Repository for such an extension and share it with others (maybe on a Forge Project?). Something like »awstats_logging« or »apache-style-logging«.
When this feature was removed I ported the removed logging code from the core into a hook of some AWStats extension, because I had to upgrade a project and support AWStats at this point of time. You can see the code here: https://bitbucket.org/pixelbrackets/typo3-xf_awstats/branch/writeLogfile
As proposed, this logging feature should be a single extension, without the AWStats analysis. I just dont want to maintain such an extension, thats why I did not put any effort in releasing it in a single extension.
As Mario states, further discussion should be made in a feature ticket.
Updated by Loek Hilgersom over 9 years ago
Just a note for anyone who still wants to use this feature, knowing that it comes with a performance penalty:
I created a little extension which copies the exact functionality for writing the Apache-style log files as used to be part of the core until v4.7.
The only issue to take care of is to add the [FE][logfile_dir] setting in AdditionalConfiguration.php instead of LocalConfiguration because the Install tool will remove it otherwise.
http://typo3.org/extensions/repository/view/retrostats
Now the feature is no longer lost, it has only become optional, all for the better :-)