search

deanlaw / cfcgenerator

Automatically exported from code.google.com/p/cfcgenerator
8 stars 6 forks source link

cfsqltype incorrect for numeric(18,0) columns in msssql #62

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. create a table with a column of type numeric(18,0) which is the default
for mssql server 2005 at least.
2. generate a dao from the table

What is the expected output? What do you see instead?
The cfqueryparam for that column should have a cfsqltype of cf_sql_numeric
but instead the cfsqltype is set to cf_sql_varchar

Original issue reported on code.google.com by judah.mc...@gmail.com on 3 Dec 2008 at 9:52

GoogleCodeExporter commented 9 years ago 
I found the culprit in the Illudium code. 

Line 85 of com.cf.model.datasource.mssql.cfc is:
<cfcase value="numeric">
  <cfreturn "cf_sql_varchar" />
</cfcase>

And should be:

<cfcase value="numeric">
  <cfreturn "cf_sql_numeric" />
</cfcase>

Original comment by judah.mc...@gmail.com on 4 Dec 2008 at 11:23

GoogleCodeExporter commented 9 years ago 
Issue is corrected in svn rev 154. shouldn't be cf_sql_numeric. Should be 
cf_sql_integer.

Original comment by brian.ri...@gmail.com on 5 Dec 2008 at 1:09