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

[PHP 8.4] fix php/doc-en#4063 (PHP 8.4: trigger_error() updates) #191

Merged
merged 4 commits into from
Nov 25, 2024

Conversation

o0h
Copy link
Contributor

@o0h o0h commented Nov 24, 2024

Issue

Refs

概ね、移行ガイドのdeprecate.xmlの内容を踏まえました。

<simpara>
<function>trigger_error</function> で
<parameter>error_level</parameter> として
<constant>E_USER_ERROR</constant> を指定するのは非推奨となりました。
</simpara>

オリジナル(en)では、

  • migration: Calling trigger_error with error_level being equal to E_USER_ERROR is now deprecated.
  • reference: Passing E_USER_ERROR as the error_level is now deprecated.

であり、やや表現が異なる箇所ですが、強調すべき内容的な差異は無いだろうという判断です。

@o0h o0h force-pushed the trigger-error-8.4 branch from 63367eb to 899361f Compare November 24, 2024 06:28
@@ -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>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

他の箇所でも「アトリビュート」「属性」の訳語が揺れていそうで、一旦「アトリビュート」にしています。
migrationの表記は以下

<simpara>
新しい <classname>Deprecated</classname> 属性を使用して、関数、メソッド、
およびクラス定数を非推奨としてマークできます。このアトリビュートで非推奨となった機能の動作は、
PHP 自身が提供する既存の非推奨メカニズムの動作と一致します。
唯一の例外は、発生するエラーコードが
<constant>E_DEPRECATED</constant> ではなく <constant>E_USER_DEPRECATED</constant> であることです。
</simpara>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

preview:
image

Copy link
Collaborator

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 -->
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

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>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

preview:
image

Copy link
Collaborator

Choose a reason for hiding this comment

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

訳はLGTMです。位置なのですが、この <warning> セクションは:

原文: <para>The designated error type ...</para>
訳文: <para>このエラーに割り当てられた...</para>

この要素のすぐ下に追加されているようです。

文言追加位置

戻り値の型が、<type>bool</type> ではなく
<type>true</type> に変更されました。
</entry>
</row>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

preview:
image

}
$hash = password_hash($password, PASSWORD_DEFAULT);
?>
]]>
</programlisting>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

preview:
image

@o0h o0h changed the title [WIP] [PHP 8.4] fix php/doc-en#4063 (PHP 8.4: trigger_error() updates) [PHP 8.4] fix php/doc-en#4063 (PHP 8.4: trigger_error() updates) Nov 24, 2024
@o0h o0h marked this pull request as ready for review November 24, 2024 15:44
@o0h
Copy link
Contributor Author

o0h commented Nov 24, 2024

表示を確認するの面倒くさいかな?と思ってビルド結果から変更箇所のスクショを貼ってみましたが、
doc(xml)のレビューをする際には邪魔だったかも知れません。適宜、削除していただいて大丈夫です 👌

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.

ありがとうございます!役はLGTMです。
xmlの構造のみ確認をお願いします。

やや表現が異なる箇所ですが、強調すべき内容的な差異は無いだろうという判断です。

はい、問題ないと思います、むしろありがとうございます!
今回くらいの言い換えは、他の場所でも比較的日常的に行われてます。
(それをやらないと日本語として逆におかしい、というような箇所なども)

かわりに、<exceptionname>Exception</exceptionname> をスローするか、
<function>exit</function> を呼び出してください。
</simpara>
</warning>
Copy link
Collaborator

Choose a reason for hiding this comment

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

訳はLGTMです。位置なのですが、この <warning> セクションは:

原文: <para>The designated error type ...</para>
訳文: <para>このエラーに割り当てられた...</para>

この要素のすぐ下に追加されているようです。

文言追加位置

@@ -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 -->
Copy link
Collaborator

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>
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

ありがとうございます!私も アトリビュート が良いと思います。

普通名詞の意味の「属性」ではなく、固有シンタックスを指していることが明確にわかりますね。
(8.4から離れてしまいますが、見つけた箇所はプルリク作成いただいて大丈夫です。私も見つけ次第修正していきます。)

@o0h
Copy link
Contributor Author

o0h commented Nov 25, 2024

@KentarouTakeda
失礼しました・・!
修正して、表示位置がこんな感じになります (php.net に出ているenとの比較)
image

@o0h o0h requested a review from KentarouTakeda November 25, 2024 12:10
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 2d5c271 into php:master Nov 25, 2024
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