Merge pull request #113 from Starwer/fix_multi_genres

Fixed: Multiple genre's is not being handled correctly

Author: decriptor

src/TagLib/Id3v2/Frames/TextIdentificationFrame.cs

@@ -916,7 +916,7 @@ protected void ParseRawData ()
 					}
 
 					if (value.Length > 0)
-						field_list.AddRange (value.Split (new char [] {'/'}));
+						field_list.AddRange (value.Split (new char [] {'/', ';'}));
 				} else {
 					field_list.Add (value);
 				}
@@ -988,7 +988,7 @@ protected override ByteVector RenderFields (byte version) {
 				StringBuilder data = new StringBuilder ();
 				foreach (string s in text) {
 					if (!prev_value_indexed) {
-						data.Append ("/").Append (s);
+						data.Append (";").Append (s);
 						continue;
 					}
 

tests/fixtures/TagLib.Tests.FileFormats/Id3V2FormatTest.cs

@@ -38,7 +38,27 @@ public void ReadTags()
 			Assert.AreEqual(7, file.Tag.TrackCount);
 			Assert.AreEqual(1234, file.Tag.Year);
 		}
-		
+
+		[Test]
+		public void MultiGenresTest()
+		{
+			string inFile = TestPath.Samples + "sample.mp3";
+			string tempFile = TestPath.Samples + "tmpwrite.mp3";
+
+			File rgFile = File.Create(inFile);
+			var tag = rgFile.Tag;
+			var genres = tag.Genres;
+
+			Assert.AreEqual(3, genres.Length);
+			Assert.AreEqual("Genre 1", genres[0]);
+			Assert.AreEqual("Genre 2", genres[1]);
+			Assert.AreEqual("Genre 3", genres[2]);
+
+			rgFile.Dispose();
+			System.IO.File.Delete(tempFile);
+		}
+
+
 		[Test]
 		public void WriteStandardTags ()
 		{