Fix for parameter types.

[charsleysa]

Solves #196

[treffynnon]

Not sure that we can always assume is_numeric is an integer. What about
float values?

Not sure about defaulting to null on execute rather than an empty array
instead. Is there a reason for this?
On May 29, 2014 7:38 AM, "Stefan Andres Charsley" notifications@github.com
wrote:

Solves #196 #196

You can merge this Pull Request by running

git pull https://github.com/charsleysa/idiorm develop

Or view, comment on, or merge it at:

#206
Commit Summary

• Fix for parameter types.

File Changes

• M idiorm.php
  https://github.com/j4mie/idiorm/pull/206/files#diff-0 (15)

Patch Links:

• https://github.com/j4mie/idiorm/pull/206.patch
• https://github.com/j4mie/idiorm/pull/206.diff

—
Reply to this email directly or view it on GitHub
#206.

[charsleysa]

Good call on the is_numeric I'll patch that.

As for defaulting to null on execute, thats only used in the Mock PDO Statement object and is so that it's easy to differentiate when someone passes an array to the function.

The variable gets replaced if its NULL and if it's an array it gets left alone.

[ghost]

Why is $parameters[$i] = +($parameters[$i]); necessary, if is_int() returned true?

[treffynnon]

@charsleysa the question from @cainmi was going to be my very next question now I am in front of a computer again instead of on my phone.

[charsleysa]

Sorry, habit from Javascript, will patch.