views:

1513

answers:

1

I've crossed posted this in the FNH Google Group but here's the scoop:

I have a table in a legacy database that has a composite ID. Well, the PK is implied since the table doesn't actually have a PK defined, but the natural PK is the combination of the WORKORDERID and IMAGEPATH columns:

CREATE TABLE [WORKORDERIMG](
       [WORKORDERID] [varchar](50) NULL,
       [IMAGEPATH] [varchar](250) NULL,
       [WOTASKID] [numeric](10, 0) NULL,
       [ATTACHEDBY] [nvarchar](100) NULL,
       [COMMENTS] [nvarchar](256) NULL,
       [DATETIMEATTACHED] [datetime] NULL
) ON [PRIMARY]

I have this class map defined:

   /// <summary>
   /// NHibernate mapping for workorder attachments
   /// </summary>
   public class AttachmentMap : ClassMap<Attachment>
   {
       public AttachmentMap()
       {
           SchemaIs(Resources.DatabaseSchema);
           WithTable(ObjectNames.TableWorkorderAttachment);
           UseCompositeId()
               .WithKeyProperty(x => x.ParentId, "WORKORDERID")
               .WithKeyProperty(x => x.FileLocation, "IMAGEPATH");
           Map(x => x.AttachedByUser, "ATTACHEDBY").WithLengthOf(100).Nullable();
           Map(x => x.AttachedOn, "DATETIMEATTACHED").Nullable();
           Map(x => x.Comments).Nullable().WithLengthOf(256);
           References(x => x.ParentWorkorder).FetchType.Join();
       }
   }

For this class:

public class Attachment
{
   public virtual string ParentId { get; set; }
   public virtual string FileLocation { get; set; }
   public virtual Workorder ParentWorkorder { get; set; }
   public virtual string AttachedByUser { get; set; }
   public virtual string Comments { get; set; }
   public virtual DateTime? AttachedOn { get; set; }

   public override bool Equals(object obj)
   {
       return this.IsEqual(obj);
   }

   public override int GetHashCode()
   {
       return HashCodeGenerator.GenerateHashCode(new object[] { FileLocation, ParentId });
   }
}

Now I know that I should be defining the WORKORDERID column in the classmap for the References line as well. With it excluded I get the expected error:

NHibernate: INSERT INTO azteca.WorkOrderImg (ATTACHEDBY,
DATETIMEATTACHED, Comments, ParentWorkorder_id, WORKORDERID,
IMAGEPATH) VALUES (@p0, @p1, @p2, @p3, @p4, @p5);@p0 = 'SYSTEM', @p1 = 3/15/2009 12:00:00 AM, @p2 = 'Some comment', @p3 = NULL, @p4 = NULL, @p5 = 'some ile\location'

System.Data.SqlClient.SqlException: Invalid column name 'ParentWorkorder_id'.

However, when I include it I get Fluent NHibernate trying to add the WORKORDERID column to the query twice:

System.IndexOutOfRangeException: Invalid index 5 for this SqlParameterCollection with Count=5.

I've been trying every combination I can think of to no avail. I'm testing the mapping with this code:

       [Test]
       public void Can_persist_an_attachment()
       {
           var sf = ObjectFactory.GetNamedInstance<ISessionFactory>   (ObjectMother.Datasource);

           using (ISession session = sf.OpenSession())
           {
               new PersistenceSpecification<Attachment>(session)
                   .CheckProperty(x => x.AttachedByUser, "SYSTEM")
                   .CheckProperty(x => x.AttachedOn, new DateTime(2009, 3, 15))
                   .CheckProperty(x => x.Comments, "Some comment")
                   .CheckProperty(x => x.FileLocation, "some\file\\location")
                   .VerifyTheMappings();
           }
       }

Any help appreciated.

+2  A: 

Figured it out. You don't need to add the reference property for the parent. Notice how the many-to-one is missing in the adjusted mapping:

public AttachmentMap()
{
    SchemaIs(Resources.CityworksDatabaseSchema);
    WithTable(ObjectNames.TableWorkorderAttachment);
    UseCompositeId()
     .WithKeyReference(x => x.ParentWorkorder, "WORKORDERID")
     .WithKeyProperty(x => x.FileLocation, "IMAGEPATH");
    Map(x => x.AttachedByUser, "ATTACHEDBY").WithLengthOf(100).Nullable();
    Map(x => x.AttachedOn, "DATETIMEATTACHED").Nullable();
    Map(x => x.Comments).Nullable().WithLengthOf(256);
    //References(x => x.ParentWorkorder, "WORKORDERID").FetchType.Join();
}

This results in the following HBM mapping file. The fix is to have the composite-id have a key-many-to-one line pointing back to the parent class. Now I can persist my Attachment class.

<hibernate-mapping xmlns="urn:nhibernate-mapping-2.2" default-access="">
  <class name="Woolpert.Cityworks.Core.Entities.Azteca.Attachment, Woolpert.Cityworks.Core, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null" table="WorkOrderImg" xmlns="urn:nhibernate-mapping-2.2" schema="azteca">
    <composite-id>
      <key-many-to-one class="Woolpert.Cityworks.Core.Entities.Azteca.Workorder, Woolpert.Cityworks.Core, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null" name="ParentWorkorder" column="WORKORDERID" />
      <key-property type="String" name="FileLocation" column="IMAGEPATH" />
    </composite-id>
    <property name="AttachedByUser" type="String">
      <column name="ATTACHEDBY" length="100" not-null="false" />
    </property>
    <property name="AttachedOn" type="System.Nullable`1[[System.DateTime, mscorlib, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089]]">
      <column name="DATETIMEATTACHED" not-null="false" />
    </property>
    <property name="Comments" type="String">
      <column name="Comments" length="256" not-null="false" />
    </property>
  </class>
</hibernate-mapping>
Dylan