Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the DBD-Oracle CPAN distribution.

Report information
The Basics
Id: 4786
Status: resolved
Priority: 0/
Queue: DBD-Oracle

People
Owner: Nobody in particular
Requestors: rob.kinyon [...] progressive-medical.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.14
Fixed in: (no value)

Attachments


Subject: Oracle 9.2.0.0 fails a test in ph_types.t
Download (untitled) / with headers
text/plain 695b
Assuming that ORA_OCI() gets set correctly when compiling against 9.2, the attached patch will work. I also tried this in SQL*Plus and was able to insert a trailing space into a VARCHAR2. (I replicated the test in ph_types.t). I did not test my patch as I installed DBD::Oracle 1.14 by setting the chops_spaces value in %test_info to 0. When I did that, everything installed fine. However, I didn't think that my solution was the best for the module, so I figured ORA_OCI should do the trick. I'm running Perl5.8.0 for Solaris2.9 going against the full Oracle build for 9.2. (I did not run into this issue, surprisingly, on Redhat9 running Perl 5.8.2, but I built against Oracle 9.1 there ...)
Download ph_type.diff
text/x-diff 507b
112,114c112,120 < if (!$ok && ORA_OCI==7 && $name eq 'VARCHAR2') { < warn " OCI7 ora_type=1 placeholder doesn't strip trailing spaces, OCI8 currently does\n"; < $ok = 1; --- > if (!$ok && $name eq 'VARCHAR2') { > if (ORA_OCI==7) { > warn " OCI7 ora_type=1 placeholder doesn't strip trailing spaces, OCI8 currently does\n"; > $ok = 1; > } > elsif (ORA_OCI==9) { > warn " OCI9 ora_type=1 placeholder doesn't strip trailing spaces\n"; > $ok = 1; > }
Download (untitled) / with headers
text/plain 135b
Thanks for the patch. I'm waiting to find out if it's an Oracle bug or not. Meanwhile I'll change the test failure to a warning. Tim.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.