This sounds like a fix that needs unit tests.

@atimmer It definitely needs unit tests, but It's not ready yet :)

--

I just discovered that innerBlocks are not supposed to be undefined from the parser's code. But it happens that we drop this property when setting blockByUid in the state when calling getFlattenedBlocks which makes sense since the order and inner blocks are kept separate in blockOrder reducer.

But the issue is that calling the getBlock() selector is returning a block without the inner blocks property. So maybe the right fix here is to enhance getBlock to include the innerBlocks. Thoughts @aduth

But the issue is that calling the getBlock() selector is returning a block without the inner blocks property. So maybe the right fix here is to enhance getBlock to include the innerBlocks. Thoughts @aduth

Heh, I had this on my mind in a totally unrelated circumstance, and agree it'd make for a wise change. Currently we only assign this in getBlocks, but it seems like getBlocks should just map to getBlock where the innerBlocks assignment is to occur.

Updated getBlock to assign innerBlocks in 046d694.

These changes look good to me and it's working good right now.