On Feb 16, 6:04 am, Clive Crous <[email protected]> wrote:
> 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/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.
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
-~----------~----~----~----~------~----~------~--~---
