Fixed issue with movement removal

sandreas · Mar 9, 2024 · ffcad81 · ffcad81
1 parent 7e5699b
commit ffcad81
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 8 deletions.
diff --git a/Sandreas.AudioMetadata.Tests/AudioMetadata/MetadataTrackTest.cs b/Sandreas.AudioMetadata.Tests/AudioMetadata/MetadataTrackTest.cs
@@ -3,15 +3,24 @@ namespace Sandreas.AudioMetadata.Tests.AudioMetadata;
 public class MetadataTrackTest
 {
     [Fact]
-    public void TestMappedProperties()
+    public void TestRemoveMovement()
     {
-        var exception = Record.Exception(() =>
+        var t = new MetadataTrack
         {
-            var track = new MetadataTrack
-            {
-                EncodingTool = "testing"
-            };
-        });
-        Assert.Null(exception);
+            Movement = "1"
+        };
+        t.RemoveMetadataPropertyValue(MetadataProperty.Movement);
+        Assert.Null(t.Movement);
+    }
+
+    [Fact]
+    public void TestRecordingDate()
+    {
+        var t = new MetadataTrack
+        {
+            RecordingDate = DateTime.Today
+        };
+        t.RemoveMetadataPropertyValue(MetadataProperty.RecordingDate);
+        Assert.True(t.RecordingDate <= DateTime.MinValue);
     }
 }
diff --git a/Sandreas.AudioMetadata/AudioMetadata/IMetadataExtensions.cs b/Sandreas.AudioMetadata/AudioMetadata/IMetadataExtensions.cs
@@ -175,6 +175,11 @@ public static void SetMetadataPropertyValue(this IMetadata metadata, MetadataPro
     /// <returns></returns>
     private static object? GetPropertyValueThatLeadsToRemoval(this IMetadata metadata, MetadataProperty property)
     {
+        if (property == MetadataProperty.Movement)
+        {
+            return null;
+        }
+
         if (metadata.GetMetadataPropertyType(property) == typeof(string))
         {
             return "";

diff --git a/Sandreas.AudioMetadata/AudioMetadata/MetadataTrack.cs b/Sandreas.AudioMetadata/AudioMetadata/MetadataTrack.cs
@@ -118,6 +118,10 @@ public string? Movement
         set {
             // movement MUST contain an integer value, which leads to an exception, if a string like 1.5 is stored
             // to store a non-integer value, use Part instead
+            if (value == "")
+            {
+                value = null;
+            }
             if (value == null || int.TryParse(value, out _))
             {
                 SeriesPart = value;