The recent commit in #1209532: Count node views via AJAX in the statistics module broke the abillity to put a statistics.module in sites/all/modules, because statistics.js hardcodes core/modules/statistics as the module path.

See #1209532-155: Count node views via AJAX in the statistics module

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

This is kind of a moot point because the 'statistics' namespace is reserved by project.module and a module with that name cannot ever exist.

timmillwood’s picture

timmillwood
Primary language English
Location 🏴󠁧󠁢󠁷󠁬󠁳󠁿 Wales, UK
commented

yes, why would you want to put it in sites/all/modules?

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented

Well we use drupal_get_path() for core modules currently. And the whole point of that is to allow to override those modules completely from contrib*.
Is that used often?
Is it useful?
I don't know, probably not.
But it is a current feature, and the mentioned issue broke it. So if we don't want to support this, we might as well drop drupal_get_path() for core modules.

*I don't really think project.module registering a path like that is an argument in this case, as you can just as well write a custom statistics.module which overrides this. If we really had a good use-case for such a module, we would register drupal.org/project/statistics2. (Also, it's really a horrible thing of project.module to register certain dedicated paths on the same level as a wildcard path, but that's a different story.)

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented
Status: Active » Closed (duplicate)

This now again duplicate of the original issue: See #1209532-188: Count node views via AJAX in the statistics module