### Author:  [@taylor](https://meta.discourse.org/u/taylor)
#### Post date: [8. Februar 2022 um 18:01 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/1 "2022-02-08T18:01:02Z")
Fortsetzung von [Configuring Twitter login and rich embeds for Discourse - #114 by Hifihedgehog](https://meta.discourse.org/t/configuring-twitter-login-and-rich-embeds-for-discourse/13395/114?u=david)
Ich habe eine E-Mail über diese Sache erhalten und war sehr verwirrt, da ich mich nicht erinnern konnte, einen solchen Beitrag verfasst zu haben.
Ich bin neugierig, wie „taylor“ hier in der Zitatantwort vertauscht wurde? Ist das ein manueller Bearbeitungsfehler (ich bin mir nicht sicher, wie man einen Zitatblock mit einer Quelle manuell erstellt oder ob das überhaupt möglich ist)? Oder gibt es irgendwo einen Fehler in der Antwortlogik? @tshenry
---
### Author:  [@JammyDodger](https://meta.discourse.org/u/JammyDodger)
#### Post date: [8. Februar 2022 um 18:34 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/2 "2022-02-08T18:34:53Z")
Ich habe gerade versucht, ein paar Beiträge zu zitieren, und ich glaube, es gibt ein Problem mit dem Benutzernamen im Vergleich zum vollständigen Namen. Wahrscheinlich dieser kürzliche Commit:
>
>
> Prioritize full name in these places (when setting active):
> See: https://meta.d…iscourse.org/t/display-full-name-not-username-when-attributing-quote-or-reply/203533/6 for context
>
> \- Reply to post
> \- Composer reply to post
> \- Quoting
---
### Author:  [@isaac](https://meta.discourse.org/u/isaac)
#### Post date: [8. Februar 2022 um 19:04 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/4 "2022-02-08T19:04:24Z")
Hallo @JammyDodger,
können Sie mir mehr Informationen zu
> [@](#):
>
> etwas Seltsames passiert
geben?
Mit den Website-Einstellungen `prioritize username in ux` **aktiviert** und `display name on posts` **deaktiviert** sollten Sie den Benutzernamen im Zitat sehen, und umgekehrt für die Anzeige des vollständigen Namens.
---
### Author:  [@JammyDodger](https://meta.discourse.org/u/JammyDodger)
#### Post date: [8. Februar 2022 um 19:09 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/5 "2022-02-08T19:09:55Z")
Entschuldigung, ich hätte dort viel hilfreicher sein können. 🙂 Ich habe gerade versucht, mehrere der Beiträge in diesem Thema zu zitieren, und es gab einige seltsame Ergebnisse in meinem Composer:
@Hifihedgehogs Beitrag:
```plaintext
[quote=", post:114, topic:13395"]du hast recht! Das dauerte nur ein paar Minuten und ich wurde sofort genehmigt. Einfache Lösungen sind immer schön! :smile:
[/quote]
```
@tshenrys Beitrag:
```plaintext
[quote="Taylor, post:113, topic:13395"]
Sie können kostenlos auf die v1.1 API im Rahmen des Elevated-Plans zugreifen. Sie müssen sich einfach von Ihrem Entwickler-Dashboard aus bewerben. Ich fand es sehr schnell und einfach. Discourse Twitter-Logins funktionierten danach einwandfrei :tada:
[/quote]
```
(das ist der Stil, für den @taylor eine Benachrichtigung erhalten hat)
---
### Author:  [@isaac](https://meta.discourse.org/u/isaac)
#### Post date: [8. Februar 2022 um 19:11 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/6 "2022-02-08T19:11:20Z")
Ah, danke, dass Sie darauf hinweisen… ja, das ist nicht wünschenswert / beabsichtigt. Ich werde mich darum kümmern!
---
### Author:  [@isaac](https://meta.discourse.org/u/isaac)
#### Post date: [8. Februar 2022 um 19:16 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/7 "2022-02-08T19:16:53Z")
> [@](#):
>
> @tshenry’s post:
>
> ```plaintext
> [quote="Taylor, post:113, topic:13395"]
> You can gain access to the v1.1 API for free under the Elevated plan. You simply need to apply from your developer dashboard. I found it very quick and easy. Discourse Twitter logins worked flawlessly after that :tada:
> [/quote]
>
> ```
Der Grund, warum @taylor in diesem Zitat markiert wurde (und eine E-Mail erhalten hat), ist, dass sein Benutzername – `Taylor` – derselbe ist wie @tshenry’s vollständiger Name – `Taylor` 😮, der (der vollständige Name) aufgrund der Änderungen in [DEV: Prioritize full name when setting active (#15820) · discourse/discourse@5a93ce4 · GitHub](https://github.com/discourse/discourse/commit/5a93ce421da4e52b9d649cf609cd7d79d21a1683) korrekt priorisiert wird.
Auf jeden Fall danke für den Tipp, ich werde das beheben!
---
### Author:  [@JammyDodger](https://meta.discourse.org/u/JammyDodger)
#### Post date: [8. Februar 2022 um 19:23 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/8 "2022-02-08T19:23:37Z")
FWIW, es lässt auch Zitate ohne vollständige Namen leer:
[quote=", post:114, topic:13395"]du hast recht! Das hat nur ein paar Minuten gedauert und ich wurde sofort genehmigt. Einfache Lösungen sind immer schön! 😄
[/quote]
---
### Author:  [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [11. Februar 2022 um 19:53 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/10 "2022-02-11T19:53:44Z")
Sind wir jetzt gut damit, @isaac?
---
### Author:  [@isaac](https://meta.discourse.org/u/isaac)
#### Post date: [11. Februar 2022 um 19:59 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/11 "2022-02-11T19:59:22Z")
Ja, wir haben die Änderung rückgängig gemacht, und ich arbeite daran, die Logik zu korrigieren, bevor wir sie wieder zusammenführen.
---
### Author:  [@isaac](https://meta.discourse.org/u/isaac)
#### Post date: [11. Februar 2022 um 22:33 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/12 "2022-02-11T22:33:32Z")
Bei der Aktualisierung der Zitatlogik, um das **Benutzernamen** oder den **vollständigen Namen** über eine Website-Einstellung umschalten zu können, bin ich auf ein Problem gestoßen (ein Teil des Problems wurde oben erwähnt). Lassen Sie mich ein hypothetisches Szenario erstellen…
- **Tim** erstellt einen Beitrag
- Eine Website-Einstellung wird hinzugefügt und aktiviert, um den Anzeigenamen des OP von Benutzername **(war aktiv)** auf vollständiger Name **(jetzt aktiv)** umzuschalten
- Ein Benutzer zitiert **Tim** in einem neuen Beitrag, aber da wir jetzt den vollständigen Namen für Zitate verwenden, erhält unser anderer Benutzer **TimTam, vollständiger Name: Tim** , eine Benachrichtigung, dass jemand ihn in einem Beitrag zitiert hat, den er nie erstellt hat.
Das erneute Backen aller Zitate ist ebenfalls keine Option. Wenn diese Website-Einstellung mehrmals umgeschaltet wurde und wir mehrmals neu backen, ist ein Zitat, das von einem **vollständigen Namen** zu einem **Benutzernamen** konvertiert wird, nicht eindeutig und könnte mehrere Benutzer finden.
Der Weg, den ich sehe, ist das Hinzufügen eines zusätzlichen Datenattributs zum Zitat von `data-full-name="true"` (oder etwas Ähnlichem), um zu prüfen, ob wir den Benutzernamen im UX durch den vollständigen Namen ersetzen sollen, anstatt `data-username` zu manipulieren.
z.B.
```plaintext
data-username="always.users.username" data-post="1" data-topic="1" data-full-name="true"
```
und uns die Mühe ersparen, uns darum zu kümmern, ob wir nach Benutzername oder vollständigem Namen abfragen oder Benachrichtigungen senden. Gedanken? Lohnt sich der Aufwand, **PrettyText** und unseren Zitat-Builder für diese Änderung zu aktualisieren?
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [14. Februar 2022 um 06:39 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/13 "2022-02-14T06:39:36Z")
Ich ziehe es vor, keine Unterstützung für eine Umschaltung hinzuzufügen. Dies ist eine Entscheidung, die Sie frühzeitig bei der Einrichtung Ihrer Community treffen. Die Unterstützung einer reibungslosen Umschaltung ist extrem teuer und bringt eine Reihe von Kompromissen mit sich.
Wenn ich Sie mit priorisiertem Benutzernamen zitiere, rendern wir diesen Markdown
```plaintext
[quote="isaacjanzen, post:12, topic:217633"]
Gedanken?
[/quote]
```
Wenn ich Sie mit priorisiertem Namen zitiere, rendern wir diesen Markdown
```plaintext
[quote="Isaac Janzen, post:12, topic:217633"]
Gedanken?
[/quote]
```
Man muss vorsichtig sein mit `,` in vollen Namen - aber ich unterstütze das Entfernen davon für diesen Anwendungsfall oder ich schätze, Sie können einen Escaping-Trick einführen.
---
### Author:  [@isaac](https://meta.discourse.org/u/isaac)
#### Post date: [14. Februar 2022 um 15:21 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/14 "2022-02-14T15:21:22Z")
> [@sam](#):
>
> Ich ziehe es vor, keine Unterstützung für einen Wechsel hinzuzufügen. Dies ist eine Entscheidung, die Sie frühzeitig bei der Einrichtung Ihrer Community treffen.
Sagen Sie damit, dass Sie uns empfehlen würden, den Versuch aufzugeben, die Möglichkeit hinzuzufügen, Zitate mit vollständigen Namen anzuzeigen? Wir werden unweigerlich auf diese Wechsel-Fälle stoßen … z. B. jede bestehende Community mit einem Zitat
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [16. Februar 2022 um 00:56 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/15 "2022-02-16T00:56:36Z")
> [@isaac](#):
>
> Sagen Sie, dass Sie empfehlen würden, zu versuchen, die Möglichkeit, Zitate mit vollständigen Namen anzuzeigen, zu streichen?
Nein, nicht ganz streichen… meta hat `prioritize username in ux` deaktiviert. Das Zitat, das ich gerade gemacht habe, sollte sein: (dem ein Avatar fehlt, der behoben werden muss)
> [@Isaac Janzen](#):
>
> Sagen Sie, dass Sie empfehlen würden, zu versuchen, die Möglichkeit, Zitate mit vollständigen Namen anzuzeigen, zu streichen?
Das Markup hier sollte sein:
```plaintext
[quote="Isaac Janzen, post:14, topic:217633"]
Sagen Sie, dass Sie empfehlen würden, zu versuchen, die Möglichkeit, Zitate mit vollständigen Namen anzuzeigen, zu streichen?
[/quote]
```
Die einzige Änderung, die vorgenommen werden muss, ist, dass das Zitat-Widget die Einstellung kennt, damit es den vollständigen Namen anstelle des Benutzernamens ausfüllt. Machen Sie sich keine Sorgen um Migration und Verlauf.
---
### Author:  [@isaac](https://meta.discourse.org/u/isaac)
#### Post date: [16. Februar 2022 um 15:16 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/16 "2022-02-16T15:16:05Z")
Ich bin mir nicht sicher, ob wir hier auf dem gleichen Stand sind … als wir den folgenden Commit zusammengeführt haben: [DEV: Prioritize full name when setting active (#15820) · discourse/discourse@5a93ce4 · GitHub](https://github.com/discourse/discourse/commit/5a93ce421da4e52b9d649cf609cd7d79d21a1683), haben wir Folgendes erreicht:
> [@sam](#):
>
> make the quote widget be aware of the setting so it fills in full name instead of username
```plaintext
# quote.js
# opts.displayName = true if site settings prioritize full name
const name = opts.displayName
? opts.name || post.name
: opts.username || post.username;
# build quote markdown
const params = [
name,
`post:${opts.post || post.post_number}`,
`topic:${opts.topic || post.topic_id}`,
];
```
um die Ausgabe von
> [@sam](#):
>
> ```plaintext
> [quote="Isaac Janzen, post:14, topic:217633"]
> Are you saying you would recommend we scrap trying to add the ability to display quotes with full names?
> [/quote]
>
> ```
zu erhalten, aber das Problem war, dass die Benutzersuche basierend auf dem vollständigen Namen [bestehende Zitate beschädigte](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/7?u=isaacjanzen), die mit Benutzernamen erstellt wurden.
Daher scheint es unmöglich zu sein, Folgendes zu ignorieren:
> [@sam](#):
>
> Do not worry about migration and history.
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [17. Februar 2022 um 00:43 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/17 "2022-02-17T00:43:15Z")
> [@isaac](#):
>
> Ich bin mir nicht sicher, ob wir hier auf derselben Seite sind
Ich bin super verwirrt, weil Meta Folgendes hat:
Warum fehlt der Name im obigen Zitat? Wurde die Änderung rückgängig gemacht?
> [@isaac](#):
>
> Es fühlt sich also unmöglich an, es zu ignorieren
Ich stimme zu, dass wir hier mehr tun müssen:
Wir müssen den Benutzernamen aus der Kombination „post / topic“ abrufen und ihn in das „cooked markdown“ einfügen (das sollten wir bedingungslos tun), sonst können wir keine Avatare für den vollständigen Namen anzeigen.
Dies ermöglicht uns auch, diesen Sonderfall zu beheben, damit mein Avatar bei diesem falschen Zitat nicht angezeigt wird. (oder sogar ein falsches Zitat hervorhebt)
> [@sam](#):
>
> Es fühlt sich also unmöglich an, es zu ignorieren
---
### Author:  [@isaac](https://meta.discourse.org/u/isaac)
#### Post date: [17. Februar 2022 um 00:45 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/18 "2022-02-17T00:45:36Z")
Ah, ja
> [@isaac](#):
>
> Ja, wir haben die Änderung rückgängig gemacht, und ich arbeite daran, die Logik zu reparieren, bevor wir sie wieder zusammenführen.
> [@sam](#):
>
> Wir müssen den Benutzernamen aus der Kombination „post / topic“ abrufen und ihn in den gekochten Markdown einfügen (wir sollten das bedingungslos tun), sonst können wir keine Avatare für den vollständigen Namen anzeigen.
Ich stimme zu und kann das hinzufügen.
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [17. Februar 2022 um 00:48 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/19 "2022-02-17T00:48:05Z")
Ich bin völlig offen dafür, hier ein paar Ausnahmefälle zu beheben:
1. Benutzername stimmt nicht überein
- Was machen wir mit dem Avatar?
- Was machen wir mit dem Benutzernamen?
2. Vollständiger Name stimmt nicht überein
- Was machen wir mit dem Avatar?
- Was machen wir mit dem Namen?
Ich denke, das Beste, was wir im Moment tun können, ist einfach „überschreiben“. Wenn der Beitrag einen Namen/Benutzernamen hat, verwende ihn anstelle desjenigen, der im Zitat angegeben wurde. Aktuellen Namen/aktuellen Benutzernamen verwenden.
---
### Author:  [@isaac](https://meta.discourse.org/u/isaac)
#### Post date: [2. März 2022 um 18:14 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/20 "2022-03-02T18:14:13Z")
Alles klar, ich habe einen [PR](https://github.com/discourse/discourse/pull/16078) erstellt, der die Dinge klärt.
## Wichtigste Erkenntnisse
Wir übergeben jetzt den Benutzernamen, wenn
```plaintext
siteSettings.display_name_on_posts && !siteSettings.prioritize_username_in_ux && post.name
```
sowie den vollständigen Namen, um sicherzustellen, dass wir keine Abweichungen bei der Abfrage nach Benutzer / Avatar erhalten.
z.B.
```plaintext
[quote="Isaac Janzen, post:3, topic:7, full:true, username:isaac.janzen"]
bing bong
[/quote]
```
Was denkst du, @sam?
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [3. März 2022 um 00:54 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/21 "2022-03-03T00:54:29Z")
> [@isaac](#):
>
> Wir übergeben jetzt den Benutzernamen
Ich schätze, ich kann eine Markdown-Änderung akzeptieren, damit wir uns nicht in einem Kaninchenbau voller enormer Sicherheitsabenteuer verlieren. Die Änderung muss jedoch hyper-chirurgisch sein.
Können Sie bestätigen, dass die neue ausführliche Syntax **nur dann vorkommt** , wenn sowohl `siteSettings.display_name_on_posts` als auch `!siteSettings.prioritize_username_in_ux` wahr sind?
```plaintext
[quote="Isaac Janzen, post:3, topic:7, username:isaac.janzen"]
....
[/quote]
```
Das bedeutet, wenn Sie weder `display_name_on_posts` aktivieren noch `prioritize_username_in_ux` deaktivieren
Dann sollte das alte Zitatformat beibehalten werden:
```plaintext
[quote="isaac.janzen, post:3, topic:7"]
....
[/quote]
```
Ich bin bereit, dies vorerst als Zwischenschritt zu akzeptieren, aber unser langfristiges Ziel wäre es, diesen Lärm aus dem Zitatblock zu entfernen (Sicherheit erhöhen, Missbrauch einschränken) und das gekürzte Zitatformat zu unterstützen, das wir immer hatten, und uns durch die Sicherheits-Büchse der Pandora zu arbeiten, genau wie wir es jetzt für Inline-Oneboxen tun müssen, z. B.: [Quoting broken when name matches username - #20 by isaac](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/20?u=sam) \< - [https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/20?u=sam](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/20?u=sam)
Erwähne @tobiaseigen hier zur Sichtbarkeit, da dies mit früheren Diskussionen zusammenhängt.
Fazit:
- Meiner Meinung nach ist es in Ordnung, das Markdown-Format vorerst für diesen Ausnahmefall zu ändern – und nur für diesen Ausnahmefall – @codinghorror zur Bestätigung.
- TBD Zeitplan für Sicherheitsarbeiten, damit der Zitatblock zu `[quote="TOTALLY OPTIONAL WILL BE FISHED FROM POST IF SECURITY LINES UP AND INFO ON POST IS PRIORITIZED OVER THIS TEXT, post:3, topic:7"]` wird. – Empfehlen, damit noch eine Weile zu warten.
---
### Author:  [@isaac](https://meta.discourse.org/u/isaac)
#### Post date: [20. April 2022 um 15:58 UTC](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/22 "2022-04-20T15:58:34Z")
Zusammengeführt und bereitgestellt [DEV: Prioritize full name when display\_name\_on\_posts active by janzenisaac · Pull Request #16078 · discourse/discourse · GitHub](https://github.com/discourse/discourse/pull/16078)
> [@sam](#):
>
> Können Sie bestätigen, dass die neue ausführliche Syntax **nur dann vorkommt** , wenn sowohl `siteSettings.display_name_on_posts` als auch `!siteSettings.prioritize_username_in_ux` gesetzt sind?
> [@sam](#):
>
> Das bedeutet, wenn Sie nicht sowohl `display_name_on_posts` aktivieren ALS AUCH `prioritize_username_in_ux` deaktivieren
>
> Dann bleibt das alte Zitatformat bestehen
Ich kann das bestätigen.
[Nächste Seite](https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633.md?page=2)