FAQ
Edit report at https://pear.php.net/bugs/bug.php?id=19187&edit=1

ID: 19187
Updated by: danielc@analysisandsolutions.com
Reported By: alan at akbkhome dot com
Summary: add DB_PORTABILITY_BOOLEAN for postgres
-Status: Open
+Status: Feedback
Type: Feature/Change Request
Package: DB
Operating System: linux
Package Version: 1.7.14
PHP Version: HEAD SVN-2012-01-04
Roadmap Versions:
New Comment:

-Status: Open
+Status: Feedback
Hi Alan:

Good idea. There are some additional things the patch needs to cover:

1) All drivers need to be supported. See tests/driver/15quote.phpt for
a list of data types.
2) Unit tests are a must.
3) Document the mode in the DB_common::setOption() docblock.
4) Document the code in the manual's DB_common::setOption() section.


Previous Comments:
------------------------------------------------------------------------

[2012-01-04 11:31:54] alan_k

Added #patch
bug:19187;patch:boolean.portability.diff;revision:1325676714;.

------------------------------------------------------------------------

[2012-01-04 11:31:24] alan_k

Description:
------------
classic postgres returning 't' or 'f' for booleans

This patch munges the values to true|false - not particularly
efficient, but works...

------------------------------------------------------------------------

Search Discussions

  • Danielc at Jan 8, 2012 at 2:32 pm
    Edit report at https://pear.php.net/bugs/bug.php?id=19187&edit=1

    ID: 19187
    Updated by: danielc@analysisandsolutions.com
    Reported By: alan at akbkhome dot com
    Summary: add DB_PORTABILITY_BOOLEAN for postgres
    Status: Feedback
    Type: Feature/Change Request
    Package: DB
    Operating System: linux
    Package Version: 1.7.14
    PHP Version: HEAD SVN-2012-01-04
    Roadmap Versions:
    New Comment:

    Oh, on second thought, this only needs to be implemented for fbsql and
    pgsql. For more info, see:
    http://www.analysisandsolutions.com/presentations/portability/slides/boolean.htm


    Previous Comments:
    ------------------------------------------------------------------------

    [2012-01-08 03:24:15] danielc

    -Status: Open
    +Status: Feedback
    Hi Alan:

    Good idea. There are some additional things the patch needs to cover:

    1) All drivers need to be supported. See tests/driver/15quote.phpt for
    a list of data types.
    2) Unit tests are a must.
    3) Document the mode in the DB_common::setOption() docblock.
    4) Document the code in the manual's DB_common::setOption() section.

    ------------------------------------------------------------------------

    [2012-01-04 11:31:54] alan_k

    Added #patch
    bug:19187;patch:boolean.portability.diff;revision:1325676714;.

    ------------------------------------------------------------------------

    [2012-01-04 11:31:24] alan_k

    Description:
    ------------
    classic postgres returning 't' or 'f' for booleans

    This patch munges the values to true|false - not particularly
    efficient, but works...

    ------------------------------------------------------------------------

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
grouppear-bugs @
categoriesphp
postedJan 8, '12 at 2:23a
activeJan 8, '12 at 2:32p
posts2
users1
websitepear.php.net

1 user in discussion

Danielc: 2 posts

People

Translate

site design / logo © 2022 Grokbase