prettier / prettier/plugin-php

Case build-in classes

Ouverte
#196 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

discussion enhancement needs investigation
Langage dominant
PHP
Étoiles
1.9k
Forks
139
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Ref: http://php.net/manual/en/reserved.classes.php
In php allowed use any case of build-in class:

$exception = new Exception('Message');
$exception = new EXCEPTION('Message');
$exception = new eXcEpTiOn('Message');

Always return exception build-in class.
Should we prettify class names? Also don't forget about namespaces.

  1. File without namespace - new Exception() === new \Exception();.
  2. File with namespace - new Exception !== new \Exception.

In theory we can do it:

  1. If namespace exists only correct new \Exception();
  2. If namespace doesn't exists correct and new Exception() and new \Exception();

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par examiner les exemples PHP de l’issue ainsi que les règles concernant les classes réservées et les espaces de noms auxquelles elle fait référence. La tâche sera considérée comme terminée lorsqu’une décision commune, tenant compte des espaces de noms, aura été prise pour gérer les noms de classes intégrées dans les fichiers avec espace de noms comme dans les fichiers sans espace de noms, puis implémentée et couverte par les tests pertinents du formatter.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
php
Domaine
tooling
Type d'issue
Fonctionnalité
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.