Merge pull request #254 from hwahrmann/Property_preventing_autocreate_ID3_TagTypes

Property preventing autocreate id3 tag types

Author: decriptor

src/TaglibSharp.Tests/FileFormats/Id3BothFormatTest.cs

@@ -109,5 +109,29 @@ public void TestRemoveTags ()
 			file.RemoveTags (TagTypes.AllTags);
 			Assert.AreEqual (TagTypes.None, file.TagTypes);
 		}
+
+		[Test]
+		public void TestCreateId3Tags ()
+		{
+			string tempFile = TestPath.Samples + "tmpwrite_sample_createid3tags.mp3";
+
+			System.IO.File.Copy (sample_file, tempFile, true);
+
+			// Remove All Tags first
+			var file = File.Create (tempFile);
+			file.RemoveTags (TagTypes.AllTags);
+			file.Save ();
+
+			// No TagTypes should exist
+			TagLib.Mpeg.AudioFile.CreateID3Tags = false;
+			file = File.Create (tempFile);
+			Assert.AreEqual (TagTypes.None, file.TagTypes);
+			file.Save ();
+
+			// Empty TagTypes should be created
+			TagLib.Mpeg.AudioFile.CreateID3Tags = true;
+			file = File.Create (tempFile);
+			Assert.AreEqual (TagTypes.Id3v1 | TagTypes.Id3v2, file.TagTypes);
+		}
 	}
 }

src/TaglibSharp/Mpeg/AudioFile.cs

@@ -68,6 +68,16 @@ public class AudioFile : TagLib.NonContainer.File
 		#endregion
 
 
+		#region Private Static Fields
+
+		/// <summary>
+		///    Specifies whether or not to create ID3v1 and
+		///    ID3v2 tags when they don't exist..
+		/// </summary>
+		private static bool create_id3_tags = true;
+
+		#endregion
+
 
 		#region Constructors
 
@@ -154,6 +164,33 @@ public AudioFile (IFileAbstraction abstraction)
 		#endregion
 
 
+		#region Public Static Properties
+
+		/// <summary>
+		///    Gets and sets whether or not to create ID3v1 and
+		///    ID3v2 tags automatically when they are not existing.
+		/// </summary>
+		/// <value>
+		///    <see langword="true" /> if tags to be created automatically.
+		///    Otherwise, <see langword="false" />.
+		/// </value>
+		/// <remarks>
+		///    <para>Sometimes a MP3 file should only contain ID3v1 and no
+		///    ID3v2 Tags. Or instead of ID3v2 Tags APE Tags should be used.
+		///    By setting this property to <see langword="false" />,
+		///    no ID3v1 and Id3v2 Tags will be created when creating the file,
+		///    if they don't exist.
+		///    They need to be created explicitly if needed.</para>
+		///    <para>The default is <see langword="true" /> which means that
+		///    ID3v1 and Id3v2 tags are created when they don't exist.</para>
+		/// </remarks>
+		public static bool CreateID3Tags {
+			get { return create_id3_tags; }
+			set { create_id3_tags = value; }
+		}
+
+		#endregion
+
 
 		#region Public Methods
 
@@ -252,9 +289,9 @@ protected override void ReadStart (long start, ReadStyle propertiesStyle)
 		/// </param>
 		protected override void ReadEnd (long end, ReadStyle propertiesStyle)
 		{
-			// Make sure we have ID3v1 and ID3v2 tags.
-			GetTag (TagTypes.Id3v1, true);
-			GetTag (TagTypes.Id3v2, true);
+			// Creation of ID3v1 and ID3v2 tags based on CreateID3Tags property
+			GetTag (TagTypes.Id3v1, create_id3_tags);
+			GetTag (TagTypes.Id3v2, create_id3_tags);
 		}
 
 		/// <summary>