On Tue, Sep 22, 2020 at 7:51 AM Nikita Popov <nikita....@gmail.com> wrote:

> Currently, placing an attribute on property (or constant) groups is not
> allowed:
>
> This is a case that was not explicitly mentioned in the RFC and we decided
> to be conversative when landing the initial implementation.
>
>
Conservative is a good default, but as Tyson pointed out, nobody is
confused about visibility modifiers applying to the entire group, I don't
believe they'd be confused about attribute application.
((Though I am curious what phpdoc syntax says here.))


> However, this restriction seems pretty arbitrary to me, and I think we
> should remove it. While there is some potential ambiguity as to whether the
> attribute applies to all properties or only the first one, I think the
> general expectation is that it should apply to all properties, just like
> the property type does.
>

Agreed. The limitation seems arbitrary and unnecessary.  Call this a
refinement of the feature previously landed, and let's get it into 8.0.

-Sara

Reply via email to