search

voodoocode / pannengeraet-krank-knarrenbauer

🍳 Replaces Abhörgerät Kommt-Kampfhubschrauber by silly names
MIT License
67 stars 7 forks source link

AKK Nickname Replacements #6

Closed ckaik closed 5 years ago

ckaik commented 5 years ago

Habe die "Granaten-Gretel" Spitznamen Replacements hinzugefügt.

Außerdem:

Beim Regex Pattern wurde darauf geachtet, dass zumindest AKK-47 oder AKK 47 nicht gematched werden, da man sonst sowas wie "AKK-47-47" bekommen kann. Sollten noch weitere Spitznamen entstehen, die mit Granaten-Gretel starten muss das Script vermutlich angepasst werden. Wäre schön da was Allgemeineres zu haben, aber da habe ich auf die Schnelle keine Lösung für.

Nachtrag:

idealerweise nur auf Seiten bei denen auch der volle Name einmal vorkommt.

Das ist jetzt nicht drin, aber finde das ist eher persönliche Präferenz (ich hätte es z.B. gern immer) und etwas was im Zweifelsfall konfigurierbar sein müsste. Der Umbau wäre jedenfalls denkbar einfach.

ThomasTheGerman commented 5 years ago

es gibt halt doch ne Menge Bedeutungen von AKK, die nichts mit Annegret Kramp-Karrenbauer zu tun haben, siehe https://de.wikipedia.org/wiki/AKK

die meisten davon sind ziemlich selten, aber es wäre doch unschön, wenn in einem Artikel über ein Kinderkrankenhaus plötzlich von Granaten-Gretel die Rede ist

ckaik commented 5 years ago

Punkt genommen. Habe die Änderung durchgeführt.

Edit: Negativ, irgendwas stimmt nicht, AKK wird bei mir nicht ersetzt. Kann mir grad noch nicht erklären warum, habe lediglich Code verschoben.

Edit 2: Problem gefunden, ich bin dran.

Edit 3: Okaay, doch nicht so trivial wie erwartet, wenn wirklich der vollständige Name vorkommen muss, damit der Spitzname ersetzt wird. Nix was ich kurzfristig lösen kann.

Auf anhieb hätte ich gesagt man könnte mit zwei Schleifen arbeiten, aber das gefällt mir nicht.