Merge pull request TheWidlarzGroup#1478 from laurent22/master

cobarx · Beau Ner · commit 943af0a5404b · 2019-10-10T20:52:24.000+01:00
Throw an error instead of crashing app when video source is empty (rebased from commit 98eb7a3)
diff --git a/Video.js b/Video.js
@@ -233,6 +233,8 @@ export default class Video extends Component {
     if (uri && uri.match(/^\//)) {
       uri = `file://${uri}`;
     }
+    
+    if (!uri) throw new Error('Trying to load empty source.');
 
     const isNetwork = !!(uri && uri.match(/^https?:/));
     const isAsset = !!(uri && uri.match(/^(assets-library|ipod-library|file|content|ms-appx|ms-appdata):/));

Original file line number	Diff line number	Diff line change
`@@ -233,6 +233,8 @@ export default class Video extends Component {`
`233`	`233`	`if (uri && uri.match(/^\//)) {`
`234`	`234`	uri = `file://${uri}`;
`235`	`235`	`}`
	`236`	`+`
	`237`	`+ if (!uri) throw new Error('Trying to load empty source.');`
`236`	`238`
`237`	`239`	`const isNetwork = !!(uri && uri.match(/^https?:/));`
`238`	`240`	`const isAsset = !!(uri && uri.match(/^(assets-library\|ipod-library\|file\|content\|ms-appx\|ms-appdata):/));`