Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CHAR should return nil for NULL values in column #2289

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sprileyMSTS
Copy link

lib/active_record/connection_adapters/oracle_enhanced/oci_connection.rb

  • Return nil for NULL value in CHAR column similar to behavior in
    lib/active_record/connection_adapters/oracle_enhanced/jdbc_connection.rb

lib/active_record/connection_adapters/oracle_enhanced/oci_connection.rb
- Return nil for NULL value in CHAR column similar to behavior in
  lib/active_record/connection_adapters/oracle_enhanced/jdbc_connection.rb
@sprileyMSTS
Copy link
Author

Let me know if it's possible to backport this fix into a patch release of the 6.0 and 6.1 versions of this adapter. If not I'll have to release something internally at my company to address this issue until we can upgrade to Rails 7.

@yahonda
Copy link
Collaborator

yahonda commented Jun 7, 2022

Thanks for opening a pull request. Would you add some specs?
Unfortunately, ActiveRecord Oracle enhanced adapter 6.1.z and 6.0.z are not actively maintained so no back port will be done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants