Removes log statements and commented lines

elsamaryv · elsamaryv · commit ebb2bc33d348 · 2025-10-07T14:44:15.000+05:30
diff --git a/app/controllers/miq_ae_customization_controller.rb b/app/controllers/miq_ae_customization_controller.rb
@@ -140,11 +140,9 @@ def explorer
   end
 
   def editor
-    puts "Paramss - #{params}"
     if params[:id].present?
       feature = 'dialog_edit_editor'
       @record = Dialog.find(params[:id])
-      puts "Record1 - #{@record.as_json}"
     elsif params[:copy].present?
       feature = 'dialog_copy_editor'
       @record = Dialog.find(params[:copy])
diff --git a/app/javascript/components/service-dialog-form/dynamic-field.jsx b/app/javascript/components/service-dialog-form/dynamic-field.jsx
@@ -11,38 +11,7 @@ import DynamicTagControl from './dynamic-fields/dynamic-tag-control';
 import { dynamicFieldDataProps } from './helper';
 /** Component to render a Field. */
 const DynamicField = ({ fieldData, onFieldAction }) => {
-  const fieldSelector = (fieldData) => {
-    console.log('Field data in DynamicField:', fieldData);
-    
-    // Make sure we have a componentId
-    const componentId = fieldData.field.componentId ||
-                       (fieldData.field.type && getComponentIdFromType(fieldData.field.type)) ||
-                       1; // Default to text box
-    
-    console.log('Using componentId:', componentId);
-    
-    switch (componentId) {
-      case 1:
-        return <DynamicTextInput dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
-      case 2:
-        return <DynamicTextArea dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
-      case 3:
-        return <DynamicCheckbox dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
-      case 4:
-        return <DynamicDropdown dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
-      case 5:
-        return <DynamicRadioButton dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
-      case 6:
-        return <DynamicDatepicker dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
-      case 7:
-        return <DynamicTimepicker dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
-      case 8:
-        return <DynamicTagControl dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
-      default:
-        return <DynamicTextInput dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
-    }
-  };
-  
+
   // Helper function to determine componentId from field type
   const getComponentIdFromType = (type) => {
     switch (type) {
@@ -67,6 +36,33 @@ const DynamicField = ({ fieldData, onFieldAction }) => {
     }
   };
 
+  const fieldSelector = (fieldData) => {
+    const componentId = fieldData.field.componentId
+                       || (fieldData.field.type && getComponentIdFromType(fieldData.field.type))
+                       || 1; // Default to text box
+
+    switch (componentId) {
+      case 1:
+        return <DynamicTextInput dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
+      case 2:
+        return <DynamicTextArea dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
+      case 3:
+        return <DynamicCheckbox dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
+      case 4:
+        return <DynamicDropdown dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
+      case 5:
+        return <DynamicRadioButton dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
+      case 6:
+        return <DynamicDatepicker dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
+      case 7:
+        return <DynamicTimepicker dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
+      case 8:
+        return <DynamicTagControl dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
+      default:
+        return <DynamicTextInput dynamicFieldData={fieldData} onFieldAction={(newFieldData) => onFieldAction(newFieldData)} />;
+    }
+  };
+
   return (
     <>
       {
diff --git a/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-checkbox.jsx b/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-checkbox.jsx
@@ -29,11 +29,6 @@ const DynamicCheckbox = ({ dynamicFieldData: { section, field, fieldPosition },
     fieldsToRefresh: refreshEnabledFields,
   });
 
-  // Log the field data for debugging
-  console.log('Field data in DynamicCheckbox:', field);
-  console.log('Field values from helper:', fieldValues);
-  console.log('Field state initialized as:', fieldState);
-
   const handleFieldUpdate = (event, updatedFields) => {
     setFieldState((prevState) => ({ ...prevState, ...updatedFields }));
     onFieldAction({ event, type: editActionType, fieldPosition, inputProps: { ...fieldState, ...updatedFields } });
diff --git a/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-dropdown.jsx b/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-dropdown.jsx
@@ -31,11 +31,6 @@ const DynamicDropdown = ({ dynamicFieldData: { section, field, fieldPosition },
     automationType: 'embedded_automate',
   });
 
-  // Log the field data for debugging
-  console.log('Field data in DynamicDropdown:', field);
-  console.log('Field values from helper:', fieldValues);
-  console.log('Field state initialized as:', fieldState);
-
   const handleFieldUpdate = (event, updatedFields) => {
     setFieldState((prevState) => ({ ...prevState, ...updatedFields }));
     onFieldAction({
diff --git a/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-tag-control.jsx b/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-tag-control.jsx
@@ -34,11 +34,6 @@ const DynamicTagControl = ({ dynamicFieldData: { section, field, fieldPosition }
     sortOrder: fieldValues.sortOrder || 'ascending',
   });
 
-  // Log the field data for debugging
-  console.log('Field data in DynamicTagControl:', field);
-  console.log('Field values from helper:', fieldValues);
-  console.log('Field state initialized as:', fieldState);
-
   useEffect(() => {
     if (fieldState.categories.length === 0) {
       tagControlCategories().then((fetchedCategories) => {
diff --git a/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-text-area.jsx b/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-text-area.jsx
@@ -29,11 +29,6 @@ const DynamicTextArea = ({ dynamicFieldData: { section, field, fieldPosition },
     fieldsToRefresh: refreshEnabledFields,
   });
 
-  // Log the field data for debugging
-  console.log('Field data in DynamicTextArea:', field);
-  console.log('Field values from helper:', fieldValues);
-  console.log('Field state initialized as:', fieldState);
-
   const handleFieldUpdate = (event, updatedFields) => {
     setFieldState((prevState) => ({ ...prevState, ...updatedFields }));
     onFieldAction({ event, type: editActionType, fieldPosition, inputProps: { ...fieldState, ...updatedFields } });
diff --git a/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-text-input.jsx b/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-text-input.jsx
@@ -27,11 +27,6 @@ const DynamicTextInput = ({ dynamicFieldData: { section, field, fieldPosition },
     fieldsToRefresh: refreshEnabledFields,
   });
 
-  // Log the field data for debugging
-  console.log('Field data in DynamicTextInput:', field);
-  console.log('Field values from helper:', fieldValues);
-  console.log('Field state initialized as:', fieldState);
-
   const handleFieldUpdate = (event, updatedFields) => {
     setFieldState((prevState) => ({ ...prevState, ...updatedFields }));
     onFieldAction({ event, type: editActionType, fieldPosition, inputProps: { ...fieldState, ...updatedFields } });
diff --git a/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-time-picker.jsx b/app/javascript/components/service-dialog-form/dynamic-fields/dynamic-time-picker.jsx
@@ -83,13 +83,6 @@ const DynamicTimePicker = ({ dynamicFieldData: { section, field, fieldPosition }
     }
   }, [fieldValues.value]);
 
-  // Log the field data for debugging
-  useEffect(() => {
-    console.log('Field data in DynamicTimePicker:', field);
-    console.log('Field values from helper:', fieldValues);
-    console.log('Field state initialized as:', fieldState);
-  }, [field, fieldValues, fieldState]);
-
   const handleDateChange = (selectedDates) => {
     if (selectedDates.length > 0) {
       const formattedDate = new Intl.DateTimeFormat('en-US', {
@@ -247,5 +240,3 @@ DynamicTimePicker.propTypes = {
 };
 
 export default DynamicTimePicker;
-
-// Made with Bob
diff --git a/app/javascript/components/service-dialog-form/dynamic-section.jsx b/app/javascript/components/service-dialog-form/dynamic-section.jsx
@@ -86,38 +86,30 @@ const DynamicSection = ({ section, sectionAction }) => {
     </div>
   );
 
-  const renderSectionContents = () => {
-    console.log('Rendering section contents for section:', section);
-    console.log('Fields in section:', section.fields);
-    
-    return (
-      <div className="dynamic-section-contents">
-        {
-          section.fields && section.fields.map((field, fieldPosition) => {
-            console.log(`Rendering field ${fieldPosition}:`, field);
-            return (
-              <div
-                className="dynamic-form-field-wrapper"
-                key={`field-${fieldPosition.toString()}`}
-                draggable
-                onDragEnter={(event) => onFieldAction({ event, fieldPosition, type: SD_ACTIONS.onDragEnterField })}
-                onDragStart={(event) => onFieldAction({ event, fieldPosition, type: SD_ACTIONS.onDragStartField })}
-              >
-                <DynamicField
-                  key={fieldPosition.toString()}
-                  fieldData={{ field, fieldPosition, section }}
-                  onFieldAction={(newFieldData) => onFieldAction(newFieldData)}
-                />
-              </div>
-            );
-          })
-        }
-        {
-          (!section.fields || section.fields.length === 0) && renderHelpText()
-        }
-      </div>
-    );
-  };
+  const renderSectionContents = () => (
+    <div className="dynamic-section-contents">
+      {
+        section.fields && section.fields.map((field, fieldPosition) => (
+          <div
+            className="dynamic-form-field-wrapper"
+            key={`field-${fieldPosition.toString()}`}
+            draggable
+            onDragEnter={(event) => onFieldAction({ event, fieldPosition, type: SD_ACTIONS.onDragEnterField })}
+            onDragStart={(event) => onFieldAction({ event, fieldPosition, type: SD_ACTIONS.onDragStartField })}
+          >
+            <DynamicField
+              key={fieldPosition.toString()}
+              fieldData={{ field, fieldPosition, section }}
+              onFieldAction={(newFieldData) => onFieldAction(newFieldData)}
+            />
+          </div>
+        ))
+      }
+      {
+        (!section.fields || section.fields.length === 0) && renderHelpText()
+      }
+    </div>
+  );
 
   return (
     <div
diff --git a/app/javascript/components/service-dialog-form/helper.js b/app/javascript/components/service-dialog-form/helper.js
@@ -207,8 +207,7 @@ export const dropComponent = (section, { componentId }) => {
     default:
       break;
   }
-  
-  console.log('Adding field to section:', fieldData);
+
   section.fields.push(fieldData);
 };
 
@@ -279,8 +278,6 @@ export const SD_PROP_SHAPES = {
  * @returns {Object} - The standardized field values
  */
 export const getFieldValues = (field) => {
-  console.log('Getting field values for:', field);
-  
   // Common properties for all field types
   const commonProps = {
     type: field.type || '',
@@ -467,16 +464,14 @@ const getSectionsInfo = (sections) => sections.map((section) => ({
   dialog_fields: getFieldsInfo(section.fields),
 }));
 
-const getTabsInfo = (tabs) => {
-  console.log("Tabs: ", tabs);
-  return tabs
+const getTabsInfo = (tabs) =>
+  tabs
     .map((tab) => ({
       label: tab.name,
       position: tab.tabId,
       dialog_groups: getSectionsInfo(tab.sections),
     }))
-    .filter((tabInfo) => tabInfo.dialog_groups.length > 0); // Filter out objects with empty dialog_groups
-};
+    .filter((tabInfo) => tabInfo.dialog_groups.length > 0);
 
 const getDialogInfo = (data) => ({
   label: data.label,
diff --git a/app/javascript/components/service-dialog-form/index.jsx b/app/javascript/components/service-dialog-form/index.jsx
@@ -59,41 +59,24 @@ const ServiceDialogForm = ({ dialogData, dialogAction }) => {
   useEffect(() => {
     // If we're editing an existing dialog, fetch its complete structure
     if (dialogData && dialogData.id && dialogAction && dialogAction.action === 'edit') {
-      console.log('Fetching complete dialog structure for editing...');
-      
       // Fetch the complete dialog structure from the API
       API.get(`/api/service_dialogs/${dialogData.id}?expand=resources&attributes=content,dialog_tabs`)
         .then((response) => {
-          console.log('API response:', response);
-
           if (response.content && response.content[0]) {
             const fullDialogData = response.content[0];
-            console.log('Full dialog data:', fullDialogData);
 
             // Extract dialog tabs from the API response
             const dialogTabs = fullDialogData.dialog_tabs || [];
-            console.log('Dialog tabs from API:', dialogTabs);
 
             if (dialogTabs.length > 0) {
               const formattedTabs = dialogTabs.map((tab, index) => {
-                console.log(`Processing tab ${index}:`, tab);
-
                 const formattedTab = {
                   tabId: index,
                   name: tab.label,
                   sections: (tab.dialog_groups || []).map((group, groupIndex) => {
-                    console.log(`Processing group ${groupIndex} in tab ${index}:`, group);
-
                     const fields = (group.dialog_fields || []).map(field => {
-                      console.log(`Processing field in group ${groupIndex}:`, field);
-                      console.log(`Field default_value:`, field.default_value);
-                      console.log(`Field value:`, field.value);
-                      console.log(`Field values:`, field.values);
-                      console.log(`Field options:`, field.options);
-
                       const componentId = getComponentIdFromType(field.type);
-                      console.log(`Mapped field type ${field.type} to componentId ${componentId}`);
-                      
+
                       return {
                         // Ensure each field has componentId based on its type
                         componentId,
@@ -107,9 +90,7 @@ const ServiceDialogForm = ({ dialogData, dialogAction }) => {
                         readOnly: field.read_only || false
                       };
                     });
-                    
-                    console.log(`Processed fields for group ${groupIndex}:`, fields);
-                    
+
                     return {
                       tabId: index,
                       sectionId: groupIndex,
@@ -120,14 +101,13 @@ const ServiceDialogForm = ({ dialogData, dialogAction }) => {
                     };
                   }),
                 };
-                
-                console.log(`Formatted tab ${index}:`, formattedTab);
+
                 return formattedTab;
               });
-              
+
               // Add the "Create new tab" option
               formattedTabs.push(createNewTab());
-              
+
               // Update the state with the fetched data
               setData({
                 list: dynamicComponents,
@@ -139,12 +119,10 @@ const ServiceDialogForm = ({ dialogData, dialogAction }) => {
           }
         })
         .catch((error) => {
-          console.error('Error fetching dialog data:', error);
         });
     } else if (dialogData && dialogData.dialog_tabs && dialogData.dialog_tabs.length > 0) {
       // If dialog_tabs is already available in the passed data, use it
-      console.log('Using dialog tabs from props:', dialogData.dialog_tabs);
-      
+
       const formattedTabs = dialogData.dialog_tabs.map((tab, index) => ({
         tabId: index,
         name: tab.label,
@@ -153,7 +131,7 @@ const ServiceDialogForm = ({ dialogData, dialogAction }) => {
           sectionId: groupIndex,
           title: group.label,
           description: group.description || '',
-          fields: (group.dialog_fields || []).map(field => ({
+          fields: (group.dialog_fields || []).map((field) => ({
             componentId: getComponentIdFromType(field.type),
             ...field,
             name: field.name || `field_${Date.now()}_${Math.random().toString(36).substring(2, 9)}`,
@@ -166,10 +144,10 @@ const ServiceDialogForm = ({ dialogData, dialogAction }) => {
           order: group.position || 0,
         })),
       }));
-      
+
       // Add the "Create new tab" option
       formattedTabs.push(createNewTab());
-      
+
       setData({
         list: dynamicComponents,
         formFields: formattedTabs,
