<?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>28357</bug_id>
          
          <creation_ts>2015-03-28 12:36:13 +0000</creation_ts>
          <short_desc>[XSLT30] (editorial) Some remnants in draft about visibility attribute of xsl:param, which is now gone</short_desc>
          <delta_ts>2015-10-29 09:50:33 +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>XSLT 3.0</component>
          <version>Member-only Editors Drafts</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows NT</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>trivial</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Abel Braaksma">abel.braaksma</reporter>
          <assigned_to name="Michael Kay">mike</assigned_to>
          
          
          <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>119004</commentid>
    <comment_count>0</comment_count>
    <who name="Abel Braaksma">abel.braaksma</who>
    <bug_when>2015-03-28 12:36:13 +0000</bug_when>
    <thetext>In Bug 26740 we decided to remove the xsl:param/@visibility attribute. This is reflected in most places, but there is one place that still refers to this.

2.3.2 Priming a stylesheet
The first Note mentions parameters with visibility:

&lt;quote&gt;
Non-static stylesheet parameters must be public, final, or abstract, which ensures that all the parameters in the stylesheet for which values can be supplied externally have distinct names. Static parameters, by contrast, can have names that are duplicated across different packages.
&lt;/quote&gt;

I think that Note is now redundant, or should be changed to &quot;non-static stylesheet parameters are implicitly public, which ensures [...]&quot;

The part on static parameters seems vague. Names can be duplicated, but the parameters cannot. One static param in one package is not visible in another package, even if they have the same name.

I did not find other mentions.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>119005</commentid>
    <comment_count>1</comment_count>
    <who name="Abel Braaksma">abel.braaksma</who>
    <bug_when>2015-03-28 14:29:19 +0000</bug_when>
    <thetext>There&apos;s another spot, though not necessarily an error. In 3.6.3.3 Overriding Components from a Used Package:

&lt;quote&gt;
The visibility of the overriding component in the using package is defined by the visibility attribute appearing on the overriding declaration.
&lt;/quote&gt;

which is not (entirely) true for xsl:param, in which case there is no such attribute. I propose:

    The visibility of the overriding component in the using package 
    is defined by the visibility attribute appearing on the overriding
    declaration, or implicitly public in the case of xsl:param.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>119598</commentid>
    <comment_count>2</comment_count>
    <who name="Michael Kay">mike</who>
    <bug_when>2015-04-16 22:39:39 +0000</bug_when>
    <thetext>Fixed as suggested.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>