Removes commented lines

Author: elsamaryv

app/controllers/miq_ae_class_controller.rb

@@ -1,5 +1,4 @@
 require "rexml/document"
-require 'byebug'
 class MiqAeClassController < ApplicationController
   include MiqAeClassHelper
   include AutomateTreeHelper
@@ -883,19 +882,11 @@ def fields_form_field_changed
 
     fields_get_form_vars
     @changed = (@edit[:new] != @edit[:current])
-    # render :update do |page|
-      # page << javascript_prologue
       unless %w[up down].include?(params[:button])
         if params[:field_datatype] == "password"
-          # page << javascript_hide("field_default_value")
-          # page << javascript_show("field_password_value")
-          # page << "$('#field_password_value').val('');"
           session[:field_data][:default_value] =
             @edit[:new_field][:default_value] = ''
         elsif params[:field_datatype]
-          # page << javascript_hide("field_password_value")
-          # page << javascript_show("field_default_value")
-          # page << "$('#field_default_value').val('');"
           session[:field_data][:default_value] =
             @edit[:new_field][:default_value] = ''
         end
@@ -906,19 +897,11 @@ def fields_form_field_changed
           def_field = "fields_default_value_" << f[1].to_s
           pwd_field = "fields_password_value_" << f[1].to_s
           if @edit[:new][:fields][f[1].to_i]['datatype'] == "password"
-            # page << javascript_hide(def_field)
-            # page << javascript_show(pwd_field)
-            # page << "$('##{pwd_field}').val('');"
           else
-            # page << javascript_hide(pwd_field)
-            # page << javascript_show(def_field)
-            # page << "$('##{def_field}').val('');"
           end
           @edit[:new][:fields][f[1].to_i]['default_value'] = nil
         end
       end
-      # page << javascript_for_miq_button_visibility_changed(@changed)
-    # end
     render :json => {:message => 'Field updated successfully'}, :status => 200
   end
 
@@ -1440,7 +1423,7 @@ def create_namespace
   # AJAX driven routine to select a classification entry
   def field_select
     assert_privileges('miq_ae_field_edit')
-    fields_get_form_vars # nothing in particular handled here
+    fields_get_form_vars
     session[:field_data] = {}
     @edit[:new_field][:substitute] = session[:field_data][:substitute] = true
     render :json => {:status => 200}
@@ -2368,23 +2351,17 @@ def fields_get_form_vars
         end
       end
     elsif params[:button] == "accept"
-      # if session[:field_data][:name].blank? || session[:field_data][:aetype].blank?
       if params[:name].blank? || params[:aetype].blank?
-        # field = session[:field_data][:name].blank? ? "Name" : "Type"
-        # field += " and Type" if field == "Name" && session[:field_data][:aetype].blank?
         field = params[:name].blank? ? "Name" : "Type"
         field += " and Type" if field == "Name" && params[:aetype].blank?
         add_flash(_("%{field} is required") % {:field => field}, :error)
         return
       end
       new_fields = {}
       field_attributes.each do |field_attribute|
-        # new_fields[field_attribute] = @edit[:new_field][field_attribute.to_sym]
-        # new_fields[field_attribute] = params[:field][field_attribute.to_sym]
         new_fields[field_attribute] = params[field_attribute.to_sym]
       end
       @edit[:new][:fields].push(new_fields)
-      # @edit[:new_field] = session[:field_data] = {}
     end
   end
 

app/javascript/components/data-tables/datastore/helper.js

@@ -1,6 +1,4 @@
 import { headerData, rowData } from '../../miq-data-table/helper';
-// import miqFlash from '../../helpers/miq-flash';
-// import miqFlashClear from '../../helpers/miq-flash-clear';
 
 const hasCheckbox = (type, { domain, schema, fields }) => ![domain, schema, fields].includes(type);
 

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

@@ -142,7 +142,7 @@ export const ClassFieldsEditor = (props) => {
           updateState(data);
         })
         .catch((error) => {
-          console.error('Failed to save new field:', error);
+          // console.error('Failed to save new field:', error);
         });
     }
   };

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

@@ -13,17 +13,15 @@ const createSchemaEditSchema = (rows, setState, isSchemaModified) => {
         }));
       })
       .catch((error) => {
-        console.error('Failed to add new field:', error);
+        // console.error('Failed to add new field:', error);
       });
   };
 
   const handleFieldDelete = (rowId) => {
     const field = rows.find((field) => field.id === rowId);
-    // const field = rows.find((field, rowIndex) => rowIndex === index);
     const arrId = rows.findIndex((field) => field.id === rowId);
     const fId = field.field_id;
     const url = `/miq_ae_class/field_delete${fId ? `/${fId}` : ''}?arr_id=${arrId}`;
-    // const url = `/miq_ae_class/field_delete${fId ? `/${fId}` : ''}?arr_id=${rowId}`;
     http.post(url, { skipErrors: [400] })
       .then(() => {
         setState((prev) => ({
@@ -33,7 +31,7 @@ const createSchemaEditSchema = (rows, setState, isSchemaModified) => {
         }));
       })
       .catch((error) => {
-        console.error('Failed to delete field:', error);
+        // console.error('Failed to delete field:', error);
       });
   };
 
@@ -52,7 +50,6 @@ const createSchemaEditSchema = (rows, setState, isSchemaModified) => {
             rows,
             headers: schemaHeaders(true),
             onCellClick: (selectedRow) => {
-              debugger
               switch (selectedRow.callbackAction) {
                 case 'editClassField':
                   setState((prev) => ({

app/javascript/components/data-tables/datastore/schema/schema-table.jsx

@@ -11,7 +11,7 @@ export const SchemaTableComponent = (props) => {
   const formOptions = useFormApi();
 
   useEffect(() => {
-    input.onChange(rows); // Whenever props.rows changes, inform the form
+    input.onChange(rows);
   }, [rows]);
 
   return (
@@ -51,4 +51,4 @@ export const SchemaTableComponent = (props) => {
       </div>
     </div>
   );
-};
+};

app/views/miq_ae_class/_class_fields.html.haml

@@ -10,4 +10,4 @@
                  :locals  => {:message => _("No schema found")}
 
     - else
-      = render :partial => 'datastore_list', :locals => {:type => MiqAeClassHelper::DATASTORE_TYPES[:schema], :data => @ae_class.ae_fields}
+      = render :partial => 'datastore_list', :locals => {:type => MiqAeClassHelper::DATASTORE_TYPES[:schema], :data => @ae_class.ae_fields}