Handles on click of save, reset and cancel with flash messages

Author: elsamaryv

app/controllers/miq_ae_class_controller.rb

@@ -281,6 +281,7 @@ def find_existing_node(parents)
 
   def replace_right_cell(options = {})
     @explorer = true
+    # byebug
     replace_trees = options[:replace_trees]
 
     # FIXME: is the following line needed?
@@ -1147,32 +1148,36 @@ def update_fields
     case params[:button]
     when "cancel"
       @sb[:action] = session[:edit] = nil # clean out the saved info
-      add_flash(_("Edit of schema for Automate Class \"%{name}\" was cancelled by the user") % {:name => @ae_class.name})
+      # add_flash(_("Edit of schema for Automate Class \"%{name}\" was cancelled by the user") % {:name => @ae_class.name})
       @in_a_form = false
-      replace_right_cell
-      # render :json => {:status => 200}
+      # replace_right_cell
+      message = _("Edit of schema for Automate Class \"%{name}\" was cancelled by the user") % {:name => @ae_class.name}
+      render :json => {:status => 200, :message => message}
     when "save"
-      # byebug
       ae_class = find_record_with_rbac(MiqAeClass, params[:id])
       begin
         MiqAeClass.transaction do
-          # byebug
           set_field_vars(ae_class)
           ae_class.ae_fields.destroy(MiqAeField.where(:id => @edit[:fields_to_delete]))
           ae_class.ae_fields.each { |fld| fld.default_value = nil if fld.default_value == "" }
           ae_class.save!
         end
       rescue StandardError => bang
-        add_flash(_("Error during 'save': %{error_message}") % {:error_message => bang.message}, :error)
+        # byebug
+        # add_flash(_("Error during 'save': %{error_message}") % {:error_message => bang.message}, :error)
         session[:changed] = @changed = true
         # javascript_flash
+        error_message = _("Error during 'save': %{error_message}") % {:error_message => bang.message}, :error
+        render :json => {:status => 500, :error => error_message}
       else
-        add_flash(_("Schema for Automate Class \"%{name}\" was saved") % {:name => ae_class.name})
+        # add_flash(_("Schema for Automate Class \"%{name}\" was saved") % {:name => ae_class.name})
         AuditEvent.success(build_saved_audit(ae_class, @edit))
         @sb[:action] = session[:edit] = nil # clean out the saved info
         # @in_a_form = false
-        replace_right_cell(:replace_trees => [:ae])
-        nil
+        # replace_right_cell(:replace_trees => [:ae])
+        # nil
+        success_message = _("Schema for Automate Class \"%{name}\" was saved") % {:name => ae_class.name}
+        render :json => {:status => 200, :message => success_message}
       end
     when "reset"
       fields_set_form_vars
@@ -1181,11 +1186,14 @@ def update_fields
       add_flash(_("All changes have been reset"), :warning)
       @button = "reset"
       @in_a_form = true
-      replace_right_cell
-      # render :json => {:status => 200}
+      # replace_right_cell
+      success_message = _("All changes have been reset")
+      render :json => {:status => 200, :message => success_message}
     else
+      byebug
       @changed = session[:changed] = (@edit[:new] != @edit[:current])
-      replace_right_cell(:replace_trees => [:ae])
+      # replace_right_cell(:replace_trees => [:ae])
+      render :json => {:status => 200}
     end
   end
 

app/javascript/components/data-tables/datastore/schema/class-fields-editor.jsx

@@ -11,6 +11,8 @@ import createClassFieldsSchema from './modal-form.schema';
 import createSchemaEditSchema from './class-fields-schema';
 import mapper from '../../../../forms/mappers/componentMapper';
 import { SchemaTableComponent } from './schema-table';
+import miqRedirectBack from '../../../../helpers/miq-redirect-back';
+import miqFlash from '../../../../helpers/miq-flash';
 
 export const ClassFieldsEditor = (props) => {
   const {
@@ -207,40 +209,40 @@ export const ClassFieldsEditor = (props) => {
   const onSchemaReset = () => {
     http.post(`/miq_ae_class/update_fields/${aeClassId}?button=reset`, { skipErrors: [400] })
       .then((response) => {
-        console.log(response);
-      }).catch((error) => {
-        console.error('Failed to reset schema:', error);
-      });
+        if (response.status === 200) {
+          miqRedirectBack(__(response.message), 'success', '/miq_ae_class/explorer');
+        } else {
+          miqSparkleOff();
+          miqFlash('error', response.error);
+        }
+      })
+      .catch(miqSparkleOff);
   };
 
-  const onSchemaSave = (values) => {
+  const onSchemaSave = () => {
     http.post(`/miq_ae_class/update_fields/${aeClassId}?button=save`, { skipErrors: [400] })
       .then((response) => {
-        console.log(response);
-      }).catch((error) => {
-        console.error('Failed to reset schema:', error);
-      });
+        if (response.status === 200) {
+          miqRedirectBack(__(response.message), 'success', '/miq_ae_class/explorer');
+        } else {
+          miqSparkleOff();
+          miqFlash('error', response.error);
+        }
+      })
+      .catch(miqSparkleOff);
   };
 
-  // const onCancel = () => {
-  //   miqSparkleOn();
-  //   const message = __('Edit of schema field was cancelled by the user');
-  //   miqRedirectBack(message, 'warning', '/miq_ae_class/explorer');
-  // };
-
-  // const onCancel = () => {
-  //   miqSparkleOn();
-  //   // miqAjaxButton(`/miq_ae_class/explorer`);
-  //   window.location.reload();
-  // };
-
   const onCancel = () => {
     http.post(`/miq_ae_class/update_fields/${aeClassId}?button=cancel`, { skipErrors: [400] })
       .then((response) => {
-        console.log(response);
-      }).catch((error) => {
-        console.error('Failed to cancel schema updates:', error);
-      });
+        if (response.status === 200) {
+          miqRedirectBack(__(response.message), 'success', '/miq_ae_class/explorer');
+        } else {
+          miqSparkleOff();
+          miqFlash('error', response.error);
+        }
+      })
+      .catch(miqSparkleOff);
   };
 
   return (