false value checks

WordPress · May 14, 2024 · a4afd42 · a4afd42
1 parent 64d782a
commit a4afd42
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/packages/editor/src/components/post-actions/actions.js b/packages/editor/src/components/post-actions/actions.js
@@ -687,7 +687,10 @@ const duplicatePostAction = {
 };
 
 const isTemplatePartRevertable = ( item ) => {
-	const hasThemeFile = item.templatePart.has_theme_file;
+	if ( ! item ) {
+		return false;
+	}
+	const hasThemeFile = item.templatePart?.has_theme_file;
 	return canDeleteOrReset( item ) && hasThemeFile;
 };
 
@@ -1020,8 +1023,12 @@ export const deletePatternAction = {
 	id: 'delete-pattern',
 	label: __( 'Delete' ),
 	isEligible: ( item ) => {
+		if ( ! item ) {
+			return false;
+		}
 		const isTemplatePart = item.type === TEMPLATE_PART_POST_TYPE;
-		const hasThemeFile = isTemplatePart && item.templatePart.has_theme_file;
+		const hasThemeFile =
+			isTemplatePart && item.templatePart?.has_theme_file;
 		return canDeleteOrReset( item ) && ! hasThemeFile;
 	},
 	hideModalHeader: true,

diff --git a/packages/editor/src/components/post-actions/export-pattern-action.js b/packages/editor/src/components/post-actions/export-pattern-action.js
@@ -36,7 +36,12 @@ export const exportPatternAsJSONAction = {
 	id: 'export-pattern',
 	label: __( 'Export as JSON' ),
 	supportsBulk: true,
-	isEligible: ( item ) => item.type === PATTERN_TYPES.user,
+	isEligible: ( item ) => {
+		if ( ! item.type ) {
+			return false;
+		}
+		return item.type === PATTERN_TYPES.user
+	},
 	callback: async ( items ) => {
 		if ( items.length === 1 ) {
 			return downloadBlob(