Manual function stores FID numbers in config unnecessarily

Created on 20 November 2023, 10 months ago
Updated 21 June 2024, 3 months ago

When using the manual delete function at /admin/config/content/fancy_file_delete/manual, the FID numbers entered into the field are saved in config when the form is submitted. This results in a diff each time drush cex is ran.

Unless I'm misunderstanding something, it seems like these config-related lines could be removed from FancyFileDeleteManual.php's submitForm() method. The class doesn't use config at all and this would prevent the input from being saved upon each use.

    $config = $this->config('fancy_file_delete.settings');

    foreach (Element::children($form) as $variable) {
      $config->set($variable, $form_state->getValue($form[$variable]['#parents']));
    }
    $config->save();
πŸ› Bug report
Status

Active

Version

2.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States cameron prince

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

Comments & Activities

Production build 0.71.5 2024