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

minor bug into monetdb_insert_id() (php_monetdb.php) #2937

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments
Closed

minor bug into monetdb_insert_id() (php_monetdb.php) #2937

monetdb-team opened this issue Nov 30, 2020 · 0 comments

Comments

@monetdb-team
Copy link

@monetdb-team monetdb-team commented Nov 30, 2020

Date: 2011-12-06 12:16:11 +0100
From: manuela <<m.sagona>>
To: clients devs <>
Version: 11.3.3 (Apr2011-SP1) [obsolete]

Last updated: 2012-01-26 15:32:03 +0100

Comment 16585

Date: 2011-12-06 12:16:11 +0100
From: manuela <<m.sagona>>

User-Agent: Mozilla/5.0 (Windows NT 5.1) AppleWebKit/535.2 (KHTML, like Gecko) Chrome/15.0.874.121 Safari/535.2
Build Identifier:

there is no method db_quote_ident()

Reproducible: Always

use monetdb_quote_ident() instead

Comment 16586

Date: 2011-12-06 12:17:22 +0100
From: manuela <<m.sagona>>

Created attachment 84
proposed patch

Attached file: php_monetdb.php_patch_quote_ident.txt (text/plain, 433 bytes)
Description: proposed patch

Comment 16593

Date: 2011-12-06 13:19:44 +0100
From: manuela <<m.sagona>>

Changeset cff56fc02d72 made by Manuela Sagona m.sagona@acsys.it in the MonetDB repo, refers to this bug.

For complete details, see http//devmonetdborg/hg/MonetDB?cmd=changeset;node=cff56fc02d72

Changeset description:

monetdb_insert_id: use existing monetdb_quote_ident

Replace db_quote_ident with monetdb_quote_ident.  Fixes bug #2937.

Comment 16594

Date: 2011-12-06 13:20:07 +0100
From: @grobian

thanks for your patch, pushed

Comment 16826

Date: 2012-01-26 15:32:03 +0100
From: @sjoerdmullender

The Dec2011 version has been release, so declaring this bug as FIXED.

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

Successfully merging a pull request may close this issue.

None yet
1 participant