Note: This issue is being posted here as an exercise by a participant in the http://drupalladder.org ladder for contributing to core. Please don't do anything with this post. The real issue has already been resolved. If you would like to try this exercise yourself, visit http://drupalladder.org and click on Drupal Ladder.

Current:
The statistics module settings (admin/config/system/statistics) need to be converted to use the new configuration system.

Proposed:

Tasks

  • Create a default config file and add it to the statistics module.
  • Convert the admin UI in statistics.admin.inc to read to/write from the appropriate config.
  • Convert any code that currently accesses the variables used to use the config system.
  • Write an upgrade path from D7 -> D8 to migrate existing data to the new system and drop the appropriate variables.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chiranjeeb2410 created an issue. See original summary.

chiranjeeb2410’s picture

Status: Active » Needs review

Re-rolled the original patch to apply to a later version of Drupal.

mallezie’s picture

Status: Needs review » Closed (won't fix)