Issue #3425506 by drunken monkey, msnassar: Fixed unnecessary storing of...

Issue #3425506 by drunken monkey, msnassar: Fixed unnecessary storing of aggregated fields config values.

CHANGELOG.txt

 Search API 1.x, dev (xxxx-xx-xx):
 ---------------------------------
+- #3425506 by drunken monkey, msnassar: Fixed unnecessary storing of aggregated
+  fields config values.
 - #3431163 by drunken monkey: Fixed pipelines against Drupal 10.1 and 10.3.
 - #3427391 by joachim, matias, drunken monkey: Fixed position of "Cancel" link
   in field edit modal.

src/Plugin/search_api/processor/Property/AggregatedFieldProperty.php

@@ -132,6 +132,12 @@ class AggregatedFieldProperty extends ConfigurablePropertyBase {
       'separator' => stripcslashes($form_state->getValue('separator')),
       'fields' => array_keys(array_filter($form_state->getValue('fields'))),
     ];
+    // Do not store the default value for "separator" if it is not even used.
+    // This avoids needlessly cluttering the config export.
+    if ($values['type'] !== 'concat'
+        && $values['separator'] === $this->defaultConfiguration()['separator']) {
+      unset($values['separator']);
+    }
     $field->setConfiguration($values);
   }