Download count settings includes download_count_last_cron which should really be a Drupal state variable instead of a setting

Created on 14 November 2020, about 4 years ago
Updated 12 July 2024, 5 months ago

Problem/Motivation

As download_count_last_cron gets updated on cron runs it then exports with config. On a site where config is tracked in git this shows up as extraneous changes.

Steps to reproduce

Install download_count, make no changes to the site but just let cron run, export config. Note that the system wants to export a new value for download_count.settings causing a developer who is deploying code to have to check whether live values have changed.

Proposed resolution

Use the States API β†’ for this variable instead. See the issue marked as related for an example.

User interface changes

None

API changes

Update the settings schema.

Data model changes

Remove download_count_last_cron from settings.

πŸ› Bug report
Status

Fixed

Version

2.0

Component

Code

Created by

πŸ‡¨πŸ‡¦Canada lolcode

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024