Disattivare "abilita nomi" fa comportare gli admin in modo strano

[mdoggydog]

@hugh

Sembra che la tua soluzione qui sia quella di sostituire l’impostazione enable names con una che accetta un elenco di gruppi, e i nomi dei membri sono visibili solo a tali gruppi.

Il codice che ho scritto non rimuove l’impostazione enable names,^[1] ma vi aggiunge:

1. Aggiungere un’impostazione full_names_visible_to_groups (che include admins e moderators come valori obbligatori).
2. Aggiungere un metodo can_see_full_names? a Guardian, che esegue un “and” di enable_names e l’appartenenza al gruppo in full_names_visible_to_groups.
3. Utilizzare questo nuovo metodo in tutti i punti appropriati in cui un nome completo viene esposto/emesso dal server.

1 e 2 sono stati facili. 3 è più complicato, e so di aver incontrato alcuni intoppi che non ero sicuro di come risolvere senza ricevere consigli/guida. Devo tornare indietro e rivedere a fondo il mio codice e i miei appunti. (Sono passati più di 2 mesi da quando mi sono immerso in questo. )

@hugh

Nota che questo dovrebbe comunque rispettare l’impostazione display name on posts (e qualsiasi altra simile che possa esistere), quindi anche i gruppi consentiti non vedono il nome nei post se tale impostazione è disabilitata.

(Se ricordo bene, display name on posts e simili sono impostazioni lato client, che influenzano la presentazione dei dati ricevuti dal server. In altre parole, una restrizione in aggiunta a qualsiasi cosa il server emetta.)

@hugh

1. I nomi dovrebbero essere sempre visibili nella vista admin del profilo di un utente. Questo sarà il caso indipendentemente da qualsiasi impostazione.
2. I nomi dovrebbero essere mostrati nelle email solo se l’utente fa parte di un gruppo consentito.

Credo che (1) sia gestito se enable_names è vero, il che probabilmente sarà ciò che quasi tutti vorranno una volta disponibile la nuova impostazione per gruppo.^[2]

Penso di aver incontrato e gestito (2) — per lo più.^[3]

Ricordo altri casi in cui vengono trapelati nomi completi.^[4]

Comunque, rivedrò gli appunti e cercherò di inviare PR questa settimana, e nel processo farò riemergere le domande aperte/punti in sospeso.

---

1. …per una serie di ragioni, tra cui: (a) non ero sicuro di quale fosse l’intento effettivo dell’impostazione (vedi la mia domanda in un post precedente sopra) e (b) lasciarla inserita fornisce un percorso di aggiornamento incrementale più sicuro. ↩︎

2. …se si assume che enable_names = false significhi “Questo sito non utilizza nomi completi in alcun modo”. ↩︎

3. Ad esempio, quando un’email di invito viene inviata a un qualche indirizzo (ovviamente non associato a un utente, altrimenti non avrebbe bisogno di un invito), l’email include il nome completo di chi invita o no? ↩︎

4. Ad esempio, oneboxer.rb, quando fa il oneboxing di un utente locale, scrive il nome completo nel contenuto del post cotto, rendendolo visibile a chiunque, per sempre. ↩︎