Possibilité d'une erreur de copie-coller dans le code du modèle utilisateur Discourse

Moin · Août 1, 2024, 10:39

J’ai l’impression que silencedTill aux lignes 401 et 402 devrait être suspendedTill

app/assets/javascripts/discourse/app/models/user.js

608e5682e


      
          @discourseComputed("suspended_till")
          suspendedForever(suspendedTill) {
            return isForever(suspendedTill);
          }
          
          @discourseComputed("silenced_till")
          silencedForever(silencedTill) {
            isForever(silencedTill);
          }
          
          @discourseComputed("suspended_till")
          suspendedTillDate(silencedTill) {
            return longDate(silencedTill);
          }
          
          @discourseComputed("silenced_till")
          silencedTillDate(silencedTill) {
            return longDate(silencedTill);
          }

j.jaffeux · Août 1, 2024, 1:33

Oui, vous avez raison, mais cela n’affecte pas le comportement, seul le nom est incorrect.

Cela fera l’affaire :

Firepup650 · Août 1, 2024, 1:35

La ligne 397 ne devrait-elle pas également être une instruction return ?

j.jaffeux · Août 1, 2024, 1:50

Oui, cela devrait l’être, c’est un changement de comportement, une erreur que nous avons commise lors d’un refactoring il y a quelques mois.

Sujet		Réponses	Vues
Old user suspension reasons have gone missing Bug	38	8191	Janvier 9, 2016
API not returning suspended fields properly Bug	6	1103	Octobre 13, 2017
Suspend API Not Working Dev rest-api	13	1289	Janvier 7, 2021
Text on suspension message is cut off Bug	3	1864	Mai 29, 2015
[missing {{tarih}} value] Bug	4	566	Mai 28, 2018

Possibilité d'une erreur de copie-coller dans le code du modèle utilisateur Discourse

Sujets connexes