- ð³ð¿New Zealand quietone
I also tested this on Drupal 10.1.x and I was able to reproduce the problem. That is, I had to clear cache for the counts to update.
Statistics is approved for removal. See ð± [Policy] Deprecate Statistics module in D10 and move to contrib in D11 Fixed
This is now Postponed. The status is set according to two policies. The Remove a core extension and move it to a contributed project â and the Extensions approved for removal â policies.
It will be moved to a contributed Statistics project once the project is created and the Drupal 11 branch is open.
- ð¬ð§United Kingdom 2dareis2do
@quietone I am experiencing the same. Do you know what is causing this?
- Status changed to Needs work
about 1 year ago 1:27pm 25 January 2024 - ð¬ð§United Kingdom 2dareis2do
Changing to 10.2 as this appears to be a bug and statistics is scheduled to be moved to contrib in 11.x
- last update
about 1 year ago Patch Failed to Apply - ð¬ð§United Kingdom 2dareis2do
Ok I have rerolled previous patch. This contains both elements of #38 (working) and #39 (not working)
drupal_issues_2962763_test2.patch works for me locally.
I have not included the test as my assumption here is we probably need a new test for this, rather than changing the previous one. Not sure.
Counts are updating for now. This is without actually changing page cache (under performance) and the default views max-age cache on the view. Changing the cache to cache tags on the view seems to make little difference.
Browser still needs reloading to show the view count so I guess edge cache is still in place here, again not really sure.
- last update
about 1 year ago 25,793 pass, 1,810 fail - Merge request !7Issue #2962763 by lokapujya, drupal-ramesh, KapilV, vsujeetkumar, Andrew... â (Open) created by fgm
- Status changed to Needs review
10 months ago 1:58pm 3 May 2024 - ð«ð·France fgm Paris, France
Converted #59 latest patch by @2dareis2do to MR https://git.drupalcode.org/project/statistics/-/commit/c00a33433c3a243d6...
- ð«ð·France fgm Paris, France
@2dareis2do 's answer on Slack :
My patch is just a collection of other peoples commits so I am not sure I am the best qualified person to review this. This patch has been working for me though. In the meantime I have decided to restrict stats view to registered users as as well.
- ð¬ð§United Kingdom 2dareis2do
Looks like there are quite a few change in statistics module. with 10.3 release.
https://github.com/drupal/drupal/tree/10.3.1/core/modules/statistics
Patch does not apply and statistics are not updating.
Patch probably needs to be re rolled.
- ð«ð·France fgm Paris, France
Note that this issue is about the contrib statistics project, not statistics in core, but the link you mention https://github.com/drupal/drupal/tree/10.3.1/core/modules/statistics is about core.
- ð¬ð§United Kingdom 2dareis2do
Right sorry. my bad.
Am i right in thinking this started as a core issue?
- ð¬ð§United Kingdom 2dareis2do
I guess the issue will never be fixed in core if the issue queue is now in contrib.
- ð«ð·France fgm Paris, France
You're 100% right. Even more so now that 11.0.0 has been released and no longer contains statistics.
Updating issue to current Statistics release.
- ð®ð³India newswatch Delhi/Bangalore
The module worked fine till I started using it as a contrib module (after it was moved out of core).
Now, the counter does not move. No matter what.
If the counter doesn't move, then this module is of no use really.
- ð«ð·France fgm Paris, France
@newswatch you will need to provide more details about your configuration, possibly a a URL, to receive help. Without that we cannot know what does not work on your site.
First, can you reproduce the problem on a new site with just D11 core and statistics ?
- ð©ðªGermany Anybody Porta Westfalica
@fgm: MR LGTM, what's the further plan here?
- ð«ð·France fgm Paris, France
@anybody : awaiting a return from @newswatch since they appear to say it doesn't work for them, but I do not see how that can happen so we need more info.
- ð©ðªGermany Anybody Porta Westfalica
@fgm thanks! So if I get you right, you assume the MR maybe isn't needed?
Anyway I'd like to propose that lazy builders *might* be a nice solution for this case!
https://www.hashbangcode.com/article/drupal-9-using-lazy-builders - ð®ð³India newswatch Delhi/Bangalore
I installed the module on a site where it had not been installed earlier.
I waited for 3 days to check. The ticker is not rolling. All stats remain 0 for all nodes.
- ð¬ð§United Kingdom stevewilson
With version 1.0.0 of Statistics module, on Drupal 10.3.10 and PHP 8.1.30, I've run the following tests - first without and then with the changes in MR !7 applied:
Clear caches;
Load a page with number of views showing;
Note number of views;
Load a different page;
Reload the first page;
Note number of views.WITHOUT the MR !7 changes in place, on reloading the first page, the displayed number of views is unchanged from when the page was first loaded. Displayed number of views only increases after clearing caches.
WITH the MR !7 changes in place, on reloading the first page, the displayed number of views is incremented by 1 from when the page was first loaded. Further reloads result in further increments of the displayed number of views - no need to clear caches.
In other words, for me, the changes in MR !7 fix the failure to update issue.
- ð®ð³India newswatch Delhi/Bangalore
Since there is no documentation on changes required for shifting from core module of Drupal 10.3.0 to Statistics 1.0, I need a confirmation about thsi, please.
Drupal change record â says that the following should be removed
# Allow access to Statistics module's custom front controller. # Copy and adapt this rule to directly execute PHP files in contributed or # custom modules or to run another PHP application in the same directory. RewriteCond %{REQUEST_URI} !/core/modules/statistics/statistics\.php$
Now, I have done that. But what should it be replaced with (if at all)?