今週の PHP 2022/10/15 〜 2022/10/21
PHP のメーリングリストから、気になった情報をピックアップします。
Internal
[RFC] Destructuring Coalesce
destructuring assignments (分割代入）
-> list などで、配列を一挙に変数にアサインするときに、初期値を指定できるようにしようという提案です。
<?php
[$a ?? "default value"] = $array;
list($a ?? "default value") = $array;
PHP のイシューにおいても、何度か要望が上がっていた機能です。
仕事でこういうコードを書くか...は、わからないですが、プライベートとか、自作ライブラリーとかだと実にほしいなと思います。
Casting array to any class
いわゆる DTO (Data Transfer Object) と呼ばれるものが、よく使われます。
この提案は、配列をクラス名でキャストすることで、DTO をかんたんに作れるようにしようというものです。
実装としては、 _set_state() を利用する形をとったらどうだろう？といっています。
コード例で書くと、こんな感じに連想配列をクラスのオブジェクトに変換できるようにしたい。と
class Fuga {
public function __construct(
public readonly int $x,
public readonly int $y,
){}
public static function _set_state(array $array)
{
$fuga = new Fuga();
$fuga->x = $array['x'];
$fuga->y = $array['y'];
}
}
$arr = ['x' => 4, 'y' => 5];
$fuga = (Fuga)$arr;
_set_state は var_export を使った際に、オブジェクトを復元するために使われるマジックメソッドです。
それを、キャストにも流用してしまおうということです。
これに対して、名前付き引数とスプレッド構文を使えば、近しいことを今でもできるという返信
class Fuga {
public function __construct(
public readonly int $x,
public readonly int $y,
){}
}
$arr = ['x' => 4, 'y' => 3];
$fuga = new Fuga(...$arr);
Should the engine be able to pass variables by reference of userland functions - Externals
[VOTE] Improve unserialize() error handling - Externals
unserialize の例外発生時の挙動改善の RFC ですが、一番メインになっている 例外の共通化については、雲行きが怪しそう。
Adding the OpenSSF Scorecards GitHub Action - Externals
OpenSSF は、セキュリティ系のチェックツールらしいのですが、それのスコアを GitHub Action で集計しようみたいな相談。
すでに、実行自体はされているようで、現状のスコアは見れます。
現在のスコアは 5.3 とのこと。
Microseconds to error log - Externals
とても、負荷の高いアプリケーションにおいて、エラーログにマイクロセコンドを出したいという相談。
現状のソースコード内では、秒に狭められてしまっているという
まあ、Monolog みたいなユーザーランドの解決策をつかってしまうな...自分は。
NULL Coercion Consistency - Externals
関数引数に、値の定まっていない変数を入れた際に、エラーになる挙動、ならない挙動が一貫していないため、一貫させようという一大事業です。
たとえば、htmlspecialchars は 8.1 以降で非推奨ですが、sprintf では null を渡しても問題なかったりします。
Compact can't resolve outer scoped variables using short closures - Externals
$x = 123;
(fn() => compact('x'))();
compact が short closure 使ったときに、外のスコープを見れないという問題。
既知の問題ではあるようです。
compact 自体が非推奨になればいいのにという
Proposal to incrementally improve timeout and signal handling - Externals
PHP の Timeout や Signal ハンドリング関連の改善提案です。
この RFC を準備している dunglas さんは、FrankenPHP の作者です。
Go で PHP を embed していく過程で、様々なハマりどころがあったということで、ついに RFC を出して PHP 自体の改善提案！！
大まかにまとめると
- timeout の実装は ZTS に対応できていない
- Zend Signals はマルチスレッド環境で Seg V とかいろんな問題がでる
この２つを改善する提案です。
FrankenPHP はシェアードナッシングモデルの PHP を維持しつつ高速化を体現するために SAPI として実装された気鋭の勢力です。
これから要注目の SAPI にまつわる提案ですので、こちらも要チェック。
Bugs
opcache: add FrankenPHP to the allow list by dunglas · Pull Request #9755 · php/php-src
FrankenPHP っていう SAPI があるらしい。
Add destructuring coalesce by bwoebi · Pull Request #9747 · php/php-src
ディストラクティング合体演算子？？
新しいRFC用 の PoC です。
キーが存在しないときのデフォルト値を設定するという仕様追加です。いまいち、用途がわからない...
Segmentation fault on PDO::sqliteCreateFunction with SQLite driver on PHP 8.2.0RC4 · Issue #9749 · php/php-src
SQLite driver のセグフォールト案件です。 PHP 8.2.0 RC4 でも出てましたが、修正されたようです。
ついでに xhprof への不具合報告もされていたようで、素晴らしいです。
Generator crashes when interrupted during a function call with extra named params · Issue #9752 · php/php-src
Generator に余分な名前付き引数を渡して停止させると、クラッシュするというイシュー
8.0系向けにもPRが飛んでいるようです。
潜在バグだったのですね。
修正内容は、たったの一行という。
Mysqli converts TINYINT to a string after SUM() if the result of SUM is outside TINYINT, even if MYSQLI_OPT_INT_AND_FLOAT_NATIVE is set to true. · Issue #9665 · php/php-src
以前にも紹介したこちらの不具合報告。 SUM をすると型が変わってしまうというものでしたが、
sum and avg is MYSQL_TYPE_NEWDECIMAL.
ということで、MySQL側の挙動に合わせるのであれば、文字列型で受けないと精度が変わってしまいますねということでした。
この挙動は、想定どおりなので、修正は無しでクローズになりました。
copy() target is blank in PHP 8.2.0-rc3 · Issue #9653 · php/php-src
VirtualBox の shared folder において、 copy でファイルが見つからない問題。
似たような事象に NFS マウントで遭遇したことがあるので、要チェックしてました。
修正を見てみると
copy_file_range というシステムコールをサポートしているかどうか？のようでした。
kernel 5.3 以降におけるファイルシステムまたぎのファイルコピーに対応させたということのようです。
これ、結構問題になりそうな気がしました。
一見して、理由がわからないやつ。
Duplicates returned by array_unique when using enums · Issue #9775 · php/php-src
ENUM を格納した配列にたいして array_unique しても unique にならないという報告
不具合として確認されました。
原因は、ENUMが比較できないからだというコメント
var_dump(Test::COURSES_ADMIN <=> Test::COURSES_ADMIN); // 0
var_dump(Test::COURSES_ADMIN <=> Test::COURSES_REPORTING_ACCESS); // 1
var_dump(Test::COURSES_REPORTING_ACCESS <=> Test::COURSES_ADMIN); // 1
まあ、Backed Enum の値を取得して、ユニークかけて、from で戻せば、できなくはないというコメントも。
比較できない中身についても整列可能な ARRAY_UNIQUE_UNSORTED というオプションを追加する PoC が出ています。
ただし、 O(n^2) に計算量が上がってしまうとのこと。
Misleading error message for array unpacking of objects/enums in constant expression · Issue #9769 · php/php-src
配列 の splat operator を使ったときのエラーメッセージがおかしいというイシュー
修正は、もう来ています。
Refactoring 関連
Girgias さんが、所々のリファクタリングをやっています。
- Refactor FCI API by Girgias · Pull Request #9723 · php/php-src https://github.com/php/php-src/pull/9723
- ext/session refactoring: INI settings by Girgias · Pull Request #9322 · php/php-src https://github.com/php/php-src/pull/9322
- Ext session refactoring part 1 (session_set_save_handler() and mod user) by Girgias · Pull Request #9781 · php/php-src https://github.com/php/php-src/pull/9781
一見すると、修正行が多いので怯んでしまうのですが、コミットごとにみると、なんとか理解できます。
Discussion