K2InformaticsGmbH / oranif

Oracle OCI driver using dirty NIF
Apache License 2.0
4 stars 2 forks source link

Fixed error wrapped twice in error tuple by the safe function. #109

Closed acautin closed 5 years ago

acautin commented 5 years ago

Current master:

 {error,
                                 {error,"c_src/dpiStmt_nif.c",47,
                                  #{action => "execute",code => 900,
                                    encoding => "UTF-8",
                                    fnName => "dpiStmt_execute",
                                    isRecoverable => false,
                                    message =>
                                     "ORA-00900: invalid SQL statement",
                                    offset => 0,sqlState => "HY000"}}}

This pr

{error,"c_src/dpiStmt_nif.c",47,
                                    #{action => "execute",code => 900,
                                      encoding => "UTF-8",
                                      fnName => "dpiStmt_execute",
                                      isRecoverable => false,
                                      message =>
                                          "ORA-00900: invalid SQL statement",
                                      offset => 0,sqlState => "HY000"}}
c-bik commented 5 years ago

@acautin process https://github.com/K2InformaticsGmbH/oranif/pull/110 into this PR first (to help with the checks)

acautin commented 5 years ago

@c-bik thanks for the fix for the unit tests 👍