refactor: Improve array handling and code consistency in WordPress service and migration libraries found by copilot

Author: AishDani

api/src/services/wordpress.service.ts

@@ -129,7 +129,7 @@ const RteJsonConverter = (html: string) => {
 }
 
 const getLocale = (master_locale: string, project: any) => {
-  for (const [key, value] of Object.entries(project?.master_locale || {})) {
+  for (const key of Object.keys(project?.master_locale || {})) {
     if (key === master_locale) {
       return key;
     }

upload-api/migration-wordpress/libs/extractAuthor.ts

@@ -54,15 +54,16 @@ const extractAuthor = async(item:any, type: string) => {
       }
     }]
   };
-  if(typeof item === 'object'){
-    const fields = await handleAuthorSchema(item);
+  if(Array.isArray(item)){
+    const fields = await handleAuthorSchema(item?.[0]);
     author?.fieldMapping?.push(...fields.map(field => ({ 
       ...field, 
       isDeleted: false, 
       advanced: { ...field?.advanced, mandatory: field?.advanced?.mandatory ?? false } 
     })));
-  }else if(Array.isArray(item)){
-    const fields = await handleAuthorSchema(item?.[0]);
+  }
+  else{
+    const fields = await handleAuthorSchema(item);
     author?.fieldMapping?.push(...fields.map(field => ({ 
       ...field, 
       isDeleted: false, 

upload-api/migration-wordpress/libs/extractItems.ts

@@ -334,7 +334,7 @@ const extractItems = async (item: any, config: DataConfig, type: string, affix:
             // If this block has similar structures
             if (similarBlocks?.length > 0) {
               // Create a unique key based on the structure/name to track processed groups
-              const groupKey = field?.attributes?.metadata?.name ?? (field?.name === 'core/missing' ? 'body' : field?.name)
+              const groupKey = field?.attributes?.metadata?.name ?? (field?.name === 'core/missing' ? 'body' : field?.name);
               // Skip if we've already processed this group of similar blocks
               if (processedSimilarBlocks?.has?.(groupKey) || existingBlock) {
                   continue;
@@ -539,7 +539,7 @@ const extractItems = async (item: any, config: DataConfig, type: string, affix:
             "contentstackUid": type?.toLowerCase(),
             "type": "content_type",
             "fieldMapping": CT
-        }
+        };
 
     try {
         await helper.writeFileAsync(

upload-api/migration-wordpress/libs/extractTerms.ts

@@ -56,15 +56,16 @@ const extractTerms = async(allTerms: any, type: string) => {
             }
           }]
         };
-        if(typeof allTerms === 'object'){
-          const fields = await handleAuthorSchema(allTerms);
+        if(Array.isArray(allTerms)){
+          const fields = await handleAuthorSchema(allTerms?.[0]);
           terms?.fieldMapping?.push(...fields.map(field => ({ 
             ...field, 
             isDeleted: false, 
             advanced: { ...field?.advanced, mandatory: field?.advanced?.mandatory ?? false } 
           })));
-        }else if(Array.isArray(allTerms)){
-          const fields = await handleAuthorSchema(allTerms?.[0]);
+        }
+        else{
+          const fields = await handleAuthorSchema(allTerms);
           terms?.fieldMapping?.push(...fields.map(field => ({ 
             ...field, 
             isDeleted: false, 

upload-api/migration-wordpress/libs/schemaMapper.ts

@@ -198,7 +198,7 @@ async function schemaMapper (key: WordPressBlock | WordPressBlock[], parentUid:
                 backupFieldType: 'json',
                 backupFieldUid: rteUid,
                 advanced: {}
-            }
+            };
         }
         break;
         case 'core/image':
@@ -217,7 +217,7 @@ async function schemaMapper (key: WordPressBlock | WordPressBlock[], parentUid:
                 backupFieldType: 'file',
                 backupFieldUid: fileUid,
                 advanced: {}
-            }
+            };
         }
         break;
 
@@ -234,7 +234,7 @@ async function schemaMapper (key: WordPressBlock | WordPressBlock[], parentUid:
                 backupFieldType: 'single_line_text',
                 backupFieldUid: textUid,
                 advanced: {}
-            }
+            };
         }
         break;
         case 'core/social-link':
@@ -251,7 +251,7 @@ async function schemaMapper (key: WordPressBlock | WordPressBlock[], parentUid:
                 backupFieldType: 'link',
                 backupFieldUid: LinkUid,
                 advanced: {}
-            }
+            };
         }
         break;
         case 'core/list':