Sorry for the delay, here is the patch that adds support for nullable types

On Fri, Oct 10, 2008 at 12:30 PM, Paul Batum <[EMAIL PROTECTED]> wrote:

> Hi Pavel,
>
> I would be very happy to apply an patch for this. I am sure there are
> plenty of other people that prefer to use nullable types for their primary
> keys.
>
> Paul Batum
>
> On Tue, Oct 7, 2008 at 10:07 PM, Pavel Samokha <[EMAIL PROTECTED]>wrote:
>
>>
>> Found another problem - Fluent NHibernate not supports nullable types
>> as Id's (I prefer null instead of 0 for unsaved values)
>>
>> throws exception - Identity type must be integral (int, long, uint,
>> ulong).
>>
>> So need to change IdentityGenerationStrategyBuilder.isIntegralType to
>> add nullables (and I think also need to change the same for Guid and
>> string).
>>
>> Also note that IdentityGenerationStrategyBuilder has camelcase - java-
>> style - method names (start with small letters), another code, that I
>> saw in Fluent NHibernate use default PascalCase style.
>>
>> If needed, I can contribute changes for this.
>>
>>
>
> >
>


-- 
Yours sincerely, Pavel Samokha
icq uin #1936398
skype generalvansickle

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Fluent NHibernate" group.
To post to this group, send email to fluent-nhibernate@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/fluent-nhibernate?hl=en
-~----------~----~----~----~------~----~------~--~---

Attachment: nullable_identity_patch.patch
Description: Binary data

Reply via email to