Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unserialize() の更新を取り込み #247

Merged
merged 3 commits into from
Jan 13, 2025

Conversation

nsfisis
Copy link
Contributor

@nsfisis nsfisis commented Jan 13, 2025

refs #150

変更点

unserialize() 関数のマニュアルに現在までに出された変更をすべて取り込みました。

https://github.com/php/doc-en/commits/master/reference/var/functions/unserialize.xml

各コミットが各 PR に対応しています (除 4328)。

備考

TypeErrorValueError をスローするようになりました。

について、同時に複数種類の例外をスローするはずはないので意味を取るために php-src を調べたところ、 $options['allowed_classes'] の各要素が string でないときに TypeError を、その文字列がクラス名として不正なとき (\n が含まれている等) に ValueError をスローするという動作のようです。

これを考えると、

TypeErrorValueError をスローするようになりました。

のように説明したほうがよいとは思うのですが、どちらかというと "and" でなく "or" を使うように原文側を直すべきだと思ったので、この PR ではそのまま「TypeErrorValueError」と訳すことにしました。

Copy link
Collaborator

@KentarouTakeda KentarouTakeda left a comment

Choose a reason for hiding this comment

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

ありがとうございます!

@KentarouTakeda KentarouTakeda merged commit 26cafd6 into php:master Jan 13, 2025
2 checks passed
@nsfisis nsfisis deleted the feature/unserialize branch January 13, 2025 13:46
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