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

MapiPP.pm unquote '0' as 'undef' #3235

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

MapiPP.pm unquote '0' as 'undef' #3235

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

Comments

@monetdb-team
Copy link

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

Date: 2013-02-15 18:46:48 +0100
From: Rémy Chibois <>
To: clients devs <>
Version: 11.15.1 (Feb2013)

Last updated: 2013-03-07 12:41:20 +0100

Comment 18480

Date: 2013-02-15 18:46:48 +0100
From: Rémy Chibois <>

User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_8_2) AppleWebKit/536.26.17 (KHTML, like Gecko) Version/6.0.2 Safari/536.26.17
Build Identifier:

while unquoting values received from the server, the unquote subroutine incorrectly returns 'undef' when the value is 0.

Reproducible: Always

Steps to Reproduce:

  1. sample Perl DBI script selecting from a table with some 0's
  2. print received data from fetchall_arrayref using Data::Dumper

Actual Results:

'undef' is printed instead of original '0'

Expected Results:

a true 0 is printed

I've included a small patch to fix this issue.

Comment 18481

Date: 2013-02-15 18:47:55 +0100
From: Rémy Chibois <>

Created attachment 179
Patch fixing the unquote issue when value is 0

Attached file: MapiPP-unquote.patch (text/plain, 596 bytes)
Description: Patch fixing the unquote issue when value is 0

Comment 18556

Date: 2013-02-25 12:35:29 +0100
From: @sjoerdmullender

Changeset 37bd050c9cda made by Sjoerd Mullender sjoerd@acm.org in the MonetDB repo, refers to this bug.

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

Changeset description:

Don't return "undef" if 0.
This fixes bug #3235.
Thanks to Rémy Chibois for the report and the fix.

Comment 18563

Date: 2013-02-25 15:27:01 +0100
From: @sjoerdmullender

Changeset d222c1c35635 made by Sjoerd Mullender sjoerd@acm.org in the MonetDB repo, refers to this bug.

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

Changeset description:

Added test for bug #3235.

Comment 18586

Date: 2013-03-07 12:41:20 +0100
From: @sjoerdmullender

Feb2013-SP1 has been released.

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