Hi,

I found a problem when passing multiple arguments to call_sproc. I'm
still quite new to Sequel's source so I'm not entirely sure if I'm
correcting the unnecessary -- details follow ;)

when calling call_sproc as follows:
  Dataset#call_sproc( :select, :foo, 1, 2 )

it becomes:
  "CALL foo((1,2))"

This patch corrects/removes that extra pair of braces which caused:
  Mysql::Error Incorrect number of arguments for PROCEDURE sandbox.test_sproc;

Patch with unit tests to test for this here:
http://github.com/clivecrous/sequel/commit/7300d2027133f125fc80fb349c85be234f84edf9

Jeremy: I already sent you a pull request on github

Thanks,
Clive

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sequel-talk" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/sequel-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to