www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 11527] New: JSONValue cannot set values through named fields

reply d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11527

           Summary: JSONValue cannot set values through named fields
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: regression
          Priority: P2
         Component: Phobos
        AssignedTo: nobody puremagic.com
        ReportedBy: k.hara.pg gmail.com


--- Comment #0 from Kenji Hara <k.hara.pg gmail.com> 2013-11-16 02:31:46 PST ---
Following test cases will throw JSONException in runtime:

import std.json;
void main()
{
    JSONValue v;
    v.integer = 1;
}

Output:

std.json.JSONException std\json.d(91): JSONValue is not an integer

This is caused by:
https://github.com/D-Programming-Language/phobos/pull/1421

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Nov 16 2013
next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11527



--- Comment #1 from Kenji Hara <k.hara.pg gmail.com> 2013-11-16 02:32:27 PST ---
This is git-head (2.065a) regression.

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Nov 16 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11527


Kenji Hara <k.hara.pg gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull, rejects-valid


--- Comment #2 from Kenji Hara <k.hara.pg gmail.com> 2013-11-16 02:41:33 PST ---
https://github.com/D-Programming-Language/phobos/pull/1693

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Nov 16 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11527


John Colvin <john.loughran.colvin gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |john.loughran.colvin gmail.
                   |                            |com


--- Comment #3 from John Colvin <john.loughran.colvin gmail.com> 2013-11-16
12:43:18 GMT ---
I thought this was by design as part of the new API? One of the goals was to
add type-safety.

auto v = JSONValue(1);

and

JSONValue v;
v.type = JSON_TYPE.INTEGER;
v.integer = 1;

are both valid.

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Nov 16 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11527



--- Comment #4 from github-bugzilla puremagic.com 2013-11-20 09:50:30 PST ---
Commits pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/5440ea8fbce0cb1147713c2538621f3cf180cd79
fix Issue 11527 - JSONValue cannot set values through named fields

https://github.com/D-Programming-Language/phobos/commit/dc4bd532d03a93a4957d3f968a059bfe704be463
Merge pull request #1693 from 9rnsr/fix11527

[REG2.065a] Issue 11527 - JSONValue cannot set values through named fields

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Nov 20 2013
prev sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11527


Andrej Mitrovic <andrej.mitrovich gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |andrej.mitrovich gmail.com
         Resolution|                            |FIXED


-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Nov 20 2013