[ 
https://issues.apache.org/jira/browse/THRIFT-3293?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Henrique Mendonça resolved THRIFT-3293.
---------------------------------------
    Resolution: Fixed
      Assignee: Henrique Mendonça

Committed, thanks Håkon!

> JavaScript: null values turn into empty structs in constructor
> --------------------------------------------------------------
>
>                 Key: THRIFT-3293
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3293
>             Project: Thrift
>          Issue Type: Bug
>          Components: JavaScript - Compiler, Node.js - Compiler
>            Reporter: Håkon Hitland
>            Assignee: Henrique Mendonça
>             Fix For: 0.9.3
>
>         Attachments: thrift-3293-null-init.patch
>
>
> Given a struct definition with an optional struct field, e.g.
> {code}
> struct Foo {
>   1: Bar bar,
> }
> {code}
> Trying to initialize it to a null value like
> {code}
> new Foo({"bar": null});
> {code}
> will instead set bar to an empty Bar object.
> This happens because the constructor tests for undefined values, but not for 
> null.
> In addition to direct user calls, the internal copyList and copyMap functions 
> introduced for THRIFT-3122 now often trigger this by initializing one object 
> from another, making it hard to work around.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to