may be like this, to minimize the change
<cfcase value="guid,uniqueidentifier">
<cfif ReFindNoCase("'*newId()'*", arguments.sqlDefaultValue)>
<cfreturn "##insert('-', MSguid, 23)##" />
<cfelse>
<cfreturn "" />
</cfif>
</cfcase>
On 3/23/07, Ilya Fedotov <[EMAIL PROTECTED]> wrote:
I say submit it, I didn't dig that deep. Your solution is much better,
there is no reason why reactor shouldn't handle this, specialy there is
already code there that intends to do so.
Did you try to add your code?
On 3/21/07, Nathan Mische <[EMAIL PROTECTED]> wrote:
>
> I ran into an issue while trying to use MGU scaffolding with a table
> which had a uniqueidentifer primary key. I saw Ilya Fedotov's post
> "UUID as primary key" but this seemed like something Rector should
> handle without any custom code. If a uniqueidentifer field has a
> default value of "(newid())" then the project TO should set this
> value.
>
> I did some digging and found that due to a bug in the MSSQL
> ObjectDao.cfc the field's default value is not properly set. The
> problem is that the ObjectDao's getDefault method has a case statement
> based on the cfDataType of the field, however the case near line 128
> is for the dbDataType "uniqueidentifier." I think that case should be
> updated to something like:
>
> <cfcase value="guid">
> <cfif ReFindNoCase("'*newId()'*", arguments.sqlDefaultValue)>
> <cfreturn "##Insert(""-"",CreateUUID(),23)##" />
> <cfelse>
> <cfreturn "" />
> </cfif>
> </cfcase>
>
> I'm new to Model-Glue and Reactor so would someone care to confirm my
> findings before I submit a bug?
>
> Thanks,
>
> --Nathan
>
>
> -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
> -- -- --
> Reactor for ColdFusion Mailing List
> [email protected]
> Archives at: http://www.mail-archive.com/reactor%40doughughes.net/
> -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
> -- -- --
>
>
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
Reactor for ColdFusion Mailing List
[email protected]
Archives at: http://www.mail-archive.com/reactor%40doughughes.net/
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
