Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:105882
MIME-Version: 1.0
References: <CAFPFaMJ8DqBgALHLE26fu1H5eqmQuRKbUpyV_0i4_1WbtYX==Q@mail.gmail.com>
In-Reply-To: <CAFPFaMJ8DqBgALHLE26fu1H5eqmQuRKbUpyV_0i4_1WbtYX==Q@mail.gmail.com>
Reply-To: bishop@php.net
Date: Wed, 12 Jun 2019 12:16:22 -0400
Message-ID: <CAEYWF=4H2X=utR0214bhzWtVMbmS_-OXPxgA1MazTaMtfo5z-g@mail.gmail.com>
To: "G. P. B." <george.banyard@gmail.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000085a52b058b22bd65"
Subject: Re: [PHP-DEV] Feedback on state of some PHP extensions, functions, etc.
From: bishop@php.net (Bishop Bettini)

--00000000000085a52b058b22bd65
Content-Type: text/plain; charset="UTF-8"

On Wed, Jun 12, 2019 at 11:35 AM G. P. B. <george.banyard@gmail.com> wrote:

>    - PharData::setAlias, PharData::setDefaultStub and  PharData::setStub
>    always throw PharException
>    <https://www.php.net/manual/en/class.pharexception.php> [11] [12] [13]
> [11] https://www.php.net/manual/en/phardata.setalias.php
> [12] https://www.php.net/manual/en/phardata.setdefaultstub.php
> [13] https://www.php.net/manual/en/phardata.setstub.php


I don't know how much this is used in the wild, but these methods exist so
that a user may treat a Phar and a PharData as interface-equivalent objects
independent of the phar.readonly INI setting. I lean toward leaving these
no-op methods as is, but I am happy to further discuss their merit.

--00000000000085a52b058b22bd65--