-
Notifications
You must be signed in to change notification settings - Fork 122
add support for BLOB data in OCI8 driver #396
base: develop
Are you sure you want to change the base?
Conversation
parktrip
commented
Oct 16, 2019
- add 2 new data types in ParameterContainer class in order to manage differently BLOB from CLOB,
- for oci8 driver: add a new case statement in bindParametersFromContainer() method to save data binded as BLOB
@parktrip thanks for your contribution. We need also test cases added to cover changes. |
dear @webimpress,
Nevertheless, test cases for oci8 run fine ;) |
@parktrip thanks for update, we can have a look on it closer before we merge it. |
dear @webimpress, I just took 5mn this morning to check travis error logs and it seems the oci_pconnect() function is not recognized. Are you sure the oci8 extension is activated in travis config file for all php versions ? |
@parktrip it might be not enabled, but you can change it in |
no pb I understand. I will have a look a little bit later hopefully by the end of the week |
… differently BLOB from CLOB, - for oci8 driver: add a new case statement in bindParametersFromContainer() method to save data binded as BLOB
…other drivers, - created 3 test cases to test the blob/clob implementation
75930e2
to
043c136
Compare
@parktrip I've tried to update it and run it on travis but it seems to be uncompleted. |
This repository has been closed and moved to laminas/laminas-db; a new issue has been opened at laminas/laminas-db#2. |
This repository has been moved to laminas/laminas-db. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:
|