Select all + paste results in an error when there is a trailing empty block

Do you want to request a feature or report a bug?

Bug

What’s the current behavior?

If there is a trailing empty block, and we select all with cmd+a and then paste anything, an error occurs:

Uncaught TypeError: Cannot read property 'key' of undefined
    at Changes.insertFragmentAtRange (https://npmcdn.com/slate@0.31.5/dist/slate.js:7158:55)
    at Change.call (https://npmcdn.com/slate@0.31.5/dist/slate.js:10521:10)
    at Change.(anonymous function) [as insertFragmentAtRange] (https://npmcdn.com/slate@0.31.5/dist/slate.js:10580:15)
    at Changes.insertFragment (https://npmcdn.com/slate@0.31.5/dist/slate.js:6193:10)
    at Change.call (https://npmcdn.com/slate@0.31.5/dist/slate.js:10521:10)
    at Change.(anonymous function) [as insertFragment] (https://npmcdn.com/slate@0.31.5/dist/slate.js:10580:15)
    at Object.onPaste (https://npmcdn.com/slate-react@0.10.20/dist/slate-react.js:11554:14)
    at Stack.run (https://npmcdn.com/slate@0.31.5/dist/slate.js:16879:38)
    at https://npmcdn.com/slate-react@0.10.20/dist/slate-react.js:9290:20
    at Change.call (https://npmcdn.com/slate@0.31.5/dist/slate.js:10521:10)

Issue can be reproduced on the default fiddle: https://jsfiddle.net/fj9dvhom/1/ Here is a gif: https://imgur.com/a/1KENt

Chrome version 62, Ubuntu 17, Slate version 0.31.5

What’s the expected behavior?

After selecting all with cmd+a, paste should work correctly no matter if there is a trailing empty block or not.

[tobiasandersen]

No, sorry if I confused things here. This was recently fixed for when inserting text (i.e. for insertTextAtRange), but it’s still an issue for insertFragmentAtRange.

[zGrav]

I did found out another one though, PR’ing 😃