Refactor controls to actions

packages/editor/src/store/actions.js

@@ -8,6 +8,11 @@ import { castArray } from 'lodash';
  */
 import { getDefaultBlockName, createBlock } from '@wordpress/blocks';
 
+/**
+ * Internal dependencies
+ */
+import { select } from './controls';
+
 /**
  * Returns an action object used in signalling that editor has initialized with
  * the specified post object and editor settings.
@@ -177,29 +182,21 @@ export function selectBlock( clientId, initialPosition = null ) {
  * given clientId should be selected.
  *
  * @param {string} clientId Block client ID.
- *
- * @return {Object} Action object.
  */
-export function selectPreviousBlock( clientId ) {
-	return {
-		type: 'SELECT_PREVIOUS_BLOCK',
-		clientId,
-	};
+export function * selectPreviousBlock( clientId ) {
+	const previousBlockClientId = yield select( 'core/editor', 'getPreviousBlockClientId', clientId );
+	yield selectBlock( previousBlockClientId );
 }
 
 /**
  * Returns an action object used in signalling that the block following the
  * given clientId should be selected.
  *
  * @param {string} clientId Block client ID.
- *
- * @return {Object} Action object.
  */
-export function selectNextBlock( clientId ) {
-	return {
-		type: 'SELECT_NEXT_BLOCK',
-		clientId,
-	};
+export function * selectNextBlock( clientId ) {
+	const nextBlockClientId = yield select( 'core/editor', 'getNextBlockClientId', clientId );
+	yield selectBlock( nextBlockClientId );
 }
 
 export function startMultiSelect() {

packages/editor/src/store/controls.js

@@ -3,18 +3,28 @@
  */
 import { createRegistryControl } from '@wordpress/data';
 
-export const SELECT_PREVIOUS_BLOCK = createRegistryControl( ( registry ) => ( action ) => {
-	const { clientId } = action;
-	const { getPreviousBlockClientId } = registry.select( 'core/editor' );
-	const { selectBlock } = registry.dispatch( 'core/editor' );
-
-	selectBlock( getPreviousBlockClientId( clientId ), -1 );
-} );
+/**
+ * Calls a selector using the current state.
+ *
+ * @param {string} storeKey     Store key.
+ * @param {string} selectorName Selector name.
+ * @param  {Array} args         Selector arguments.
+ *
+ * @return {Object} control descriptor.
+ */
+export function select( storeKey, selectorName, ...args ) {
+	return {
+		type: 'SELECT',
+		storeKey,
+		selectorName,
+		args,
+	};
+}
 
-export const SELECT_NEXT_BLOCK = createRegistryControl( ( registry ) => ( action ) => {
-	const { clientId } = action;
-	const { getNextBlockClientId } = registry.select( 'core/editor' );
-	const { selectBlock } = registry.dispatch( 'core/editor' );
+const controls = {
+	SELECT: createRegistryControl( ( registry ) => ( { storeKey, selectorName, args } ) => {
+		return registry.select( storeKey )[ selectorName ]( ...args );
+	} ),
+};
 
-	selectBlock( getNextBlockClientId( clientId ) );
-} );
+export default controls;

packages/editor/src/store/index.js

@@ -10,7 +10,7 @@ import reducer from './reducer';
 import applyMiddlewares from './middlewares';
 import * as selectors from './selectors';
 import * as actions from './actions';
-import * as controls from './controls';
+import controls from './controls';
 
 /**
  * Module Constants