php / php/php-src

Connect attributes to their targets

Aperta
#18,553 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Extension: reflection Feature Status: Needs Triage
Lingua principale
C
Stelle
40.4k
Fork
8.2k
Merge medio
2g 13h
PR unite (30g)
96

Descrizione

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?

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia esaminando il comportamento della reflection degli attributi relativo a getTarget() e alla API proposta getPropertyReflection(), insieme alla relazione esistente tra classe e proprietà di ReflectionProperty. Determina se gli attributi delle proprietà debbano esporre la classe e la proprietà dichiaranti oppure un oggetto target più ampio. Il lavoro è considerato completato quando esiste un design dell’API concordato, con il comportamento corrispondente e i test per l’esempio che coinvolge più proprietà.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
php
Ambito
backend
Tipo di issue
Funzionalità
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.