slabbi / uddeIM

udde Instant Messages (uddeIM) - Instant Messages System for Mambo and Joomla
12 stars 9 forks source link

No messages in Backend #62

Closed jojo12 closed 6 months ago

jojo12 commented 6 months ago

A friend asked me, why he can't see any messages in the backend. So I saw in the menue, that there should be this possibility, to send and receive PMS in the backend. [@joomod: Ich hoffe, Du siehst mich nicht als Spielverderber! Das Ganze ist jetzt schon soo gut]

A second thing: font color there (in the admin tables is not visible: I tried in admin.uddeim.css with adding color: #000 to table.adminlist td

joomod commented 6 months ago

nein, das ist super - es soll ja funktionieren, und alle Optionen selbst zu testen ist sehr mühsam

slabbi commented 6 months ago

There is no function to create uddeIM messages in backend (only Jommla PMS messages).

joomod commented 6 months ago

hmm, bin mir nicht sicher was Du meinst? Nachrichten im backend? ich habe gerade bei meiner J3 installation geschaut, da sind auch keine Nachrichten im backend, zumindest nicht solange keine freigegeben werden müssen (wenn moderiert auf JA gesetzt ist) ich denke Du meinst den Menu button ganz oben (Nachrichten bearbeiten)?

das mit der Schrift schau ich gerade (wo genau meinst Du - unter Nachrichten bearbeiten?)

slabbi commented 6 months ago

Genau diese meint er vermutlich. Jommla hatte zu meiner Zeit eine primitive Nachrichtenfunktion, die aber nichts mit uddeIM Nachrichten zu tun hat. Es macht mMn auch keinen Sinn eine solche Funktion ins Backend einzubauen. Wer sich dort einloggen kann, hat auch Zugriff auf das Frontend. "Nachrichten lesen" ist auch keine typische Backend-Funktion. Hier sollten Administrationsaufgaben erledigt werden.

jojo12 commented 6 months ago

So müsste man das eigentlich verstecken?

joomod commented 6 months ago

vielleicht sollte ich die Buttobeschriftung einfach in "Nachrichten freigeben" ändern? (und den button ggf. ganz ausblenden, wenn keine Moderation ausgewählt ist)

jojo12 commented 6 months ago

Ah, das hat zu tun mit der Moderation von Neuen Usern?

joomod commented 6 months ago

...oder von Gästen (falls public frontend aktiviert ist)

jojo12 commented 6 months ago

I am learning a lot new things 👍

joomod commented 6 months ago

ich schau mir den code mal an, vielleicht kann man ja sogar mit anzeigen ob Freigaben erforderlich sind? (bei Null inaktiv ansonsten (1,2,3...x)

joomod commented 6 months ago

hat sich das mit der Schrift auch erledigt? (ich finde nix)

joomod commented 6 months ago

habe gerade festgestellt, dass die beiden Button (auch Meldungen bearbeiten) ja nur durch die direkte Einbindung der premium addons MCP und Spamcontroll erscheinen (waren/sind also standardmäßig nicht aktiv)

slabbi commented 6 months ago

Das MCP Plugin dient zum Kontrollieren von Störern. Es darf nicht zum generellen Überwachen benutzt werden. Wenn ein Konto überwacht wird, wird dem User auch eine entsprechende Meldung angezeigt.

joomod commented 6 months ago

ja, aber unter Nachrichten bearbeiten wird doch task=mcp aufgerufen ? ich bin nämlich gerade am überlegen die beiden Infozeilen als Button darzustellen (dort wird ja auch die Anzahl angezeigt) und die beiden MenuButtons ganz auszublenden?

aber wie meinst Du das Kontrollieren von Störern?

slabbi commented 6 months ago

Ja ud nein, es erscheint ein Button, wenn das MCP installiert ist. "Spam Control" und "MCP" arbeiten wie folgt:

"Spam Control": Der Benutzer hat im Frontend einen Knopf mit der er eine Nachricht melden kann. Diese wird im Backend dem Admin in einer Liste angezeigt. Der Admin entscheidet, was damit geschieht.

"MCP": Ein Konto mit diesem Flag wird moderiert. Der User bekommt einen Hinweis angezeigt, dass das Konto moderiert wird, der Admin damit alles sieht, was er macht. Sendet der Benutzer eine E-Mail, bekommt der Admin diese im Backend in einer Liste angezeigt. Dieser entscheidet, was damit zu tun ist.

Da muss kein zusätzlicher Botton etc. hinzugefügt werden.

jojo12 commented 6 months ago

vielleicht wäre im Deutschen "Warnmeldung" für Report besser?

joomod commented 6 months ago

Da muss kein zusätzlicher Botton etc. hinzugefügt werden.

ich meinte auch nicht zusätzlich, sondern einfach die beiden Infozeilen (die bei installierten MCP und Spamcontroll angezeigt werden) mit class="btn..." versehen und dann die beiden oberen buttons im Menu ausblenden/weglassen.

die ganze Config ist schon sehr umfangreich, warum dann den Zugang zum Nachrichten/Meldungen Control zweimal anzeigen? (es verbirgt sich ja immer derselbe link dahinter)

btw. wo/wie kann ein Admin/Moderator diesen "flag" setzen? (ist es das Mod in den user-settings?)

slabbi commented 6 months ago

Ja, es ist Mod in den User Settings.

jojo12 commented 6 months ago

joomod wrote:

hat sich das mit der Schrift auch erledigt? (ich finde nix)

nein, ich habe gesehen mit dem atum light template sieht man die Schrift, mit dem dark mode ist nur weiss auf weiss!

slabbi commented 6 months ago

Wenn das gefixt ist, wer könnte noch einmal die Komponenten, Module und Plugins aus dem main testweise installieren? Wenn das funktioniert, würde ich ein neues Release herausbringen.

joomod commented 6 months ago

so, hab die Nachrichten und Meldungen buttons zusammengefasst, einfach mal testen und anschauen (2 Dateien) admin.zip

aktuell überarbeite ich noch die Meldbuttons im Postfach (ist ein bisschen raw style :)

slabbi commented 6 months ago

Ich bin mir nicht sicher, ob das sinnvoll ist. Ich habe schon lange kein Joomla mehr laufen, kann das so nicht beurteilen. Die Toolbar Buttons hatten damals schon ihren Sinn. Kannst du mal ein Screenshot posten von Vorher/Nachher?

jojo12 commented 6 months ago

@joomod: habs installiert und probiere: @1: (Warn-)Meldung kommt, aber ich kann sie nicht anschauen, kann nur "Meldung entfernen", heisst das die "Warnung macht keinen Sinn"? oder "Nachricht löschen" weil es SPAM oder sonst was ist? Wenn ich sie aber nicht anschauen kann, wie soll ich dann beurteilen? Hier funktioniert auswählen und zeigen nicht! @2: bei Moderation, wie kann ich etwas moderieren, wenn ich Inhalt nicht sehe. EDIT: ah, ich muss auswählen und dann zeigen drücken. (funktioniert) Moderation funktioniert gut!

slabbi commented 6 months ago

Ganz ehrlich, ich würde die Funktion so belassen, wie sie ist.

joomod commented 6 months ago

ich hab am funktions code eigentlich nichts geändert, nur bei den Informationszeilen class="btn ..." hinzugefügt und die menubuttons nur auskommentiert.

alt uddeim-alt

neu uddeim-neu

@jojo mit deinerAussage zu @1 komm ich nicht klar oder meinst Du im frontend? da gibt es zuerst: (=immer) "Nachricht melden" wenn eine Nachricht gemeldet wurde steht dann da: "Nachricht gemeldet" (nur als Hinweis) und "Meldung zurückrufen" zum anklicken. die Nachricht selbst bleibt solange sichtbar bis der Admin im backend etwas damit macht

joomod commented 6 months ago

@jojo habs gefunden. ist im backend - du hast Recht, die gezeigte (gemeldete) Nachricht lässt sich nicht öffnen. schau ich mir gleich an

joomod commented 6 months ago

hat irgendwas mit dem javascript:uddeIMtoggleLayer() zu tun unabhängig ob altes oder neues layout. leider zeigt mir die Konsolole auch keinen javascript fehler (im frontend funktioniert es problemlos)

joomod commented 6 months ago

die js funktion funktioniert. Problem ist, dass der Nachrichtentext nicht mit geladen wird - deshalb kann auch nichts angezeigt werden ...weitersuchen :)

slabbi commented 6 months ago

Könnte es sein, dass die Nachricht verschlüsselt abgelegt ist? In diesem Fall kann nichts angezeigt werden.

joomod commented 6 months ago

nein, ist eine normale Nachricht. der code müßte ja irgendwo in admin.spamcontrol bzw mcp stehen?

joomod commented 6 months ago

du hast doch irgendwie Recht ??? die Nachricht steht zwar im Klartext in der Datenbank, aber über das query kommt sie verschlüsselt an ??? irgend eine Idee wieso (cryptmode ist auch 0) ???

edit: gefunden, in der uddeim_spam tabelle werden die Nachrichten verschlüsselt abgelegt (wieso?) also funzt was mit der Entschlüsselung nicht

joomod commented 6 months ago

gefunden - war mein Fehler (hab beim Überarbeiten der crypt.class eine if Kette falsch interpretiert) mit dieser Version sollte es gehen crypt.class.zip

ist im main branch schon geändert

slabbi commented 6 months ago

Checkst du bitte noch die aus admin.zip mit ein?

joomod commented 6 months ago

wollte gerade fragen :) was haltet ihr von der neuen Button Läsung? ich finds verständlicher (und oben wird es auch übersichtlicher)

slabbi commented 6 months ago

Für mich ok

joomod commented 6 months ago

alles klar - mach ich dann mal

slabbi commented 6 months ago

closed

jojo12 commented 6 months ago

super, das kann man gebrauchen.