laravel/pint

Asymmetric Visibility Issue

Geschlossen

#383 geöffnet am 18.07.2025

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)PHP (189 Forks)user submission
help wanted

Repository-Metriken

Stars
 (3.127 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 2T 1h) (12 gemergte PRs in 30 T)

Beschreibung

Pint Version

1.24.0

PHP Version

8.4

Description

There seems to be a bug with how asymmetric visibility is handled. The following class properties are fine:

  • public private(set) int $concurrency;
  • private int $concurrency;

However if I try to use the abbreviated form private(set) int $concurrency; as specified in the RFC then I get the error Cannot access offset of type null on SplFixedArray

Steps To Reproduce

Set a class property like private(set) int $propertyName and then run pint on the file.

Contributor Guide

Research-Richtung
Untersuche den Fehler, indem du analysierst, wie Pint Eigenschaftsmodifikatoren parst, insbesondere die Behandlung von `private(set)` ohne explizite Sichtbarkeit. Sieh dir die Grammatikdatei oder den Lexer (wahrscheinlich im `src/` Verzeichnis) an, der Eigenschaftsdeklarationen tokenisiert und parst. Der einzelne Kommentar zum Issue könnte zusätzliche Hinweise liefern. Teste mit dem bereitgestellten Reproduktionsfall und stelle sicher, dass die Korrektur mit dem PHP RFC zu asymmetrischer Sichtbarkeit übereinstimmt.
Tech Stack
php
Domain
tooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PHP 8.4asymmetric visibility syntax
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
35