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-issue-importer opened this issue Sep 18, 2010 · 11 comments
Closed

Comments

@firebird-issue-importer

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-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Sep 19, 2010

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-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Oct 28, 2010

Modified by: @asfernandes

assignee: Adriano dos Santos Fernandes [ asfernandes ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Oct 29, 2010

Modified by: @asfernandes

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

resolution: Fixed [ 1 ]

Fix Version: 3.0 Alpha 1 [ 10331 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Oct 29, 2010

Modified by: @asfernandes

Version: 3.0 Initial [ 10301 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Feb 14, 2011

Modified by: @pcisar

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

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jun 18, 2011

Modified by: @asfernandes

Link: This issue is duplicated by CORE3528 [ CORE3528 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jun 18, 2011

Modified by: @asfernandes

Version: 2.5.0 [ 10221 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jun 18, 2011

Modified by: @asfernandes

Fix Version: 2.5.1 [ 10333 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Sep 2, 2013

Commented by: Vladimir Arkhipov (arkinform)

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

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Sep 2, 2013

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-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented May 29, 2015

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