Skip to content

Commit ce7dc68

Browse files
aman19Kaman19K
committed
fix: Cannot convert undefined or null to object
1 parent 37fa5d6 commit ce7dc68

File tree

16 files changed

+40
-40
lines changed

16 files changed

+40
-40
lines changed

‎packages/contentstack-export/src/export/modules/custom-roles.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ export default class ExportCustomRoles extends BaseClass {
104104
progress.completeProcess(PROCESS_NAMES.PROCESS_MAPPINGS, true);
105105

106106
log.debug(
107-
`Custom roles export completed. Total custom roles: ${Object.keys(this.customRoles).length}`,
107+
`Custom roles export completed. Total custom roles: ${Object.keys(this.customRoles || {}).length}`,
108108
this.exportConfig.context,
109109
);
110110
this.completeProgress(true);
@@ -177,7 +177,7 @@ export default class ExportCustomRoles extends BaseClass {
177177
this.progressManager?.tick(true, `locale: ${locale.name}`, null, PROCESS_NAMES.FETCH_LOCALES);
178178
}
179179

180-
log.debug(`Mapped ${Object.keys(this.sourceLocalesMap).length} locales`, this.exportConfig.context);
180+
log.debug(`Mapped ${Object.keys(this.sourceLocalesMap || {}).length} locales`, this.exportConfig.context);
181181
}
182182

183183
async getCustomRolesLocales() {

‎packages/contentstack-export/src/export/modules/environments.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ export default class ExportEnvironments extends BaseClass {
5757

5858
progress.updateStatus('Fetching environments...');
5959
await this.getEnvironments();
60-
log.debug(`Retrieved ${Object.keys(this.environments).length} environments`, this.exportConfig.context);
60+
log.debug(`Retrieved ${Object.keys(this.environments || {}).length} environments`, this.exportConfig.context);
6161

6262
if (this.environments === undefined || isEmpty(this.environments)) {
6363
log.info(messageHandler.parse('ENVIRONMENT_NOT_FOUND'), this.exportConfig.context);
@@ -66,7 +66,7 @@ export default class ExportEnvironments extends BaseClass {
6666
log.debug(`Writing environments to: ${environmentsFilePath}`, this.exportConfig.context);
6767
fsUtil.writeFile(environmentsFilePath, this.environments);
6868
log.success(
69-
messageHandler.parse('ENVIRONMENT_EXPORT_COMPLETE', Object.keys(this.environments).length),
69+
messageHandler.parse('ENVIRONMENT_EXPORT_COMPLETE', Object.keys(this.environments || {}).length),
7070
this.exportConfig.context,
7171
);
7272
}
@@ -130,7 +130,7 @@ export default class ExportEnvironments extends BaseClass {
130130
}
131131

132132
log.debug(
133-
`Sanitization complete. Total environments processed: ${Object.keys(this.environments).length}`,
133+
`Sanitization complete. Total environments processed: ${Object.keys(this.environments || {}).length}`,
134134
this.exportConfig.context,
135135
);
136136
}

‎packages/contentstack-export/src/export/modules/extensions.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ export default class ExportExtensions extends BaseClass {
5858

5959
progress.updateStatus('Fetching extensions...');
6060
await this.getExtensions();
61-
log.debug(`Retrieved ${Object.keys(this.extensions).length} extensions`, this.exportConfig.context);
61+
log.debug(`Retrieved ${Object.keys(this.extensions || {}).length} extensions`, this.exportConfig.context);
6262

6363
if (this.extensions === undefined || isEmpty(this.extensions)) {
6464
log.info(messageHandler.parse('EXTENSION_NOT_FOUND'), this.exportConfig.context);
@@ -67,7 +67,7 @@ export default class ExportExtensions extends BaseClass {
6767
log.debug(`Writing extensions to: ${extensionsFilePath}`, this.exportConfig.context);
6868
fsUtil.writeFile(extensionsFilePath, this.extensions);
6969
log.success(
70-
messageHandler.parse('EXTENSION_EXPORT_COMPLETE', Object.keys(this.extensions).length),
70+
messageHandler.parse('EXTENSION_EXPORT_COMPLETE', Object.keys(this.extensions || {}).length),
7171
this.exportConfig.context,
7272
);
7373
}
@@ -131,7 +131,7 @@ export default class ExportExtensions extends BaseClass {
131131
}
132132

133133
log.debug(
134-
`Sanitization complete. Total extensions processed: ${Object.keys(this.extensions).length}`,
134+
`Sanitization complete. Total extensions processed: ${Object.keys(this.extensions || {}).length}`,
135135
this.exportConfig.context,
136136
);
137137
}

‎packages/contentstack-export/src/export/modules/labels.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ export default class ExportLabels extends BaseClass {
5858

5959
progress.updateStatus('Fetching labels...');
6060
await this.getLabels();
61-
log.debug(`Retrieved ${Object.keys(this.labels).length} labels`, this.exportConfig.context);
61+
log.debug(`Retrieved ${Object.keys(this.labels || {}).length} labels`, this.exportConfig.context);
6262

6363
if (this.labels === undefined || isEmpty(this.labels)) {
6464
log.info(messageHandler.parse('LABELS_NOT_FOUND'), this.exportConfig.context);
@@ -67,7 +67,7 @@ export default class ExportLabels extends BaseClass {
6767
log.debug(`Writing labels to: ${labelsFilePath}`, this.exportConfig.context);
6868
fsUtil.writeFile(labelsFilePath, this.labels);
6969
log.success(
70-
messageHandler.parse('LABELS_EXPORT_COMPLETE', Object.keys(this.labels).length),
70+
messageHandler.parse('LABELS_EXPORT_COMPLETE', Object.keys(this.labels || {}).length),
7171
this.exportConfig.context,
7272
);
7373
}

‎packages/contentstack-export/src/export/modules/locales.ts‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -74,8 +74,8 @@ export default class LocaleExport extends BaseClass {
7474
progress.updateStatus('Fetching locale definitions...');
7575
await this.getLocales();
7676
log.debug(
77-
`Retrieved ${Object.keys(this.locales).length} locales and ${
78-
Object.keys(this.masterLocale).length
77+
`Retrieved ${Object.keys(this.locales || {}).length} locales and ${
78+
Object.keys(this.masterLocale || {}).length
7979
} master locales`,
8080
this.exportConfig.context,
8181
);
@@ -89,8 +89,8 @@ export default class LocaleExport extends BaseClass {
8989
log.success(
9090
messageHandler.parse(
9191
'LOCALES_EXPORT_COMPLETE',
92-
Object.keys(this.locales).length,
93-
Object.keys(this.masterLocale).length,
92+
Object.keys(this.locales || {}).length,
93+
Object.keys(this.masterLocale || {}).length,
9494
),
9595
this.exportConfig.context,
9696
);
@@ -156,8 +156,8 @@ export default class LocaleExport extends BaseClass {
156156
});
157157

158158
log.debug(
159-
`Sanitization complete. Master locales: ${Object.keys(this.masterLocale).length}, Regular locales: ${
160-
Object.keys(this.locales).length
159+
`Sanitization complete. Master locales: ${Object.keys(this.masterLocale || {}).length}, Regular locales: ${
160+
Object.keys(this.locales || {}).length
161161
}`,
162162
this.exportConfig.context,
163163
);

‎packages/contentstack-export/src/export/modules/marketplace-apps.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ export default class ExportMarketplaceApps extends BaseClass {
232232
fsUtil.writeFile(marketplaceAppsFilePath, this.installedApps);
233233

234234
log.success(
235-
messageHandler.parse('MARKETPLACE_APPS_EXPORT_COMPLETE', Object.keys(this.installedApps).length),
235+
messageHandler.parse('MARKETPLACE_APPS_EXPORT_COMPLETE', Object.keys(this.installedApps || {}).length),
236236
this.exportConfig.context,
237237
);
238238
}

‎packages/contentstack-export/src/export/modules/taxonomies.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ export default class ExportTaxonomies extends BaseClass {
7979
await this.getAllTaxonomies();
8080
progress.completeProcess(PROCESS_NAMES.FETCH_TAXONOMIES, true);
8181

82-
const actualTaxonomyCount = Object.keys(this.taxonomies)?.length;
82+
const actualTaxonomyCount = Object.keys(this.taxonomies || {})?.length;
8383
log.debug(
8484
`Found ${actualTaxonomyCount} taxonomies to export (API reported ${totalCount})`,
8585
this.exportConfig.context,
@@ -105,7 +105,7 @@ export default class ExportTaxonomies extends BaseClass {
105105
log.info('No taxonomies found to export detailed information', this.exportConfig.context);
106106
}
107107

108-
const taxonomyCount = Object.keys(this.taxonomies).length;
108+
const taxonomyCount = Object.keys(this.taxonomies || {}).length;
109109
log.success(messageHandler.parse('TAXONOMY_EXPORT_COMPLETE', taxonomyCount), this.exportConfig.context);
110110
this.completeProgress(true);
111111
} catch (error) {
@@ -178,7 +178,7 @@ export default class ExportTaxonomies extends BaseClass {
178178
}
179179

180180
log.debug(
181-
`Sanitization complete. Total taxonomies processed: ${Object.keys(this.taxonomies).length}`,
181+
`Sanitization complete. Total taxonomies processed: ${Object.keys(this.taxonomies || {}).length}`,
182182
this.exportConfig.context,
183183
);
184184
}
@@ -189,7 +189,7 @@ export default class ExportTaxonomies extends BaseClass {
189189
*/
190190
async exportTaxonomies(): Promise<any> {
191191
log.debug(
192-
`Exporting ${Object.keys(this.taxonomies)?.length} taxonomies with detailed information`,
192+
`Exporting ${Object.keys(this.taxonomies || {})?.length} taxonomies with detailed information`,
193193
this.exportConfig.context,
194194
);
195195

‎packages/contentstack-export/src/export/modules/webhooks.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ export default class ExportWebhooks extends BaseClass {
5555

5656
progress.updateStatus('Fetching webhooks...');
5757
await this.getWebhooks();
58-
log.debug(`Retrieved ${Object.keys(this.webhooks).length} webhooks`, this.exportConfig.context);
58+
log.debug(`Retrieved ${Object.keys(this.webhooks || {}).length} webhooks`, this.exportConfig.context);
5959

6060
if (this.webhooks === undefined || isEmpty(this.webhooks)) {
6161
log.info(messageHandler.parse('WEBHOOK_NOT_FOUND'), this.exportConfig.context);
@@ -64,7 +64,7 @@ export default class ExportWebhooks extends BaseClass {
6464
log.debug(`Writing webhooks to: ${webhooksFilePath}`, this.exportConfig.context);
6565
fsUtil.writeFile(webhooksFilePath, this.webhooks);
6666
log.success(
67-
messageHandler.parse('WEBHOOK_EXPORT_COMPLETE', Object.keys(this.webhooks).length),
67+
messageHandler.parse('WEBHOOK_EXPORT_COMPLETE', Object.keys(this.webhooks || {}).length),
6868
this.exportConfig.context,
6969
);
7070
}
@@ -130,7 +130,7 @@ export default class ExportWebhooks extends BaseClass {
130130
}
131131

132132
log.debug(
133-
`Sanitization complete. Total webhooks processed: ${Object.keys(this.webhooks).length}`,
133+
`Sanitization complete. Total webhooks processed: ${Object.keys(this.webhooks || {}).length}`,
134134
this.exportConfig.context,
135135
);
136136
}

‎packages/contentstack-export/src/export/modules/workflows.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ export default class ExportWorkFlows extends BaseClass {
5858
progress.updateStatus('Fetching workflow definitions...');
5959
await this.getWorkflows();
6060

61-
log.debug(`Retrieved ${Object.keys(this.workflows).length} workflows`, this.exportConfig.context);
61+
log.debug(`Retrieved ${Object.keys(this.workflows || {}).length} workflows`, this.exportConfig.context);
6262

6363
if (this.workflows === undefined || isEmpty(this.workflows)) {
6464
log.info(messageHandler.parse('WORKFLOW_NOT_FOUND'), this.exportConfig.context);
@@ -67,7 +67,7 @@ export default class ExportWorkFlows extends BaseClass {
6767
log.debug(`Writing workflows to: ${workflowsFilePath}`, this.exportConfig.context);
6868
fsUtil.writeFile(workflowsFilePath, this.workflows);
6969
log.success(
70-
messageHandler.parse('WORKFLOW_EXPORT_COMPLETE', Object.keys(this.workflows).length),
70+
messageHandler.parse('WORKFLOW_EXPORT_COMPLETE', Object.keys(this.workflows || {}).length),
7171
this.exportConfig.context,
7272
);
7373
}

‎packages/contentstack-import/src/import/modules/assets.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -297,12 +297,12 @@ export default class ImportAssets extends BaseClass {
297297

298298
if (!isVersion) {
299299
if (!isEmpty(this.assetsUidMap)) {
300-
const uidMappingCount = Object.keys(this.assetsUidMap).length;
300+
const uidMappingCount = Object.keys(this.assetsUidMap || {}).length;
301301
log.debug(`Writing ${uidMappingCount} UID mappings`, this.importConfig.context);
302302
this.fs.writeFile(this.assetUidMapperPath, this.assetsUidMap);
303303
}
304304
if (!isEmpty(this.assetsUrlMap)) {
305-
const urlMappingCount = Object.keys(this.assetsUrlMap).length;
305+
const urlMappingCount = Object.keys(this.assetsUrlMap || {}).length;
306306
log.debug(`Writing ${urlMappingCount} URL mappings`, this.importConfig.context);
307307
this.fs.writeFile(this.assetUrlMapperPath, this.assetsUrlMap);
308308
}

0 commit comments

Comments
 (0)