bindParam doesn't use type and leng informations about pdooci HOT 6 CLOSED

What's the result of the regular PDOOCI implementation?

from pdooci.

It work's fine with the regular PDOOCI implementation. You get a proper int as a result.

from pdooci.

Could you please change, on Statement.php:

\oci_bind_by_name($this->_stmt, $param, $value);

to

\oci_bind_by_name($this->_stmt, $param, $value, $leng, $type);

and see if it works? Or provide a test to check the desired behaviour?

from pdooci.

Ok, thank you. I'll try that and provide a test.

from pdooci.

Thanks. :-)

from pdooci.

We really need to run tests. I just made the change I told above and everything broke. We really need to send the default expected values, so:

bindParam expects:

1. parameter
2. variable
3. optional data type, defaults to PDO::PARAM_STR
4. optional lenght, defaults to ... nothing, or null!

oci_bind_by_name expects:

1. statement
2. name
3. variable
4. length, defaults to -1
5. type, defaults to SQLT_CHR

Then I changed again the code sending PDO::PARAM_STR and -1 as default values, and it broke again. Then I created an if checking if length and type were not null (need both because the last optional on bindParam is length, so type would be filled even with null), if so, they are send to oci_bind_by_name (on inverse order, duh!), and seems it worked on the way that didn't broke tests and previous behaviours. I'll commit on the next minutes so you can test if it worked.

from pdooci.