Skip to content

[Sync EN] Major rewrite of the setcookie reference page#252

Merged
msamesch merged 8 commits into
php:masterfrom
lacatoire:sync-en/setcookie
Jun 12, 2026
Merged

[Sync EN] Major rewrite of the setcookie reference page#252
msamesch merged 8 commits into
php:masterfrom
lacatoire:sync-en/setcookie

Conversation

@lacatoire

@lacatoire lacatoire commented May 29, 2026

Copy link
Copy Markdown
Member

Sync reference/network/functions/setcookie.xml with upstream 6122a8317c. Most inline <para> switched to <simpara>, prose moved to impersonal style, new errors section (E_WARNING before PHP 8.0.0, ValueError as of 8.0.0 for unrecognised options keys) and the matching changelog row, plus the samesite="None" requires secure=true note. Examples and common pitfalls refreshed. Maintainer sammywg and Reviewed: yes kept.

Brings reference/network/functions/setcookie.xml up to date with
the upstream rewrite (EN-Revision 6122a8317ca0068fc71f6a5167e0a2d99166ec7c):

- Most inline <para> blocks converted to <simpara> (mirror EN).
- Description and notes rephrased to impersonal/factual style
  ("in the script", "this function is called") instead of the
  old "your script"/"you place calls to" phrasing.
- options parameter: new <note> about samesite="None" requiring
  secure=true, ValueError from PHP 8.0.0 onward.
- New <refsect1 role="errors"> section: E_WARNING before PHP 8.0.0,
  ValueError as of PHP 8.0.0 for unknown options.
- Changelog: new 8.0.0 row, structural simpara cleanups.
- Examples reworked: new intro about browser developer tools
  (Storage/Application tab), German code comments in the send
  example, two short <simpara>s for the delete example, refreshed
  Array von Cookies wording.
- Common Pitfalls bullet 4 updated: browser stores separate cookies
  for each path/domain, json_encode/json_decode instead of explode.
- Output-buffering note rewritten in impersonal form.
- See-also block: wrapping <para> dropped.
- Obsolete <!-- $Revision$ --> SVN marker dropped.

Existing Maintainer (sammywg) and Reviewed: yes / Rev-Revision kept.
Comment thread reference/network/functions/setcookie.xml Outdated
Comment thread reference/network/functions/setcookie.xml Outdated
Comment thread reference/network/functions/setcookie.xml Outdated
Comment thread reference/network/functions/setcookie.xml Outdated
Comment thread reference/network/functions/setcookie.xml Outdated
Comment thread reference/network/functions/setcookie.xml Outdated
Comment thread reference/network/functions/setcookie.xml Outdated

@msamesch msamesch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks a lot, looks good to me! Just a bit of nitpicking..

lacatoire and others added 7 commits June 12, 2026 17:51
Co-authored-by: Martin Samesch <samesch@php.net>
Co-authored-by: Martin Samesch <samesch@php.net>
Co-authored-by: Martin Samesch <samesch@php.net>
Co-authored-by: Martin Samesch <samesch@php.net>
Co-authored-by: Martin Samesch <samesch@php.net>
Co-authored-by: Martin Samesch <samesch@php.net>
Co-authored-by: Martin Samesch <samesch@php.net>
@msamesch msamesch merged commit f4f88c8 into php:master Jun 12, 2026
1 check passed
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.

2 participants