Fix two issues caused by earlier embedded player NC28-compatibility f…

…ixes

- It turned out that appending the OC.requestToken to the play URL was not
  totally unnecessary after all: without it, using the Aurora.js fallback
  player (to play e.g. AIFF files) didn't work although all the natively
  supported files played fine.

- The logic to show "Import playlist" or "Import radio" didn't work correctly
  because it was based on the assumption that only external streams may have
  the property `url` set

js/embedded/main.js

@@ -53,6 +53,18 @@ function initEmbeddedPlayer() {
 
 	register();
 
+	function urlForFile(file) {
+		let url = file.url ?? mFileList.getDownloadUrl(file.name, file.path);
+
+		// Append request token unless this is a public share. This is actually unnecessary for most files
+		// but needed when the file in question is played using our Aurora.js fallback player.
+		if (!mShareToken) {
+			let delimiter = _.includes(url, '?') ? '&' : '?';
+			url += delimiter + 'requesttoken=' + encodeURIComponent(OC.requestToken);
+		}
+		return url;
+	}
+
 	function onClose() {
 		mCurrentFile = null;
 		mPlayingListFile = false;
@@ -89,7 +101,7 @@ function initEmbeddedPlayer() {
 
 		// disable/enable the "Import list to Music" item
 		let inLibraryFilesCount = _(mPlaylist.files()).filter('in_library').size();
-		let extStreamsCount = _(mPlaylist.files()).filter('url').size();
+		let extStreamsCount = _(mPlaylist.files()).filter('external').size();
 		let outLibraryFilesCount = mPlaylist.length() - inLibraryFilesCount;
 
 		let $importListItem = $menu.find('#playlist-menu-import');
@@ -171,7 +183,7 @@ function initEmbeddedPlayer() {
 				mPlayer.playExtUrl(file.url, file.caption, mShareToken);
 			} else {
 				mPlayer.playFile(
-					file.url ?? mFileList.getDownloadUrl(file.name, file.path),
+					urlForFile(file),
 					file.mimetype,
 					file.id,
 					file.name,