Code Quality: Update: Reuse view revisions action on templates and pa…

…ges. (#57208)
WordPress · Jan 4, 2024 · 2266df4 · 2266df4
1 parent 1cce8f7
commit 2266df4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 23 deletions.
diff --git a/packages/edit-site/src/components/page-templates/index.js b/packages/edit-site/src/components/page-templates/index.js
@@ -45,8 +45,8 @@ import {
 	useResetTemplateAction,
 	deleteTemplateAction,
 	renameTemplateAction,
-	seeRevisionsAction,
 } from './template-actions';
+import { postRevisionsAction } from '../actions';
 import usePatternSettings from '../page-patterns/use-pattern-settings';
 import { unlock } from '../../lock-unlock';
 import PostPreview from '../post-preview';
@@ -333,7 +333,7 @@ export default function DataviewsTemplates() {
 			resetTemplateAction,
 			deleteTemplateAction,
 			renameTemplateAction,
-			seeRevisionsAction,
+			postRevisionsAction,
 		],
 		[ resetTemplateAction ]
 	);

diff --git a/packages/edit-site/src/components/page-templates/template-actions.js b/packages/edit-site/src/components/page-templates/template-actions.js
@@ -15,7 +15,6 @@ import {
 	__experimentalHStack as HStack,
 	__experimentalVStack as VStack,
 } from '@wordpress/components';
-import { addQueryArgs } from '@wordpress/url';
 
 /**
  * Internal dependencies
@@ -208,23 +207,3 @@ export const renameTemplateAction = {
 		);
 	},
 };
-
-export const seeRevisionsAction = {
-	id: 'see-revisions',
-	label: __( 'See revisions' ),
-	isEligible: ( template ) => {
-		if ( template?._links && template?._links[ 'predecessor-version' ] ) {
-			const predecessorVersions =
-				template._links[ 'predecessor-version' ];
-			return predecessorVersions.length > 0;
-		}
-		return false;
-	},
-	callback( template ) {
-		const lastRevisionId =
-			template?._links[ 'predecessor-version' ][ 0 ].id;
-		document.location.href = addQueryArgs( 'revision.php', {
-			revision: lastRevisionId,
-		} );
-	},
-};