Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:128194
Precedence: bulk
MIME-Version: 1.0
References: <CAEKnhAF3KOGcWZ6CYxpWkpZbNOGo79DD7p=WsAoJvqdJ5xG_Uw@mail.gmail.com>
In-Reply-To: <CAEKnhAF3KOGcWZ6CYxpWkpZbNOGo79DD7p=WsAoJvqdJ5xG_Uw@mail.gmail.com>
Date: Wed, 23 Jul 2025 16:06:41 +0200
Message-ID: <CAMU7irLYM9JAsNQjotxA9QzZtFRoQq9c9PSn90eaf_Dz_x4JiA@mail.gmail.com>
Subject: Re: [PHP-DEV] [RFC] JSON Schema validation support
To: Jakub Zelenka <bukka@php.net>
Cc: PHP internals list <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
From: alex.daubois+php@gmail.com (Alexandre Daubois)

Hi Jakub

Le mer. 23 juil. 2025 =C3=A0 15:33, Jakub Zelenka <bukka@php.net> a =C3=A9c=
rit :
>
> Hello,
>
> I would like introduce and open discussion for RFC proposing the addition=
 of JSON Schema validation support to JSON extension:
>
> https://wiki.php.net/rfc/json_schema_validation
>
> If this is successful, it should be just the first for further improvemen=
ts (see the future scope).
>
> Kind regards,
>
> Jakub

Thank you for proposing such a feature! I personally really like it
and would be glad to see this land into PHP.
I think it would be even more useful if JsonSchema offers more ways to
load a schema. Schemas are often either remotely stored or stored in a
file.
It would be great to add two new factory methods:
JsonSchema::fromUri() and JsonSchema::fromFile(). Maybe
JsonSchema::createFromString() should also be renamed to fromString()
to be consistent with the recent additions to XMLReader.

Best,
Alex