Merge pull request #1228 from 10up/fix/sync-patterns-error-2

Follow react rules of hooks

assets/js/gutenberg-plugin.js

@@ -153,45 +153,47 @@ const DistributorIcon = () => (
  * Add the Distributor panel to Gutenberg
  */
 const DistributorPlugin = () => {
-	// Ensure the user has proper permissions
-	if (
-		dtGutenberg.noPermissions &&
-		1 === parseInt( dtGutenberg.noPermissions )
-	) {
-		return null;
-	}
-
-	// eslint-disable-next-line no-shadow, react-hooks/rules-of-hooks -- permission checks are needed.
+	// eslint-disable-next-line no-shadow
 	const postType = useSelect( ( select ) =>
 		select( 'core/editor' ).getCurrentPostType()
 	);
 
-	// eslint-disable-next-line no-shadow, react-hooks/rules-of-hooks -- permission checks are needed.
+	// eslint-disable-next-line no-shadow
 	const postStatus = useSelect( ( select ) =>
 		select( 'core/editor' ).getCurrentPostAttribute( 'status' )
 	);
 
-	// Ensure we are on a supported post type
-	if (
-		dtGutenberg.supportedPostTypes &&
-		dtGutenberg.supportedPostTypes[ postType ] === undefined
-	) {
-		return null;
-	}
-
+	// eslint-disable-next-line @wordpress/no-unused-vars-before-return
 	const distributorTopMenu = document.querySelector(
 		'#wp-admin-bar-distributor'
 	);
 
-	// eslint-disable-next-line no-shadow, react-hooks/rules-of-hooks -- permission checks are needed.
+	// eslint-disable-next-line no-shadow
 	const post = useSelect( ( select ) =>
 		select( 'core/editor' ).getCurrentPost()
 	);
+
+	// Ensure the user has proper permissions.
+	if (
+		dtGutenberg.noPermissions &&
+		1 === parseInt( dtGutenberg.noPermissions )
+	) {
+		return null;
+	}
+
+	// Ensure we are on a supported post type.
+	if (
+		dtGutenberg.supportedPostTypes &&
+		dtGutenberg.supportedPostTypes[ postType ] === undefined
+	) {
+		return null;
+	}
+
 	// Make the post title and status available to the top menu.
 	dt.postTitle = post.title;
 	dt.postStatus = post.status;
 
-	// If we are on a non-supported post status, change what we show
+	// If we are on a non-supported post status, change what we show.
 	if (
 		dtGutenberg.supportedPostStati &&
 		! dtGutenberg.supportedPostStati.includes( postStatus )