Skip to content

Commit f946b86

Browse files
authored
Merge pull request #56 from TimGels/implement-workaround-mkv
Implement workaround for mkv files not showing
2 parents 2361575 + 5a0a43d commit f946b86

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

OneDrive-Cloud-Player/Package.appxmanifest

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
<Identity
1010
Name="22674TimGels.OneDriveCloudPlayer"
1111
Publisher="CN=3095B4D2-D597-43C2-BFA6-AC26DB73293D"
12-
Version="1.2.0.0" />
12+
Version="1.2.1.0" />
1313

1414
<mp:PhoneIdentity PhoneProductId="329101d2-9ad1-4065-a890-d72be5ed3714" PhonePublisherId="00000000-0000-0000-0000-000000000000"/>
1515

OneDrive-Cloud-Player/Services/Helpers/CacheHelper.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
using OneDrive_Cloud_Player.Services.Utilities;
77
using System;
88
using System.Collections.Generic;
9+
using System.IO;
910
using System.Linq;
1011
using System.Threading;
1112
using System.Threading.Tasks;
@@ -454,8 +455,9 @@ public CachedDriveItem ConvertGraphItem(DriveItem GraphItem)
454455
return null;
455456
}
456457

458+
//TODO: Remove the mkv extension workaround when Graph is fixed or a better solution is implemented.
457459
// Check that files have the correct mimetype (either contains audio or video).
458-
if (GraphItem.File != null && !(GraphItem.File.MimeType.Contains("video") || GraphItem.File.MimeType.Contains("audio")))
460+
if (GraphItem.File != null && !(GraphItem.File.MimeType.Contains("video") || GraphItem.File.MimeType.Contains("audio") || Path.GetExtension(GraphItem.Name).Equals(".mkv")))
459461
{
460462
return null;
461463
}

0 commit comments

Comments
 (0)