[ad_1]
Hi @bouk
That’s a good catch. The idea behind the change was to avoid a dynamic property deprecation under PHP 8.2, but I hadn’t realised/thought about other code outside of the class could be directly accessing it too.
I’m not keen, but I’ve made it a public property for the next release. See. https://github.com/Automattic/fb-instant-articles/pull/1110/commits/cf51b09c844a3b4cfa583e40884a17c279f1eaf6
Thread Starter
BoUk
(@bouk)
Hi Gary,
and thanks for the swift response. Understand. Yeah, making this property public will do for me for the time being. Nice one.
