Gldapo

Typemapping for Boolean type

Details

  • Type: New Feature New Feature
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 0.7
  • Fix Version/s: 0.7
  • Component/s: API
  • Labels:
    None
  • Testcase included:
    yes
  • Patch Submitted:
    Yes
  • Number of attachments :
    2

Description

Here's a few lines of code that adds a Boolean Type Map to the default type mappings and adds relevant test cases.

Activity

Hide
Luke Daley added a comment -

Implemented. Thanks for the patch!

Show
Luke Daley added a comment - Implemented. Thanks for the patch!
Hide
Joel Reed added a comment -

When I first submitted this patch I tested the mapToBoolean but just copied and pasted the mapFrom function and didn't really see what it was used for in the documentation.

Later on, I tried creating a new entry and realized the mapFrom version needed to uppercase the resultant TRUE or FALSE.

See:

http://www.alvestrand.no/objectid/1.3.6.1.4.1.1466.115.121.1.7.html

The attached patch is incremental and fixes this issue.

Show
Joel Reed added a comment - When I first submitted this patch I tested the mapToBoolean but just copied and pasted the mapFrom function and didn't really see what it was used for in the documentation. Later on, I tried creating a new entry and realized the mapFrom version needed to uppercase the resultant TRUE or FALSE. See: http://www.alvestrand.no/objectid/1.3.6.1.4.1.1466.115.121.1.7.html The attached patch is incremental and fixes this issue.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: