Cleanup email module

Hast du einen Fehler in der aktuellen Version gefunden ? Diesen kannst du uns hier mitteilen und uns bei der Fehlersuche helfen.
Antworten
Offe
Beiträge: 568
Registriert: 30. Aug 2007, 17:42

Cleanup email module

Beitrag von Offe »

Hallo zusammen,

ich habe gerade die Änderungen von Stefan im Commit "245 cleanup email module" in der Testumgebung eingespielt und erhalte folgenden SQL Fehler, wenn ich eine PM schreiben will:
S Q L - E R R O R

CODE: 1054
Unknown column 'first_name.usd_value' in 'field list'

B A C K T R A C E

FILE: adm_program/system/classes/dbmysql.php
LINE: 67
CALL: DBCommon->db_error()

FILE: adm_program/system/classes/dbmysql.php
LINE: 202
CALL: DBMySQL->db_error()

FILE: adm_program/modules/messages/messages_write.php
LINE: 105
CALL: DBMySQL->query()
Der Fehler hängt mit dem SQL ab Zeile 83ff in der Datei adm_program/modules/messages/messages_write.php zusammen.

Ein weiterer Punkt ist im Bereich "Email schreiben". Mit den Änderungen werden nun auch die "versteckten" Terminrollen im Auswahlbereich Absender einer Email angezeigt. Dies ist sehr ungeschickt, denn an diese Rollen sollen keine Emails verschickt werden, ggfs. optional anbieten.

Gruß Offe
Benutzeravatar
fasse
Administrator
Beiträge: 6190
Registriert: 12. Nov 2005, 16:06

Re: Cleanup email module

Beitrag von fasse »

Hallo Offe,

ich plane dahingehend noch eine Erweiterung, so dass dann hoffentlich nur noch die richtigen Rollen und User angezeigt werden. Gerne schicke ich dir dann eine Version zum Testen zu.

Viele Grüße
Fasse
Offe
Beiträge: 568
Registriert: 30. Aug 2007, 17:42

Re: Cleanup email module

Beitrag von Offe »

Hallo fasse,

gerne kannst du mir eine neue Version zum Testen zukommen lassen.

Gruß Offe
Benutzeravatar
fasse
Administrator
Beiträge: 6190
Registriert: 12. Nov 2005, 16:06

Re: Cleanup email module

Beitrag von fasse »

Hallo Offe,

anbei deine neue Version der messages_write.php mit folgenden Verbesserungen:

* Terminrollen werden nicht mehr angezeigt
* Einzeluser werden nur noch angezeigt, wenn sie in einer Rolle sind, die der Benutzer auch sehen darf und die gleichzeitig für den Email-Versand freigegeben ist.
* evlt. ist dein Problem mit den PMs auch behoben. Dieses trat bei mir nicht auf. Vielleicht hängt das mit einer anderen Datenbankversion oder Einstellungen zusammen.

Viele Grüße
Fasse
Dateianhänge
messages_write.php.zip
(6.7 KiB) 699-mal heruntergeladen
Offe
Beiträge: 568
Registriert: 30. Aug 2007, 17:42

Re: Cleanup email module

Beitrag von Offe »

Hallo fasse,

die neue Version sieht doch ganz gut aus. Folgende Punkte sind gelöst:

* SQL-Fehler beim Klick auf PM
* Terminrollen werden angezeigt
* Einzeluser können nur einzelne Emails an User setzen, die sie sehen dürfen.

Folgende Punkte sind mir noch aufgefallen:

* der dritte Punkt oben mit den Einzelusern betrifft die PM nicht, hier kann der User an alle User eine PM schreiben --> sollte doch entsprechend auch für PM angepasst werden

* Sortierung: unter PM wird nach Nachname, Vorname sortiert, bei Email ist die Sortierung nach dem Anmeldezeitraum (ID) --> Emailsortierung entsprechend der Sortierung unter PM anpassen

Gruß Offe
Benutzeravatar
fasse
Administrator
Beiträge: 6190
Registriert: 12. Nov 2005, 16:06

Re: Cleanup email module

Beitrag von fasse »

Hi Offe,

über Abfrage bei den PMs hatte ich eben auch nachgedacht, war mir da aber nicht sicher, ob das Sinn machen würde. Evtl. will man doch allen registrierten eine Nachricht schreiben können. Aber einheitlicher ist es so schon. Hab es jetzt eingebaut.

Bei der Sortierung hatte ich wohl Tomaten auf den Augen. Das hat ja schon mal einer im Forum gepostet und ich hab mehrfach ins Script geschaut und den Grund nicht gefunden. Wahrscheinlich hab ich mir immer das PM-SQL angeschaut. Naja, jetzt sollte es auch passen.

Viele Grüße
Fasse
Dateianhänge
messages_write.php.zip
(6.69 KiB) 443-mal heruntergeladen
Offe
Beiträge: 568
Registriert: 30. Aug 2007, 17:42

Re: Cleanup email module

Beitrag von Offe »

Hi fasse,

kannst du auch die Sortierung der Rollen noch nach ABC durchführen, aktuell wird dies noch nach der ID durchgeführt.

Danke und Gruß
Offe
Benutzeravatar
fasse
Administrator
Beiträge: 6190
Registriert: 12. Nov 2005, 16:06

Re: Cleanup email module

Beitrag von fasse »

Überraschenderweise der schwierigste Teil von meinen Anpassungen.

Hoffe jetzt passt alles.
Dateianhänge
messages_write.php.zip
(6.63 KiB) 376-mal heruntergeladen
Offe
Beiträge: 568
Registriert: 30. Aug 2007, 17:42

Re: Cleanup email module

Beitrag von Offe »

Hallo fasse,

irgendwas scheint nun hier schief gegangen zu sein. Das Emailformular sieht nun beim Admin wie auch beim Testuser wie folgt aus:
Screenshot.jpg
Screenshot.jpg (22.22 KiB) 6273 mal betrachtet
Das An-Feld ist nun komplett nach unten gerutscht und auch wenn man ins Feld klickt, lautet die Meldung "keine Übereinstimmungen gefunden".

Der dazugehörige Fehler lautet:
Warning: array_merge(): Argument #2 is not an array in ~/adm_program/modules/messages/messages_write.php on line 310 Warning: array_merge(): Argument #1 is not an array in ~/adm_program/modules/messages/messages_write.php on line 364 Warning: reset() expects parameter 1 to be array, null given in ~/adm_program/system/classes/htmlform.php on line 1071
Gruß Offe
Benutzeravatar
fasse
Administrator
Beiträge: 6190
Registriert: 12. Nov 2005, 16:06

Re: Cleanup email module

Beitrag von fasse »

Hi Offe,

bei mir war der Fehler nicht aufgetreten, da ich Ehemalige mit angezeigt habe.

Jetzt funktioniert es auch mit deaktivierten Ehemaligen.

Gruß
Fasse
Dateianhänge
messages_write.php.zip
(6.65 KiB) 212-mal heruntergeladen
Offe
Beiträge: 568
Registriert: 30. Aug 2007, 17:42

Re: Cleanup email module

Beitrag von Offe »

Hi fasse,

meine Tests waren nun alle erfolgreich. Ich denke so kann man ins Rennen gehnen :lol:

Danke dir

Gruß Offe
Benutzeravatar
fasse
Administrator
Beiträge: 6190
Registriert: 12. Nov 2005, 16:06

Re: Cleanup email module

Beitrag von fasse »

Hat ja jetzt auch ein paar Versuche gebraucht ...
Antworten