Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:115970
MIME-Version: 1.0
References: <CAF+90c-Xze=9A66hfDggan-JrebJDNvS1LdU45_Lreg+LXGhuw@mail.gmail.com>
 <CAG9XoMTAcKyaVrGFwXDsJUNnU0-oTPOKGQELyFpfedMMzcWhcg@mail.gmail.com> <B9F52FB5-6A26-448D-999C-401286955973@cschneid.com>
In-Reply-To: <B9F52FB5-6A26-448D-999C-401286955973@cschneid.com>
Date: Mon, 6 Sep 2021 17:49:59 +0100
Message-ID: <CAGBsUreLnE5R_vKn-7qDV+mkbm2_t+kqYoCwkmJOZcrezE-nKw@mail.gmail.com>
To: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000021767705cb5671c0"
Subject: Re: [PHP-DEV] Alias stdClass to DynamicObject?
From: tekiela246@gmail.com (Kamil Tekiela)

--00000000000021767705cb5671c0
Content-Type: text/plain; charset="UTF-8"

Hi Nikita,

I think this might be a good idea, but I would like to propose yet another
variant.
Replace stdClass with DynamicObject and keep stdClass as an alias. It can
be deprecated in 8.3.
If we only add an alias, I am afraid that it will not catch on quickly
enough. What I am proposing is that the cast to object will create
DynamicObject by default.

$arr = [1,2];
var_dump((object) $arr);
Output:
object(DynamicObject)#1 (2) {
  ["0"]=>
  int(1)
  ["1"]=>
  int(2)
}

It will break unit tests and it might break some code (e.g. `if ('stdClass'
=== $class)`), but it will help people understand what is the preferred
name going forward without deprecating it right now.

Regards,
Kamil

--00000000000021767705cb5671c0--