Errore: massimo topic nel primo giorno, 4 giorni dopo il primo topic

Bug
1

Il messaggio di errore era:

Hai raggiunto il numero massimo di argomenti che un nuovo utente può creare nel suo primo giorno. Attendi 2 ore prima di scrivere di nuovo.

Questo sembra essere controllato da: ‘max topics in first day’ - la modifica di questo ha rimosso l’errore.

Il problema è che l’utente ha 8 argomenti precedenti, il primo dei quali risale a 4 giorni fa. Quindi penso che non dovrebbe essere influenzato da questa impostazione, dato che è destinata a influenzare solo il periodo di 24 ore dopo il primo post dell’utente. L’account di pubblicazione è un utente TL0.

Descrizione dell’opzione:

Il numero massimo di argomenti che un utente può creare nel periodo di 24 ore dopo aver creato il suo primo post

Potrebbe valere la pena verificare se questo è vero anche per l’impostazione ‘max replies in first day’.

2

Yes, the code seems to be checking to see if a user is TL0 - not checking that it’s the user’s first 24 hours on the site. If that’s the case, possibly the message that is displayed to the user should be changed. The default requirements for achieving TL1 are fairly low though. I’d expect that considering TL0 users to be new users is correct for most cases.

1 Mi Piace
3

@simon Yeah, I’m experimenting with adjusting the difficulty for users to achieve TL1+.

If the functionality of these settings are kept the same (and assuming it is the same case for max replies in first day), I think it could be good to rename them to be consistent with the other ‘newuser’ settings. As well as update the message to users as you mentioned and the setting descriptions.

E.g.
max topics in first day -> newuser max topics per day
max replies in first day -> newuser max replies per day

4

Questo non è ancora come dovrebbe essere? Questo codice
mi sembra sbagliato, controlla se il primo post dell’utente ha più di un giorno?

 def new_user_posting_on_first_day?
    !staff? && trust_level < TrustLevel[2] &&
      (
        trust_level == TrustLevel[0] || self.first_post_created_at.nil? ||
          self.first_post_created_at >= 24.hours.ago
      )
  end
7

Ok, è stato un po’ complicato da districare, ma penso che questo correggerà il problema:

In sostanza, abbiamo cambiato l’intento di new_user_posting_on_first_day? anni fa in modo che non corrispondesse più alla descrizione e agli errori. Questo ripristina il vecchio comportamento e non considera gli utenti TL0 come nuovi utenti in ogni momento ai fini di questo controllo.

3 Mi Piace