Relax collection-id mapping

- Allow specifying generator as attribute
- Allow using column element instead of requiring attribute
- Set default value for type to Int32

Author: hazzik

src/NHibernate.Test/CollectionTest/IdBagFixture.hbm.xml

@@ -5,12 +5,12 @@
 			<generator class="native" />
 		</id>
 		<property name="Name" column="aname" />
-		<idbag name="Items" cascade="all-delete-orphan">
-			<collection-id type="Int32" column="item_id">
-				<generator class="increment" />
-			</collection-id>
-			<key column="a_id" />
-			<element type="string" />
-		</idbag>
+    <idbag name="Items" cascade="all-delete-orphan">
+      <collection-id type="Int32" column="item_id">
+        <generator class="increment" />
+      </collection-id>
+      <key column="a_id" />
+      <element type="string" />
+    </idbag>
 	</class>
 </hibernate-mapping>

src/NHibernate.Test/CollectionTest/IdBagRelaxedFixture.cs

@@ -0,0 +1,61 @@
+using System;
+using System.Collections;
+using System.Collections.Generic;
+using NUnit.Framework;
+
+namespace NHibernate.Test.CollectionTest
+{
+	[TestFixture]
+	public class IdBagRelaxedFixture : TestCase
+	{
+		protected override string[] Mappings
+		{
+			get { return new string[] { "CollectionTest.IdBagRelaxedFixture.hbm.xml" }; }
+		}
+
+		protected override string MappingsAssembly
+		{
+			get { return "NHibernate.Test"; }
+		}
+
+		protected override void OnTearDown()
+		{
+			using( ISession s = OpenSession() )
+			{
+				s.Delete( "from A" );
+				s.Flush();
+			}
+		}
+
+		[Test]
+		public void Simple()
+		{
+			A a = new A();
+			a.Name = "first generic type";
+			a.Items = new List<string>();
+			a.Items.Add( "first string" );
+			a.Items.Add( "second string" );
+
+			ISession s = OpenSession();
+			s.SaveOrUpdate( a );
+			// this flush should test how NH wraps a generic collection with its
+			// own persistent collection
+			s.Flush();
+			s.Close();
+			Assert.IsNotNull( a.Id );
+			Assert.AreEqual( "first string", ( string ) a.Items[ 0 ] );
+
+			s = OpenSession();
+			a = ( A ) s.Load( typeof( A ), a.Id );
+			Assert.AreEqual( "first string", ( string ) a.Items[ 0 ], "first item should be 'first string'" );
+			Assert.AreEqual( "second string", ( string ) a.Items[ 1 ], "second item should be 'second string'" );
+			// ensuring the correct generic type was constructed
+			a.Items.Add( "third string" );
+			Assert.AreEqual( 3, a.Items.Count, "3 items in the list now" );
+
+			a.Items[ 1 ] = "new second string";
+			s.Flush();
+			s.Close();
+		}
+	}
+}

src/NHibernate.Test/CollectionTest/IdBagRelaxedFixture.hbm.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<hibernate-mapping xmlns="urn:nhibernate-mapping-2.2" assembly="NHibernate.Test" namespace="NHibernate.Test.CollectionTest">
+	<class name="A" table="a" lazy="false">
+		<id name="Id" column="id" unsaved-value="null">
+			<generator class="native" />
+		</id>
+		<property name="Name" column="aname" />
+		<idbag name="Items" cascade="all-delete-orphan">
+			<collection-id column="item_id" generator="increment" />
+			<key column="a_id" />
+			<element type="string" />
+		</idbag>
+	</class>
+</hibernate-mapping>

src/NHibernate.Tool.HbmXsd/Program.cs

@@ -10,7 +10,7 @@ public class Program
 		private static void Main(string[] args)
 		{
 			string outFile = Path.GetFullPath(args.Length == 0
-				? @"..\..\..\..\NHibernate\Cfg\MappingSchema\Hbm.generated.cs"
+				? Path.Combine("..", "..", "..", "..", "NHibernate", "Cfg", "MappingSchema", "Hbm.generated.cs")
 				: args[0]);
 			if (!Directory.Exists(Path.GetDirectoryName(outFile)))
 			{
@@ -19,11 +19,12 @@ private static void Main(string[] args)
 				Environment.ExitCode = -1;
 				return;
 			}
+
 			var currentUiCulture = new CultureInfo("en-us");
 			Thread.CurrentThread.CurrentCulture = currentUiCulture;
 			Thread.CurrentThread.CurrentUICulture = currentUiCulture;
 			new HbmCodeGenerator().Execute(outFile);
 			Console.WriteLine("Done");
 		}
 	}
-}
+}