Splitting of long lines with closures and imported use variables
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
Piste de recherche
L’issue ne mentionne aucun fichier source ni test. Commencez par reproduire l’entrée PHP fournie dans le playground du plugin lié, puis repérez la logique du formateur pour les paramètres de closure et les variables use ; la tâche est terminée lorsque la sortie formatée conserve les variables use avec les arguments de la fonction dans la mise en page préférée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
The way closure imports ("use" variables) are split when a line is too long is not ideal. I'm wondering if there is a way the priority can be changed so that it prefers to keep use variables together with the function arguments?
@prettier/plugin-php v0.16.1
Playground link
Input:
<?php
function test()
{
expectNotification(VerifyEmail::class)->toBeSentTo(
$user,
function ($notification, $channels, $notifiable) use ($user, &$actionUrl) {
$actionUrl = $notification->toMail($user)->actionUrl;
return $channels === ['mail'] && $notifiable->is($user);
},
);
}
Output:
<?php
function test()
{
expectNotification(VerifyEmail::class)->toBeSentTo($user, function ($notification, $channels, $notifiable) use (
$user,
&$actionUrl
) {
$actionUrl = $notification->toMail($user)->actionUrl;
return $channels === ['mail'] && $notifiable->is($user);
});
}
Of course Javascript doesn't have an equivalent of "use" imports, but I tried to approximate it below, and Prettier does what I would expect/prefer.
Prettier 2.2.1
Playground link
--parser babel
--print-width 120
--single-quote
--tab-width 4
--trailing-comma all
Input:
function test()
{
expectNotification(VerifyEmail.name).toBeSentTo(
$user,
function ($notification, $channels, $notifiable, use, $user, A$actionUrl) {
$actionUrl = $notification.toMail($user).actionUrl;
return $channels === ['mail'] && $notifiable.is($user);
},
);
}
Output:
function test() {
expectNotification(VerifyEmail.name).toBeSentTo(
$user,
function ($notification, $channels, $notifiable, use, $user, A$actionUrl) {
$actionUrl = $notification.toMail($user).actionUrl;
return $channels === ['mail'] && $notifiable.is($user);
},
);
}
- Langage dominant
- PHP
- Étoiles
- 1.9k
- Forks
- 139
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de prettier/plugin-php
-
PHP 8.3 not allowed in options? Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 62/100
prettier/plugin-php#2416 · 2 commentaires · 6 réactions ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 64/100
prettier/plugin-php#2498 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
prettier/plugin-php#2495 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
prettier/plugin-php#2494 · 1 commentaire ·
-
strange format issue Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
prettier/plugin-php#2490 ·
Toutes les issues de prettier/plugin-php
Issues similaires
-
tooling
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
UX
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
ProfessionalWiki/NeoWiki#1525 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
OpenConext/OpenConext-engineblock#2122 ·
-
Bug
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Automattic/safe-publish#594 ·