-
Notifications
You must be signed in to change notification settings - Fork 68
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
[PHP 8.4] fix php/doc-en#4063 (PHP 8.4: trigger_error() updates) #191
Conversation
63367eb
to
899361f
Compare
@@ -129,6 +155,7 @@ if ($divisor == 0) { | |||
<member><function>set_error_handler</function></member> | |||
<member><function>restore_error_handler</function></member> | |||
<member><link linkend="errorfunc.constants">エラーレベル定数</link></member> | |||
<member><classname>Deprecated</classname> アトリビュート</member> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
他の箇所でも「アトリビュート」「属性」の訳語が揺れていそうで、一旦「アトリビュート」にしています。
migrationの表記は以下
doc-ja/appendices/migration84/new-features.xml
Lines 130 to 136 in c378f9b
<simpara> | |
新しい <classname>Deprecated</classname> 属性を使用して、関数、メソッド、 | |
およびクラス定数を非推奨としてマークできます。このアトリビュートで非推奨となった機能の動作は、 | |
PHP 自身が提供する既存の非推奨メカニズムの動作と一致します。 | |
唯一の例外は、発生するエラーコードが | |
<constant>E_DEPRECATED</constant> ではなく <constant>E_USER_DEPRECATED</constant> であることです。 | |
</simpara> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
ありがとうございます!私も アトリビュート が良いと思います。
普通名詞の意味の「属性」ではなく、固有シンタックスを指していることが明確にわかりますね。
(8.4から離れてしまいますが、見つけた箇所はプルリク作成いただいて大丈夫です。私も見つけ次第修正していきます。)
@@ -1,6 +1,6 @@ | |||
<?xml version="1.0" encoding="utf-8"?> | |||
<!-- $Revision$ --> | |||
<!-- EN-Revision: 210d382b5b139444fe2e16a5a3211076a65ff2ba Maintainer: hirokawa Status: ready --> | |||
<!-- EN-Revision: 72b70d7c3c3b2b87423641906da2db407c32c3c3 Maintainer: hirokawa Status: ready --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
かわりに、<exceptionname>Exception</exceptionname> をスローするか、 | ||
<function>exit</function> を呼び出してください。 | ||
</simpara> | ||
</warning> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
戻り値の型が、<type>bool</type> ではなく | ||
<type>true</type> に変更されました。 | ||
</entry> | ||
</row> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} | ||
$hash = password_hash($password, PASSWORD_DEFAULT); | ||
?> | ||
]]> | ||
</programlisting> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
表示を確認するの面倒くさいかな?と思ってビルド結果から変更箇所のスクショを貼ってみましたが、 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます!役はLGTMです。
xmlの構造のみ確認をお願いします。
やや表現が異なる箇所ですが、強調すべき内容的な差異は無いだろうという判断です。
はい、問題ないと思います、むしろありがとうございます!
今回くらいの言い換えは、他の場所でも比較的日常的に行われてます。
(それをやらないと日本語として逆におかしい、というような箇所なども)
かわりに、<exceptionname>Exception</exceptionname> をスローするか、 | ||
<function>exit</function> を呼び出してください。 | ||
</simpara> | ||
</warning> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1,6 +1,6 @@ | |||
<?xml version="1.0" encoding="utf-8"?> | |||
<!-- $Revision$ --> | |||
<!-- EN-Revision: 210d382b5b139444fe2e16a5a3211076a65ff2ba Maintainer: hirokawa Status: ready --> | |||
<!-- EN-Revision: 72b70d7c3c3b2b87423641906da2db407c32c3c3 Maintainer: hirokawa Status: ready --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -129,6 +155,7 @@ if ($divisor == 0) { | |||
<member><function>set_error_handler</function></member> | |||
<member><function>restore_error_handler</function></member> | |||
<member><link linkend="errorfunc.constants">エラーレベル定数</link></member> | |||
<member><classname>Deprecated</classname> アトリビュート</member> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
ありがとうございます!私も アトリビュート が良いと思います。
普通名詞の意味の「属性」ではなく、固有シンタックスを指していることが明確にわかりますね。
(8.4から離れてしまいますが、見つけた箇所はプルリク作成いただいて大丈夫です。私も見つけ次第修正していきます。)
@KentarouTakeda |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます!お手数おかけしました!
Issue
Refs
概ね、移行ガイドのdeprecate.xmlの内容を踏まえました。
doc-ja/appendices/migration84/deprecated.xml
Lines 148 to 152 in c378f9b
オリジナル(en)では、
であり、やや表現が異なる箇所ですが、強調すべき内容的な差異は無いだろうという判断です。