Skip to content
New issue

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

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

Already on GitHub? # to your account

Improve 18c support #208

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

Improve 18c support #208

wants to merge 20 commits into from

Conversation

joschug
Copy link

@joschug joschug commented Oct 27, 2021

This fixes #204, #207 and probably #200.

  • Resolve type synonyms
  • Support cross-schema type references
  • Added support for package types declared as %ROWTYPE
  • Added support for %ROWTYPE as procedure parameter
  • Exclude hidden columns from table types
  • Remap TIMESTAMP(n) WITH LOCAL TIMEZONE to TIMESTAMP WITH LOCAL TIMEZONE
  • Remap PL/SQL PLS/BINARY INTEGER to PLS_INTEGER/BINARY_INTEGER

# Conflicts:
#	lib/plsql/procedure.rb
"timestamp(6) with local timezone"; before 18c, this was represented as
"timestamp with local timezone"
# Conflicts:
#	lib/plsql/procedure.rb
@yahonda
Copy link
Collaborator

yahonda commented May 12, 2023

Could you please break this pull request into smaller pieces?

@Faq
Copy link

Faq commented Aug 23, 2024

Using this for a year with Rails 5.2 and 18c, now upgraded Rails to v7.0, still works.

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

Successfully merging this pull request may close these issues.

ruby-plsql 0.8.0: Procedure with parameters referencing system packages fails
3 participants