RRZE-Webteam / rrze-shorturl

WordPress-Plugin to enable a shorturl redirection service run on two servers (WP + PHP-only)
https://www.shorturl.rrze.fau.de
GNU General Public License v3.0
0 stars 1 forks source link

Liste der ShortURLs und Bearbeiten #145

Open xwolfde opened 3 weeks ago

xwolfde commented 3 weeks ago

Beim Aufruf der Funktion "Liste an ShortURLs" sehe ich erstmal nur meine eigenen Links: grafik

Das ist gut so.

Wenn ich dann auf Bearbeiten klicke, hab ich plötzlich alle gefühlte Tausende ShortURLs oben.

grafik

Bitte es so machen, dass beim Bearbeiten eines Links auch nur die eigenen Links sichtbar sind.

Eigentlich muss man ohnehin nicht die anderen Links fremder Leute sehen. Der Mehrwert ist sehr gering. Daher ggf. es so machen, dass man grundsätzlich die Links anderer nicht sehen kann, auch nicht über eine Optikn "Alle anzeigen" o.ä..

Dann wird dieser issue "mitgelöst".

thenickless commented 3 weeks ago

@xwolfde

"Eigentlich muss man ohnehin nicht die anderen Links fremder Leute sehen. Der Mehrwert ist sehr gering. Daher ggf. (..)"

Bitte spezifizieren. Es war gewünscht, dass alle Links, auch die fremder Leute zu sehen sind. Darum die Checkbox. Ich kann es freilich problemlos so umsetzen, dass es keine Checkbox "Nur meine Links" gibt und immer nur die eigenen Links angezeigt werden und die Checkbox erscheint, wenn man eingeloggt als Admin ist. Ich kann dabei auch den bisherigen Code nur auskommentieren und nicht direkt löschen, wenn es dann doch zweckmässig ist, dass alle User alle Links sehen können. Welche editiert werden können, ist bereits berücksichtigt: nur die eigenen.

"gegebenenfalls" ist hierbei zu unscharf für mich. ;-)

Kurz: Soll ich es erstmal im Code auskommentieren und so umsetzen, wie du es jetzt in diesem Issue geschrieben hast? Admins sollen dabei immer alle Links sehen und editieren können (so der letzte Info-Stand) => das beibehalten?

xwolfde commented 3 weeks ago

Ja, war in der Tat von mir gewünscht, daher auch kein Vorwurf. Meine Schuld ;) Aber jetzt sieht man im Betrieb ja, dass es vielleicht doch nicht so gut war.

Mach es bitte erstmal so, das man nur noch sein eigenen Links sieht und keine anderen mehr. Ohne Checkbox. Nur Admins sollen auch die von anderen sehen un dbearbeiten können, aber das auch nur im Backend, nicht im Frontend.

thenickless commented 3 weeks ago

@xwolfde

Alles gut, hab's nicht als Vorwurf verstanden. Ich bin dran. BTW, gerade weil es ein Plugin ist, das weitestgehend Funktionalität im Frontend bereitstellt, die für gewöhnlich im Backend gehandhabt wird, hatte ich mich (mit dem OK von Rolf @rvdforst ) für eine Datenbank-Lsg entschieden, die mittels Constraints & Triggern (wobei die Trigger redundant wurden), auf bestmögliche Performance ausgerichtet ist. Es ist ein deutlicher Unterschied zwischen zB "INSERT ... ON DUPLICATE KEY UPDATE ..." und zig WP_Query mit meta_query(s), um Datenkonsitenz zu gewährleisten ... ist long_url da, ist short_url bereits errrechnet, ist uri unique, ... (disclamer: kein Vorwurf, nur eine Erklärung meines Denkansatzes beruhend auf à la "Tut, wie ihr meint, mit den Tools eurer Wahl, Hauptsache es läuft und das schnell." :-D Aber ich habe es freilich refactored https://github.com/RRZE-Webteam/rrze-shorturl/issues/132 Dass die CPT im Backend bewusst nicht sichtbar sind, liegt daran, dass alles im Frontend läuft und ein einfaches show_ui, show_in_menu reicht da nicht. Gerade bei IdM muss aufgepasst werden. Aber vor allem, weil es ja alles nur Meta-Daten sind, die aber dennoch datenkonsitent gespeichert werden müssen. Ich möchte nicht, dass da Admins durch Editieren alles durcheinander bringen können, was die Relationen der Daten zueinander angeht. Ich bin aber dran, dass alles so läuft, wie gewünscht. Ist kein großer Aufwand mehr.