prefs peut indiquer le statut_tmp - fixes #5035 #5167

Open
JLuc wants to merge 2 commits from JLuc/spip:#5035_prefs_delirium into master
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
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.
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.
Loading…
There is no content yet.