I think there's mismatch between SP parameters and parameters you're passing. Use Strtofile function to write lcStr variable to a file, open the file in VFP editor and than compare it to the order of parameters SP expects. Also, it's customary to use [] as the string delimiters when dynamicaly building strings. That allows to use quote and double quote inside such string and make code easier to read.

lcStr = lcStr + ['SA']

>>Can you post the code that shows how you call SP and pass parameters?
>
>
>Here's a code snippet :
>
>
>* Connect to DS
>lnConnection=SQLConnect("dimensions","Sa","Sa")
>
>* Name of stored procedure
>lcStr="AA_SL_ACCOUNT_INSERT_EDIT_S "
>
>* Numeric, I think this is causing the problem, This field is a tinyint in *SQL
>lcStr=lcStr+"1"+","
>
>&& String
>lcStr=lcStr+Chr(34)+'SA'+Chr(34)
>* Actual code has about 80 params
>* but they are all structured like the two above
>
>lnresult = SQLExec(lnConnection)
>
>lnResult = SQLDisconnect(lnconnection)