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

Error in UPDATE OR INSERT syntax, 2.1 Rlsnotes [DOC35] #44

Closed
firebird-automations opened this issue Jan 11, 2009 · 9 comments
Closed

Comments

@firebird-automations
Copy link

Submitted by: @paulvink

Attachments:
screenshot-1.jpg

In the 2.1 Release Notes, the MATCHING clause of the UPDATE OR INSERT statement is given as:

[MATCHING <column_list>]

This should be:

[MATCHING (<column_list>)]

README.update_or_insert.txt contains the same error (which explains the error in the Notes)

@firebird-automations
Copy link
Author

Modified by: @paulvink

assignee: Paul Vinkenoog [ paulvink ] => Helen Borrie [ helebor ]

@firebird-automations
Copy link
Author

Modified by: Rishi Karott (rishikarott)

Attachment: screenshot-1.jpg [ 11730 ]

@firebird-automations
Copy link
Author

Commented by: @helebor

I don't know which version of release notes is referred to here. Syntax is correct in v.0213_14 dated 4 Feb 2010. I don't have anything local that's earlier than that, to figure out when the correction was done.

Paul V, if README.update_or_insert.txt hasn't been corrected, it's Adriano you'll need to contact. I doubt he ever looks at DOC tickets!

@firebird-automations
Copy link
Author

Commented by: @paulvink

My report was based on the 2.1.1 notes (15 Jul 2008).
The error was still there in 2.1.2 RC1 (31 Jan 2009, doc.v. 0212_02) but fixed in RC2 (25 Feb 2009, doc.v. 0212_04), so you can close this issue.

In the README, the error is still present in 2.1.3 final; I'll let Adriano know.

@firebird-automations
Copy link
Author

Commented by: @helebor

Fixed prior to v.2.1.2 release; keep open until Adriano has fixed the Readme.

@firebird-automations
Copy link
Author

Modified by: @helebor

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

resolution: Fixed [ 1 ]

@firebird-automations
Copy link
Author

Commented by: @paulvink

Adriano just emailed me that he fixed the Readme.
But he pointed out that the RETURNING clause has an error too. It should be:

\[RETURNING <value\_list\> \[INTO <variable\_list\>\]\]

(value_list instead of column_list)

@firebird-automations
Copy link
Author

Commented by: @helebor

Ugggh, I wonder what this <value_list> looks like...

@firebird-automations
Copy link
Author

Commented by: @paulvink

...and the same correction should be made (several times) in section "The RETURNING Clause" (2.1.x Notes)
and section "RETURNING Clause for Insert Statements" (2.0.x Notes).
It's always a value list; you can enter all kinds of expressions here, not just column names.

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

No branches or pull requests

2 participants