Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Error with BigInt parameter on callProcedure() #47

Closed
zacharyyates opened this Issue Aug 24, 2012 · 4 comments

Comments

Projects
None yet
3 participants

I have a stored procedure in SQL Server 2008 that takes a BigInt as it's only parameter - I ran into this error while trying to call that proc:

TypeError: Object #<Object> has no method 'writeParameterData'
at new RpcRequestPayload (C:\Projects\WT\source\server\api\node_modules\tedious\lib\rpcrequest-payload.js:52:22)
at Connection.callProcedure (C:\Projects\WT\source\server\api\node_modules\tedious\lib\connection.js:575:56)
at Connection.Repository.execute (C:\Projects\WT\source\server\api\lib\data\repository.js:71:16)
at Connection.EventEmitter.emit (events.js:85:17)
at Connection.processedInitialSql (C:\Projects\WT\source\server\api\node_modules\tedious\lib\connection.js:538:17)
at Connection.STATE.LOGGED_IN_SENDING_INITIAL_SQL.events.message (C:\Projects\WT\source\server\api\node_modules\tedious\lib\connection.js:170:23)
at Connection.dispatchEvent (C:\Projects\WT\source\server\api\node_modules\tedious\lib\connection.js:433:59)
at MessageIO.Connection.connectOnPort (C:\Projects\WT\source\server\api\node_modules\tedious\lib\connection.js:390:20)
at MessageIO.EventEmitter.emit (events.js:85:17)
at MessageIO.eventData (C:\Projects\WT\source\server\api\node_modules\tedious\lib\message-io.js:59:21)

Not sure that it matters that much, since I can pass the param as Int.

Collaborator

pekim commented Aug 24, 2012

This is likely to be resolved when I get my act together and deal with pull rest #46.

Contributor

zachaller commented Aug 24, 2012

pull request #45 :)

Collaborator

pekim commented Sep 29, 2012

Pull request #45 has been taken. Can you confirm that this is no longer a problem?

I tested a few sprocs and they work fine, go ahead and close it. Thanks!

@momow momow pushed a commit to momow/tedious that referenced this issue Oct 15, 2014

@bretcope bretcope Implemented automated flushing script. ref #47 ca089f2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment