Skip to content

Updated possible values for display errors INI configurations#5486

Open
Narkunan wants to merge 6 commits into
php:masterfrom
Narkunan:display_errors-only-documents-2-values
Open

Updated possible values for display errors INI configurations#5486
Narkunan wants to merge 6 commits into
php:masterfrom
Narkunan:display_errors-only-documents-2-values

Conversation

@Narkunan
Copy link
Copy Markdown
Contributor

@Narkunan Narkunan commented Apr 11, 2026

Fix issue #2525

Comment thread reference/errorfunc/ini.xml
Comment thread reference/errorfunc/ini.xml Outdated
Comment thread reference/errorfunc/ini.xml Outdated
@Narkunan
Copy link
Copy Markdown
Contributor Author

@lacatoire Updated PR please review it let me know if any changes needed

Comment thread reference/errorfunc/ini.xml Outdated
Comment thread reference/errorfunc/ini.xml Outdated
Comment thread reference/errorfunc/ini.xml Outdated
@Narkunan
Copy link
Copy Markdown
Contributor Author

@lacatoire Done made all suggested changes

Comment on lines +211 to +213
<simpara>
Possible Values:
</simpara>
Copy link
Copy Markdown
Member

@jordikroon jordikroon May 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This block is redundant with the line below. Nevertheless given the structure I would remove this completely.

</note>
<simpara>
The bundled <literal>php.ini-development</literal> sets this to
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should go above the note.

<para>
Value <literal>"stderr"</literal> sends the errors to <literal>stderr</literal>
instead of <literal>stdout</literal>.
The accepted values are:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you take inspiration from syslog.filter? In other words it is using <itemizedlist>.

<varlistentry xml:id="ini.display-errors">
 <term>
  <parameter>display_errors</parameter>
  <type>string</type>
 </term>
 <listitem>
  <para>
   This determines whether errors should be printed to the screen
   as part of the output or if they should be hidden from the user.
   <itemizedlist>
    <listitem>
     <simpara>
      <literal>Off</literal> - do not display any errors.
     </simpara>
    </listitem>
    <listitem>
     <simpara>
      <literal>On</literal> or <literal>stdout</literal> - display errors to <literal>stdout</literal>. This is the default.
     </simpara>
    </listitem>
    <listitem>
     <simpara>
      <literal>stderr</literal> - display errors to <literal>stderr</literal>. Only effective for the <literal>CLI,</literal><literal>phpdbg</literal>and<literal>CGI</literal> SAPIs.
     </simpara>
    </listitem>
   </itemizedlist>
  </para>
 </listitem>
</varlistentry>

@jordikroon
Copy link
Copy Markdown
Member

Sorry this took so long to review. Overall looks good, just some structural/style comments to make it more in consistent.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants