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

Preserve comments for parameters after altering procedures [CORE3140] #3517

Closed
firebird-automations opened this issue Sep 18, 2010 · 11 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: Vladimir Arkhipov (arkinform)

Is duplicated by CORE3528

Now if I alter procedure (even with the same parameters list) then all comments become empty again. It is inconvenient. I think that new comments should be assigned from old comments by name if existed before.

Commits: 6a46929 17c249f 097426f

@firebird-automations
Copy link
Collaborator Author

Modified by: Vladimir Arkhipov (arkinform)

description: Now if I alter procedure (even with the same parameters list) then all comments become empty again. It is inconvenient. I think that new comments should be assigned from old comments by name if existed before. Similar for views. => Now if I alter procedure (even with the same parameters list) then all comments become empty again. It is inconvenient. I think that new comments should be assigned from old comments by name if existed before.

summary: Preserve comments for parameters after altering procedures and views => Preserve comments for parameters after altering procedures

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

assignee: Adriano dos Santos Fernandes [ asfernandes ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 3.0 Alpha 1 [ 10331 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

Version: 3.0 Initial [ 10301 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pcisar

status: Resolved [ 5 ] => Closed [ 6 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

Link: This issue is duplicated by CORE3528 [ CORE3528 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

Version: 2.5.0 [ 10221 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

Fix Version: 2.5.1 [ 10333 ]

@firebird-automations
Copy link
Collaborator Author

Commented by: Vladimir Arkhipov (arkinform)

Firebird 2.5.2:
Does not work for output parameters. For input parameters description is preserved.

@firebird-automations
Copy link
Collaborator Author

Modified by: Vladimir Arkhipov (arkinform)

Version: 3.0 Alpha 1 [ 10331 ]

Version: 2.5.2 Update 1 [ 10521 ]

Version: 2.1.5 Update 1 [ 10522 ]

Version: 2.5.2 [ 10450 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Closed [ 6 ] => Closed [ 6 ]

QA Status: Done successfully

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment