laravel/pint

Asymmetric Visibility Issue

Fermée

#383 ouverte le 18 juil. 2025

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)PHP (189 forks)user submission
help wanted

Métriques du dépôt

Stars
 (3 127 étoiles)
Métriques de merge PR
 (Merge moyen 1j 7h) (4 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Étudiez le bug en examinant comment Pint analyse les modificateurs de propriété, en particulier la gestion de `private(set)` sans visibilité explicite. Examinez le fichier de grammaire ou le lexer (probablement dans le répertoire `src/`) qui tokenise et analyse les déclarations de propriété. Le seul commentaire sur l'issue peut fournir des indices supplémentaires. Testez avec le cas de reproduction fourni et assurez vous que la correction est conforme au RFC PHP sur la visibilité asymétrique.
Stack technique
php
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PHP 8.4asymmetric visibility syntax
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35