Quando un watched-word-link si trova in un campo utente, viene modificato nel link HTML (e sembra terribile)

Quando un utente inserisce una parola monitorata in un campo utente, accadono cose strane.

Questo è un esempio che utilizza il campo utente Datore di lavoro/i e Te Whatu Ora come collegamento a parola monitorata.

Baseline:


Campo modificato in Te Whatu Ora:

Ha anche modificato il campo stesso:

Suggerimento

Che i collegamenti a parole monitorate vengano modificati in modo che non si applichino ai campi utente.

Non sono sicuro riguardo ad altre parole monitorate (non le ho testate).

4 Mi Piace

Ciao @nathank – Apprezzo molto i tuoi post, fornisci sempre molti dettagli su ciò che trovi!

Vedo qualcosa di simile anche quando faccio questo. Ho inserito ‘la mucca fa buh’ come parola chiave monitorata e quando la aggiungo a un campo utente mostra il link completo. Tuttavia, anche a me viene mostrata la prima lettera della parola monitorata. Anche tu vedi questo? Ho cambiato il link, l’ho rimosso e aggiunto di nuovo all’elenco dei link e mostra ancora ‘c’ davanti al link:


3 Mi Piace

@nbianca ha recentemente migliorato l’implementazione secondo:

Stai eseguendo l’ultima versione?

1 Mi Piace

Posso replicare questo secondo problema (versione: af912b4b0e):

  • Aggiungi, ad esempio, ‘Meta’ a /admin/customize/watched_words/action/link con un link di https://meta.discourse.org/
  • Crea un campo utente personalizzato (modificabile)
  • Inserisci ‘Meta’ in un campo utente personalizzato e salva
  • Aggiorna la pagina
  • ‘Meta’ è stato sostituito con ‘Mhttps://meta.discourse.org/’

4 Mi Piace

Sì, sto eseguendo l’ultima versione (aggiornata dopo quella fusione). Mi è sembrato un nuovo bug, forse correlato a quella modifica del codice?

1 Mi Piace

@loic hai qualche possibilità di dare un’occhiata a questo?

1 Mi Piace

Ci darò un’occhiata :+1:

1 Mi Piace

Non riesco a riprodurlo localmente. Penso che questo bug sia stato corretto da questa PR che è stata unita ieri:

5 Mi Piace

Possiamo chiudere? Non abbiamo riproduzioni, sei sull’ultima versione @nathank e riesci a riprodurre?

2 Mi Piace

@nathank supponendo che questo sia stato risolto ora, segnala per riaprire se è ancora un problema.

1 Mi Piace

Mi scuso Sam, sono state un paio di settimane intense.

Temo che il problema persista. Per riprodurlo è necessario che il campo utente sia impostato su :ballot_box_with_check: Mostra sul profilo pubblico?. Sono stato in grado di farlo su un sito quasi vanilla così come sul mio sito normale, molto trafficato / pieno di plugin.

Si verifica indipendentemente dall’impostazione della sensibilità alle maiuscole/minuscole e per Sostituisci così come per Link (e probabilmente anche per gli altri, ma non li ho testati).

1 Mi Piace

Ho appena notato che le parole monitorate > i link funzionano anche sui campi personalizzati.

ad es.
discourse = https://meta.discourse.com

Il campo del profilo personalizzato X ha valore: discourse > mostrerà il link nel campo del profilo.

Non credo che questo sia il comportamento previsto.

3 Mi Piace

Ciao @MarcP
Sto testando questo sul più recente branch di discourse test-passed ma non riesco a riprodurre il problema. Puoi condividere quale versione di discourse stai usando?

Prima di segnalare questo, ho eseguito una ricostruzione, quindi sono su: 2.9.0.beta13 (0cb6d8c8fc).

Ho notato questo in un campo personalizzato utente. Ogni volta che modifico il campo e faccio clic su Salva, restituisce il valore dell’URL che ho specificato per la parola.

1 Mi Piace

Sì, sono in grado di riprodurre questo problema.

Passaggi per la riproduzione:

  1. Visita AdminPersonalizzaParole monitorateLink
  2. Crea un link con parole monitorate
  3. Visita AdminPersonalizzaCampi utente
  4. Crea un campo personalizzato con tipo di campo: Campo di testo
  5. Naviga Profilo utentePreferenzeProfilo
  6. Inserisci la parola monitorata creata al punto 2.
  7. Salva le preferenze.
  8. Ricarica la pagina e troverai la parola monitorata sostituita con l’URL specificato al punto 2.

ecco la registrazione dello schermo:

3 Mi Piace

Perfetto.

Nota anche che “Test Custom Field” dovrebbe avere lo stesso stile di intestazione di “Web Site”, giusto? Lo stesso bug dell’interfaccia utente è visibile sulla mia istanza.

1 Mi Piace

~~Penso che questo sia stato menzionato in precedenza anche qui https://meta.discourse.org/t/when-a-watched-word-link-is-in-a-user-field-it-is-changed-to-the-html-link-and-it-looks-terrible/234460~~

@MarcP Nathan ha confermato che si tratta dello stesso problema e ha unito i tuoi argomenti per un più facile tracciamento. :+1:

8 Mi Piace

Ho appena notato che questo problema è ancora presente.

1 Mi Piace