backup_migrate_schedule_last_run variable should be store as state, not config

backdrop-contrib / backup_migrate

Port of the Drupal backup_migrate module to Backdrop

GNU General Public License v2.0

7 stars 14 forks source link

backup_migrate_schedule_last_run variable should be store as state, not config #43

Closed opi closed 6 years ago

opi commented 6 years ago

The variable backup_migrate_schedule_last_run_ID change upon each schedule run. It results as a diff in config managment, even if real config has no change.

I recommend saving this variable as a state (using state_set & state_get).

Before crafting a PR, I'd like some principle agreement.

herbdool commented 6 years ago

Absolutely. I agree.

opi commented 6 years ago

Created PR #44