@wordpress/block-editor 14.8.1-next.cd6172eb0.0 → 14.10.0

package/build-module/store/selectors.js

@@ -1,3 +1,4 @@
+/* wp:polyfill */
 /**
  * WordPress dependencies
  */
@@ -102,7 +103,7 @@ export function isBlockValid(state, clientId) {
  * @param {Object} state    Editor state.
  * @param {string} clientId Block client ID.
  *
- * @return {Object?} Block attributes.
+ * @return {?Object} Block attributes.
  */
 export function getBlockAttributes(state, clientId) {
   const block = state.blocks.byClientId.get(clientId);
@@ -1384,11 +1385,11 @@ export function getTemplateLock(state, rootClientId) {
  * @param {string|Object} blockNameOrType The block type object, e.g., the response
  *                                        from the block directory; or a string name of
  *                                        an installed block type, e.g.' core/paragraph'.
- * @param {Set}           checkedBlocks   Set of block names that have already been checked.
+ * @param {?string}       rootClientId    Optional root client ID of block list.
  *
  * @return {boolean} Whether the given block type is allowed to be inserted.
  */
-const isBlockVisibleInTheInserter = (state, blockNameOrType, checkedBlocks = new Set()) => {
+const isBlockVisibleInTheInserter = (state, blockNameOrType, rootClientId = null) => {
   let blockType;
   let blockName;
   if (blockNameOrType && 'object' === typeof blockNameOrType) {
@@ -1408,18 +1409,14 @@ const isBlockVisibleInTheInserter = (state, blockNameOrType, checkedBlocks = new
   if (!isBlockAllowedInEditor) {
     return false;
   }
-  if (checkedBlocks.has(blockName)) {
-    return false;
-  }
-  checkedBlocks.add(blockName);
 
   // If parent blocks are not visible, child blocks should be hidden too.
-  if (Array.isArray(blockType.parent)) {
-    return blockType.parent.some(name => blockName !== name && isBlockVisibleInTheInserter(state, name, checkedBlocks) ||
-    // Exception for blocks with post-content parent,
-    // the root level is often consider as "core/post-content".
-    // This exception should only apply to the post editor ideally though.
-    name === 'core/post-content');
+  const parents = (Array.isArray(blockType.parent) ? blockType.parent : []).concat(Array.isArray(blockType.ancestor) ? blockType.ancestor : []);
+  if (parents.length > 0) {
+    const rootBlockName = getBlockName(state, rootClientId);
+    // This is an exception to the rule that says that all blocks are visible in the inserter.
+    // Blocks that require a given parent or ancestor are only visible if we're within that parent.
+    return parents.includes('core/post-content') || parents.includes(rootBlockName) || getBlockParentsByBlockName(state, rootClientId, parents).length > 0;
   }
   return true;
 };
@@ -1438,7 +1435,7 @@ const isBlockVisibleInTheInserter = (state, blockNameOrType, checkedBlocks = new
  * @return {boolean} Whether the given block type is allowed to be inserted.
  */
 const canInsertBlockTypeUnmemoized = (state, blockName, rootClientId = null) => {
-  if (!isBlockVisibleInTheInserter(state, blockName)) {
+  if (!isBlockVisibleInTheInserter(state, blockName, rootClientId)) {
     return false;
   }
   let blockType;
@@ -1720,7 +1717,7 @@ const getItemFromVariation = (state, item) => variation => {
  * Returns the calculated frecency.
  *
  * 'frecency' is a heuristic (https://en.wikipedia.org/wiki/Frecency)
- * that combines block usage frequenty and recency.
+ * that combines block usage frequency and recency.
  *
  * @param {number} time  When the last insert occurred as a UNIX epoch
  * @param {number} count The number of inserts that have occurred.
@@ -1789,6 +1786,7 @@ const buildBlockTypeItem = (state, {
     category: blockType.category,
     keywords: blockType.keywords,
     parent: blockType.parent,
+    ancestor: blockType.ancestor,
     variations: inserterVariations,
     example: blockType.example,
     utility: 1 // Deprecated.
@@ -1803,7 +1801,7 @@ const buildBlockTypeItem = (state, {
  * inserter and handle its selection.
  *
  * The 'frecency' property is a heuristic (https://en.wikipedia.org/wiki/Frecency)
- * that combines block usage frequenty and recency.
+ * that combines block usage frequency and recency.
  *
  * Items are returned ordered descendingly by their 'utility' and 'frecency'.
  *
@@ -1862,7 +1860,7 @@ export const getInserterItems = createRegistrySelector(select => createSelector(
   if (options[isFiltered] !== false) {
     blockTypeInserterItems = blockTypeInserterItems.filter(blockType => canIncludeBlockTypeInInserter(state, blockType, rootClientId));
   } else {
-    blockTypeInserterItems = blockTypeInserterItems.filter(blockType => isBlockVisibleInTheInserter(state, blockType)).map(blockType => ({
+    blockTypeInserterItems = blockTypeInserterItems.filter(blockType => isBlockVisibleInTheInserter(state, blockType, rootClientId)).map(blockType => ({
       ...blockType,
       isAllowedInCurrentRoot: canIncludeBlockTypeInInserter(state, blockType, rootClientId)
     }));
@@ -1916,7 +1914,7 @@ export const getInserterItems = createRegistrySelector(select => createSelector(
  * transform list and handle its selection.
  *
  * The 'frecency' property is a heuristic (https://en.wikipedia.org/wiki/Frecency)
- * that combines block usage frequenty and recency.
+ * that combines block usage frequency and recency.
  *
  * Items are returned ordered descendingly by their 'frecency'.
  *
@@ -2007,7 +2005,7 @@ export const __experimentalGetAllowedBlocks = createSelector((state, rootClientI
  * @typedef {Object} WPDirectInsertBlock
  * @property {string}         name             The type of block.
  * @property {?Object}        attributes       Attributes to pass to the newly created block.
- * @property {?Array<string>} attributesToCopy Attributes to be copied from adjecent blocks when inserted.
+ * @property {?Array<string>} attributesToCopy Attributes to be copied from adjacent blocks when inserted.
  */
 export function getDirectInsertBlock(state, rootClientId = null) {
   var _state$blockListSetti;
@@ -2074,7 +2072,7 @@ export const __experimentalGetAllowedPatterns = createRegistrySelector(select =>
     const availableParsedPatterns = parsedPatterns.filter(pattern => checkAllowListRecursive(getGrammar(pattern), allowedBlockTypes));
     const patternsAllowed = availableParsedPatterns.filter(pattern => getGrammar(pattern).every(({
       blockName: name
-    }) => options[isFiltered] !== false ? canInsertBlockType(state, name, rootClientId) : isBlockVisibleInTheInserter(state, name)));
+    }) => options[isFiltered] !== false ? canInsertBlockType(state, name, rootClientId) : isBlockVisibleInTheInserter(state, name, rootClientId)));
     return patternsAllowed;
   }, getAllowedPatternsDependants(select));
 });
@@ -2544,9 +2542,7 @@ export const getBlockEditingMode = createRegistrySelector(select => (state, clie
     const isContent = hasContentRoleAttribute(name);
     return isContent ? 'contentOnly' : 'disabled';
   }
-  // Otherwise, check if there's an ancestor that is contentOnly
-  const parentMode = getBlockEditingMode(state, rootClientId);
-  return parentMode === 'contentOnly' ? 'default' : parentMode;
+  return 'default';
 });
 
 /**