Update a non-existing filter shows an error message.

Author: martineiber

src/Controller/Admin/Customers/FilterDefinitionController.php

@@ -115,6 +115,15 @@ public function updateAction(Request $request, CustomerViewInterface $customerVi
     {
         // fetch object parameters from request
         $filterDefinition = $this->getFilterDefinitionFromRequest($request, true, true);
+
+        // check if a filter is selected
+        if (empty($filterDefinition)) {
+            // add error message for missing filter
+            $errors[] = $customerView->translate('cmf_filter_definition_errors_filter_missing');
+
+            return $this->getRedirectToFilter(0, $errors);
+        }
+
         // check mandatory FilterDefinition name
         if (empty($filterDefinition->getName())) {
             // add error message for missing filter name

src/Resources/translations/admin.en.yml

@@ -116,6 +116,7 @@ cmf_filter_definition_errors_access: "Not allowed to access filter."
 cmf_filter_definition_errors_change: "Not allowed to change filter."
 cmf_filter_definition_errors_deletion_failed: "Deletion of filter failed. (%s)"
 cmf_filter_definition_errors_name_missing: "Please provide a filter name."
+cmf_filter_definition_errors_filter_missing: "Please select a filter."
 cmf_filter_definition_errors_save_failed: "Save of filter failed. (%s)"
 cmf_filter_definition_errors_share_failed: "Sharing filter failed. (%s)"