From 0ac2ed93226b2fdd43d23461486c1bb0873c28aa Mon Sep 17 00:00:00 2001
From: Tomi Blinnikka <tomi@bigpiventures.com>
Date: Thu, 27 Aug 2020 13:56:55 -0500
Subject: [PATCH] Fix #7958 by checking for valid `firstChild`.

---
 packages/ckeditor5-paste-from-office/src/filters/list.js | 2 +-
 .../lists/empty-list-item/input.html                     | 1 +
 .../lists/empty-list-item/model.html                     | 1 +
 .../tests/_data/paste-from-google-docs/lists/index.js    | 9 +++++++--
 4 files changed, 10 insertions(+), 3 deletions(-)
 create mode 100644 packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/empty-list-item/input.html
 create mode 100644 packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/empty-list-item/model.html

diff --git a/packages/ckeditor5-paste-from-office/src/filters/list.js b/packages/ckeditor5-paste-from-office/src/filters/list.js
index b17575c697f..e282b8bf469 100644
--- a/packages/ckeditor5-paste-from-office/src/filters/list.js
+++ b/packages/ckeditor5-paste-from-office/src/filters/list.js
@@ -93,7 +93,7 @@ export function unwrapParagraphInListItem( documentFragment, writer ) {
 			// Google Docs allows on single paragraph inside LI.
 			const firstChild = element.getChild( 0 );
 
-			if ( firstChild.is( 'element', 'p' ) ) {
+			if ( firstChild && firstChild.is( 'element', 'p' ) ) {
 				writer.unwrapElement( firstChild );
 			}
 		}
diff --git a/packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/empty-list-item/input.html b/packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/empty-list-item/input.html
new file mode 100644
index 00000000000..9bcac36d077
--- /dev/null
+++ b/packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/empty-list-item/input.html
@@ -0,0 +1 @@
+<meta charset='utf-8'><meta charset="utf-8"><b style="font-weight:normal;" id="docs-internal-guid-7f8963f3-7fff-683a-b3de-156ff66d4860"><ul style="margin-top:0;margin-bottom:0;"><li dir="ltr" style="list-style-type:disc;font-size:11pt;font-family:Arial;color:#000000;background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre;"><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt;" role="presentation"><span style="font-size:11pt;font-family:Arial;color:#000000;background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre;white-space:pre-wrap;">Item 1</span></p></li><li dir="ltr" style="list-style-type:disc;font-size:11pt;font-family:Arial;color:#000000;background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre;"><br /></li></ul></b>
diff --git a/packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/empty-list-item/model.html b/packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/empty-list-item/model.html
new file mode 100644
index 00000000000..c1d222b3978
--- /dev/null
+++ b/packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/empty-list-item/model.html
@@ -0,0 +1 @@
+<listItem listIndent="0" listType="bulleted">Item 1</listItem><listItem listIndent="0" listType="bulleted"></listItem>
\ No newline at end of file
diff --git a/packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/index.js b/packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/index.js
index 1e9926b3729..eee01a36938 100644
--- a/packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/index.js
+++ b/packages/ckeditor5-paste-from-office/tests/_data/paste-from-google-docs/lists/index.js
@@ -15,18 +15,23 @@ import repeatedlyNestedListModel from './repeatedly-nested-list/model.html';
 import partiallySelected from './partially-selected/input.html';
 import partiallySelectedModel from './partially-selected/model.html';
 
+import emptyListItem from './empty-list-item/input.html';
+import emptyListItemModel from './empty-list-item/model.html';
+
 export const fixtures = {
 	input: {
 		nestedOrderedList,
 		mixedList,
 		repeatedlyNestedList,
-		partiallySelected
+		partiallySelected,
+		emptyListItem
 	},
 	model: {
 		nestedOrderedList: nestedOrderedListModel,
 		mixedList: mixedListModel,
 		repeatedlyNestedList: repeatedlyNestedListModel,
-		partiallySelected: partiallySelectedModel
+		partiallySelected: partiallySelectedModel,
+		emptyListItem: emptyListItemModel
 	}
 };