<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>27930</bug_id>
          
          <creation_ts>2015-01-30 10:19:01 +0000</creation_ts>
          <short_desc>[fo31] Missing error condition for parse-json</short_desc>
          <delta_ts>2015-03-11 21:27:59 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>XPath / XQuery / XSLT</product>
          <component>Functions and Operators 3.1</component>
          <version>Candidate Recommendation</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Kay">mike</reporter>
          <assigned_to name="Michael Kay">mike</assigned_to>
          <cc>christian.gruen</cc>
          
          <qa_contact name="Mailing list for public feedback on specs from XSL and XML Query WGs">public-qt-comments</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>117619</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Kay">mike</who>
    <bug_when>2015-01-30 10:19:01 +0000</bug_when>
    <thetext>The spec of parse-json does not define an error code to raise if an option is present with an incorrect value, e.g. &apos;liberal&apos;:&apos;maybe&apos;. I propose to use FOJS0004.

The error code FOJS0002 appears in the appendix but is not used (the condition it describes is no longer an error).

The table of options is a little inconsistent in its presentation. For some options it uses a bold &quot;must&quot; to say what the value must be; for some options it is explicit about the default. I propose to use &quot;must&quot;, and describe the default, for all three options.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>117641</commentid>
    <comment_count>1</comment_count>
    <who name="Christian Gruen">christian.gruen</who>
    <bug_when>2015-01-31 18:45:41 +0000</bug_when>
    <thetext>&gt; I propose to use FOJS0004.

I would suggest to use a general code for invalid map arguments. In EXPath (and many of our own modules), we decided to raise the existing code FORG0006 whenever the type of an argument, or the key or value of a map argument, is invalid. FORG0006 is also raised by fn:boolean, fn:sum, and some other functions.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>117667</commentid>
    <comment_count>2</comment_count>
    <who name="Michael Kay">mike</who>
    <bug_when>2015-02-03 16:20:25 +0000</bug_when>
    <thetext>The WG agreed to fix the bug as suggested.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>117819</commentid>
    <comment_count>3</comment_count>
    <who name="Michael Kay">mike</who>
    <bug_when>2015-02-10 17:20:57 +0000</bug_when>
    <thetext>The change has been applied.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>118532</commentid>
    <comment_count>4</comment_count>
    <who name="Michael Kay">mike</who>
    <bug_when>2015-03-11 21:27:59 +0000</bug_when>
    <thetext>In my efforts to converge the specifications of the various JSON-related functions, this code changed to FOJS0005.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>