Skip to content

/ utPLSQL-SQLDeveloper

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

Jump to bottom

Improve error handling when no utPLSQL is installed #64

Closed
PhilippSalvisberg opened this issue Jul 11, 2019 · 1 comment
Closed

Improve error handling when no utPLSQL is installed #64

PhilippSalvisberg opened this issue Jul 11, 2019 · 1 comment
Assignees
@PhilippSalvisberg
Labels
enhancement
Milestone
v1.1.0

Comments

@PhilippSalvisberg
Copy link
Member

@PhilippSalvisberg PhilippSalvisberg commented Jul 11, 2019

In version < 1.0.0 the worksheet reporter was started, even if no utPLSQL was installed in the database. In that case the user got the following error in the script output:

ORA-06550: line 1, column 57:
PLS-00201: identifier 'UT.RUN' must be declared
ORA-06550: line 1, column 57:
PL/SQL: Statement ignored
06550. 00000 -  "line %s, column %s:\n%s"
*Cause:    Usually a PL/SQL compilation error.
*Action:

This was good.

Starting with version 1.0.0 the realtime reporter is the default. When utPLSQL is not installed, nothing happens. Based on the statement log, it's clear that the ut.version statement fails.

I see two options for a user feedback.

a) an alert message informing the user, that utPLSQL is not installed
b) run the worksheet reporter as in previous versions and let the user figure out what the message means.
@PhilippSalvisberg PhilippSalvisberg added this to the v1.1.0 milestone Jul 11, 2019
@PhilippSalvisberg PhilippSalvisberg self-assigned this Jul 11, 2019
@PhilippSalvisberg
Copy link
Member Author

@PhilippSalvisberg PhilippSalvisberg commented Jul 13, 2019

I decided to go for option b). It's simpler to implement and it gives the user a feedback.
PhilippSalvisberg added a commit that referenced this issue Jul 13, 2019
@PhilippSalvisberg
Merge pull request #68 from PhilippSalvisberg/feature/issue_64_no_utp…
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
e8130c6 
…lsql

#64 - Improve error handling when no utPLSQL is installed
@PhilippSalvisberg PhilippSalvisberg mentioned this issue Jul 14, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PhilippSalvisberg PhilippSalvisberg

Labels
enhancement
Projects
None yet
Milestone
v1.1.0
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@PhilippSalvisberg
You can’t perform that action at this time.