Erreur d’écriture de spip_log() ? #4992

Open
opened 1 week ago by marcimat · 0 comments
marcimat commented 1 week ago
Owner

Note: En attendant d’avoir un jour une classe de log avec monolog ou je ne sais quoi de natif ou non qui étendrait la Psr\Log\LoggerInterface…

En attendant donc, j’ai vu des spip_log mal formées (ça m’étonne pas vu la syntaxe) :

  • spip_log('texte', 'module' . _LOG_XX) au lieu de :
  • spip_log('texte', 'module.' . _LOG_XX)

Donc il manque le . à la fin du module.

Enfin c’est ce que je croyais…

Parce que le code utilise un point facultatif pour analyser le second paramètre :

preg_match('/^([a-z_]*)\.?(\d)?$/iS', (string)$name, $regs);

Mais le phpdoc ne le signale pas lui…

Ça me parait un peu abscons, peut être encore plus qu’avant :p

Note: En attendant d’avoir un jour une classe de log avec monolog ou je ne sais quoi de natif ou non qui étendrait la Psr\Log\LoggerInterface… En attendant donc, j’ai vu des spip_log mal formées (ça m’étonne pas vu la syntaxe) : - `spip_log('texte', 'module' . _LOG_XX)` au lieu de : - `spip_log('texte', 'module.' . _LOG_XX)` Donc il manque le `.` à la fin du module. Enfin c’est ce que je croyais… Parce que le code utilise un point facultatif pour analyser le second paramètre : ```php preg_match('/^([a-z_]*)\.?(\d)?$/iS', (string)$name, $regs); ``` Mais le phpdoc ne le signale pas lui… Ça me parait un peu abscons, peut être encore plus qu’avant :p
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.