On Feb 16, 9:09 pm, Jeremy Evans <[email protected]> wrote: > On Feb 16, 6:04 am, Clive Crous <[email protected]> wrote: > > 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/7300d2027133f125fc80fb349c... > > > Jeremy: I already sent you a pull request on github > > Certainly from your description this sounds like a bug. I'll look > into applying your patch tomorrow.
I merged your branch. Thanks! Jeremy --~--~---------~--~----~------------~-------~--~----~ 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 -~----------~----~----~----~------~----~------~--~---
