Journalisation : ajout de l'id et du nom de l'auteur, correction du message. fix #4913 #4933

Merged
b_b merged 3 commits from MathieuAlphamosa/spip:issue_4913 into master 12 months ago

fix #4913

fix : id et nom de l'auteur
fix : caractères encodés et suppression du "L apostrophe"

fix #4913 fix : id et nom de l'auteur fix : caractères encodés et suppression du "L apostrophe"
MathieuAlphamosa added 1 commit 1 year ago
b_b approved these changes 1 year ago
b_b commented 1 year ago
Owner

Ha misère, ya conflit maintenant, désolé @MathieuAlphamosa mais peux-tu mettre à jour la branche de ta PR pour qu'on puisse merger (sinon dis le moi et je m'en charge).

Ha misère, ya conflit maintenant, désolé @MathieuAlphamosa mais peux-tu mettre à jour la branche de ta PR pour qu'on puisse merger (sinon dis le moi et je m'en charge).

Je veux bien que tu t'en charge @b_b, mon GitKraken a expiré et j'ai pas pris le temps de passer à la caisse...

Je veux bien que tu t'en charge @b_b, mon GitKraken a expiré et j'ai pas pris le temps de passer à la caisse...
b_b commented 1 year ago
Owner

Et voilà c'est up, je te laisse vérifier que c'est bon pour le merge :)

Et voilà c'est up, je te laisse vérifier que c'est bon pour le merge :)
Owner
@rastapopoulos @cerdic @marcimat un avis ?
Owner

Sur le principe, ok. Pas d’avis spécifique.

Sur le code :

    1. il suit pas trop nos règles d’écriture (il faudrait repasser un coup)
    1. isset(x) and x => !empty(x) plus concis
Sur le principe, ok. Pas d’avis spécifique. Sur le code : - 1) il suit pas trop nos règles d’écriture (il faudrait repasser un coup) - 2) `isset(x) and x` => `!empty(x)` plus concis
b_b force-pushed issue_4913 from 2278b15b1d to 92847abff0 12 months ago
Owner

Sur le principe, ok. Pas d’avis spécifique.

Sur le code :

    1. il suit pas trop nos règles d’écriture (il faudrait repasser un coup)
    1. isset(x) and x => !empty(x) plus concis

Wala c'est fait :)

> Sur le principe, ok. Pas d’avis spécifique. > > Sur le code : > > - 1) il suit pas trop nos règles d’écriture (il faudrait repasser un coup) > - 2) `isset(x) and x` => `!empty(x)` plus concis > > Wala c'est fait :)
b_b merged commit f32c859197 into master 12 months ago
b_b deleted branch issue_4913 12 months ago

Reviewers

b_b approved these changes 1 year ago
The pull request has been merged as f32c859197.
Sign in to join this conversation.
Loading…
There is no content yet.