On Sun, Jul 17, 2016 at 5:31 PM, Levi Morrison <le...@php.net> wrote:
> On Mon, Jul 11, 2016 at 10:30 AM, Aaron Piotrowski <aa...@trowski.com> > wrote: > > Hi Levi, > > > > On Jul 9, 2016, at 10:12 AM, Levi Morrison <le...@php.net> wrote: > > > > On Sat, Jul 9, 2016 at 8:16 AM, Aaron Piotrowski <aa...@trowski.com> > wrote: > > > > > > Additionally, I propose adding a getName() method to ReflectionType that > > returns only the name of the type, regardless of nullability. Casting > should > > not be required to get information from an object, but currently this is > the > > only way to get the type name from ReflectionType. Most other reflection > > classes include a getName() method, this seems to have been an oversight. > > > > > > This wasn't an oversight. If we add union or intersection types then > > not all types are named (for instance `ArrayAccess & Countable & > > Traversable` is not a named type). This is why it doesn't exist on the > > base ReflectionType. > > > > > > Good point, then I agree getName() should be in an extending class as in > the > > RFC. > > > > > > I have surveyed some of the people who have voted no. Their reasons > > vary but based on these conversations it seems to me that by dropping > > ReflectionClassType and the associated autoloading mechanism that > > overall we'd be happier. I do agree with Aaron that at least *some* > > changes really need to go into 7.1. How do people feel about my > > proposal to just drop autoloading and `ReflectionClassType`? > > > > > > This sounds reasonable to me. > > > > Aaron Piotrowski > > This has been quiet for a few days. I am assuming this means there are > no objections. I will work on updating the RFC and revisit this as > soon as I can. > Sadly I wasn't able to get this done in time. If anyone else has time to add this to 7.1 and the RM's will accept it great; otherwise I'll revisit this in version 7.2.