Patch for db2hbm

10 views
Skip to first unread message

John Weldon

unread,
Dec 30, 2010, 5:29:56 PM12/30/10
to NHibernate Contrib - Development Group
I ran into an issue with Configured Foreign Keys in the config, and
traced it down to a typo in the constructor for
ConfiguredForeignKeyColumnInfo.


I'm having trouble building the dll myself, I can't get dll references
to the Microsoft Templating dll's right.

Any chance we could get a new binary drop with this fix?




Index: KeyAwareMetadataStrategy.cs
===================================================================
--- KeyAwareMetadataStrategy.cs (revision 1530)
+++ KeyAwareMetadataStrategy.cs (working copy)
@@ -197,7 +197,7 @@
ForeignKeyColumnName = cref.localcolumn;
ForeignKeyTableCatalog = metaData.Catalog;
ForeignKeyTableSchema = metaData.Schema;
- ForeignKeyTableSchema = metaData.Name;
+ ForeignKeyTableName = metaData.Name;
}
#region IForeignKeyColumnInfo Members

John Weldon

unread,
Dec 30, 2010, 5:43:40 PM12/30/10
to NHibernate Contrib - Development Group
FWIW, I got this to compile and it did fix the defect.

FaticaLabs

unread,
Dec 31, 2010, 4:09:44 AM12/31/10
to NHibernate Contrib - Development Group
John
Thankf for the fix, I will put it in the nex version,

FaticaLabs

unread,
Jan 3, 2011, 3:32:08 AM1/3/11
to NHibernate Contrib - Development Group
John,
Your fix is now in the trunk, and the binary in teh NHforge download
are now updated.
There is actually some testing leak on the configuration part of
foreign keys, mostly because foreign key metadata are fetched from
database.
But no excuse, I need to put more test in that part.
Thanks again,
Felix.

John Weldon

unread,
Jan 3, 2011, 12:15:09 PM1/3/11
to nhc...@googlegroups.com
Felix,

Thanks for the update, and your hard work on this project!

Cheers,
--
John Weldon


--
You received this message because you are subscribed to the Google Groups "NHibernate Contrib - Development Group" group.
To post to this group, send email to nhc...@googlegroups.com.
To unsubscribe from this group, send email to nhcdevs+u...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/nhcdevs?hl=en.


Reply all
Reply to author
Forward
0 new messages