Social: Default to the current post ID for share status (#39112)

Automattic · Aug 28, 2024 · e097da3 · e097da3
1 parent 2b38242
commit e097da3
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 10 deletions.
diff --git a/...licize-components/changelog/update-social-default-to-the-current-post-id-for-share-status b/...licize-components/changelog/update-social-default-to-the-current-post-id-for-share-status
@@ -0,0 +1,4 @@
+Significance: patch
+Type: changed
+
+Social: Default to the current post ID for share status selector
diff --git a/projects/js-packages/publicize-components/src/social-store/resolvers.js b/projects/js-packages/publicize-components/src/social-store/resolvers.js
@@ -63,15 +63,14 @@ export function getConnections() {
 /**
  * Resolves the post share status.
  *
- * @param {number} postId - The post ID.
+ * @param {number} _postId - The post ID.
  *
  * @return {Function} Resolver
  */
-export function getPostShareStatus( postId ) {
-	return async ( { dispatch } ) => {
-		if ( ! postId ) {
-			return;
-		}
+export function getPostShareStatus( _postId ) {
+	return async ( { dispatch, registry } ) => {
+		// Default to the current post ID if none is provided.
+		const postId = _postId || registry.select( editorStore ).getCurrentPostId();
 
 		try {
 			dispatch( fetchPostShareStatus( postId ) );

diff --git a/projects/js-packages/publicize-components/src/social-store/selectors/share-status.ts b/projects/js-packages/publicize-components/src/social-store/selectors/share-status.ts
@@ -1,3 +1,5 @@
+import { createRegistrySelector } from '@wordpress/data';
+import { store as editorStore } from '@wordpress/editor';
 import { PostShareStatus, SocialStoreState } from '../types';
 
 /**
@@ -6,11 +8,17 @@ import { PostShareStatus, SocialStoreState } from '../types';
  * @param {SocialStoreState} state  - State object.
  * @param {number}           postId - The post ID.
  *
- * @return {SocialStoreState[number]} - The post share status.
+ * @return {PostShareStatus} - The post share status.
  */
-export function getPostShareStatus( state: SocialStoreState, postId: number ): PostShareStatus {
-	return state.shareStatus?.[ postId ] ?? { shares: [] };
-}
+export const getPostShareStatus = createRegistrySelector(
+	select =>
+		( state: SocialStoreState, postId?: number ): PostShareStatus => {
+			// Default to the current post ID if none is provided.
+			const id = postId || select( editorStore ).getCurrentPostId();
+
+			return state.shareStatus?.[ id ] ?? { shares: [] };
+		}
+);
 
 /**
  * Whether the share status modal is open.