spip
/
spip
29
15
Fork 19
Code Issues 405 Pull Requests 8 Projects Releases Wiki Activity
Labels Milestones

prefs peut indiquer le statut_tmp - fixes #5035 #5167

Open
JLuc wants to merge 2 commits from JLuc/spip:#5035_prefs_delirium into master
Conversation 2 Commits 2 Files Changed 1
JLuc commented 2 months ago

Également petites corrections de typages et phpdoc + j'ai hésité à remplacer 'boo' par 'foo' pour normaliser un peu le truc.

Également petites corrections de typages et phpdoc + j'ai hésité à remplacer 'boo' par 'foo' pour normaliser un peu le truc.
JLuc added 1 commit 2 months ago
cerdic commented 2 months ago
Owner

c'est pas génial du tout d'envoyer dans une PR qui fix une issue d'autres modifs, qui plus est dans le même commit : ça nuit à la lisibilité du patch et de la discussion, on doit parler de plusieurs choses à la fois etc.

c'est pas génial du tout d'envoyer dans une PR qui fix une issue d'autres modifs, qui plus est dans le même commit : ça nuit à la lisibilité du patch et de la discussion, on doit parler de plusieurs choses à la fois etc.
JLuc commented 2 months ago
Poster

Au passage en ligne 2114 il y a probablement une erreur.

Mais je ne saurais pas la corriger (mon réseau neuronal de reconnaissance du pattern strpos($host, ':') == false erroné n'étant pas connecté à la compréhension de ce que sont ces variables et de ce que fait ce bout de code).

Au passage [en ligne 2114](https://git.spip.net/spip/spip/src/commit/a1c06668631104e233228adb7e756e6944ab1eec/ecrire/inc/utils.php#L2114) il y a probablement une erreur. Mais je ne saurais pas la corriger (mon réseau neuronal de reconnaissance du pattern `strpos($host, ':') == false` erroné n'étant pas connecté à la compréhension de ce que sont ces variables et de ce que fait ce bout de code).
JLuc added 1 commit 2 months ago
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
Labels
Apply labels
Clear labels
accessibilité amélioration
Amélioration, nouvelle fonctionnalité APIs authentification base de données bug
Ca ne fonctionne pas code généré compilo css divers documentation doublon
Ce ticket est un doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide
Ticket invalide javascript langues LDAP plugin PostgreSQL refusé
Ignoré, c'est comme Ca... sécurité traduction
No Label accessibilité amélioration APIs authentification base de données bug code généré compilo css divers documentation doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide javascript langues LDAP plugin PostgreSQL refusé sécurité traduction
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'JLuc/spip:#5035_prefs_delirium'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes