php / php/php-src

Connect attributes to their targets

Abierto
#18,553 3 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Extension: reflection Feature Status: Needs Triage
Lenguaje dominante
C
Estrellas
40.4k
Forks
8.2k
Merge medio
2 d 13 h
PR fusionados (30 d)
96

Descripción

Description

There currently seems to be no way to get the actual target of the attribute. The getTarget method returns the bitmask of the constants. Example usage:

class X {

#[Tag]
private string $property;

#[Tag]
private string $otherProperty;

private string $extraProperty;
}

...
$attributeReflections = $propertyReflection->getAttributes(Tag::class);
// now I want to pass the reflections somewhere else and do

$property = $attributeReflection->getPropertyReflection();
$property->getValue();

Obviously there is a workaround to build some datastructure to pass the reference in along with the reflection. But it seems like it should be part of the API. Given I can limit to what type of object I can attach it to, I should be able to get the actual "instance" of the object somehow IMHO. But maybe I'm missing some important context why this doesn't make sense, sorry if that's the case :)

Essentially it's the same as when ReflectionProperty has $class even though one could argue that you always get the property from class, so you should be able to maintain the relation yourself. I believe that property attribute should at least have $class and $property, if the above proposed API is too heavy.

Does it make sense?

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza revisando el comportamiento de la reflexión de atributos en torno a getTarget() y la API propuesta getPropertyReflection(), junto con la relación existente entre clase y propiedad de ReflectionProperty. Determina si los atributos de propiedades deberían exponer su clase y propiedad declarantes o un objeto objetivo más amplio. Se considera terminado cuando exista un diseño de API acordado, con el comportamiento correspondiente y pruebas para el ejemplo que involucra varias propiedades.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
php
Área
backend
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.