Terminanmeldung - Email an Liste

Topics from older beta tests.
Antworten
Offe
Beiträge: 568
Registriert: 30. Aug 2007, 17:42

Terminanmeldung - Email an Liste

Beitrag von Offe »

Wenn man an ein die Liste der Terminanmeldung eine Email senden will, dann erscheint der Fooder (Admidio Logo) doppelt unter dem Mailformular.

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

Re: Terminanmeldung - Email an Liste

Beitrag von fasse »

Hallo Offe,

dieses Problem kennen wir leider schon länger:
https://github.com/Admidio/admidio/issues/609

Bisher haben wir aber noch nicht herausgefunden, woran das liegt :(

Gruß
Fasse
XimeX
Developer
Beiträge: 1149
Registriert: 12. Jul 2015, 13:21
Wohnort: Austria

Re: Terminanmeldung - Email an Liste

Beitrag von XimeX »

Auf welcher Seite genau soll das Problem auftreten? Kann es weder in meiner v3.3 Beta, noch in der "demo" Installation nachvollziehen
Benutzeravatar
fasse
Administrator
Beiträge: 5131
Registriert: 12. Nov 2005, 16:06

Re: Terminanmeldung - Email an Liste

Beitrag von fasse »

Bei einer Liste an Terminanmeldung kann ich es gerade auch nicht nachstellen, wenn ich aber eine Rollenliste aufrufe und dort dann die Funktion "Email an Liste" dann kann ich es auch in der 3.3 nachstellen.

Gruß
Fasse
Offe
Beiträge: 568
Registriert: 30. Aug 2007, 17:42

Re: Terminanmeldung - Email an Liste

Beitrag von Offe »

Ich konnte es auch nur mit folgendem Vorgehen nachstellen.

Rollenverwaltung --> Rollen Teilnahmebestätigung --> Mitglieder anzeigen --> Weitere Funktionen --> Email an Liste
XimeX
Developer
Beiträge: 1149
Registriert: 12. Jul 2015, 13:21
Wohnort: Austria

Re: Terminanmeldung - Email an Liste

Beitrag von XimeX »

Hab den Fehler. Find ich aber etwas seltsam warum das überhaupt so gemacht wurde. Schau dir mal folgende Zeilen an:
https://github.com/Admidio/admidio/blob ... #L392-L395
https://github.com/Admidio/admidio/blob ... #L435-L442

Hier wird nicht wirklich auf die andere Seite weitergeleitet sonder der content ersetzt. Dadurch sind auch 2 Seiten ineinander gerendert
Benutzeravatar
fasse
Administrator
Beiträge: 5131
Registriert: 12. Nov 2005, 16:06

Re: Terminanmeldung - Email an Liste

Beitrag von fasse »

Hmm, das mit dem Load-Event ist eigentlich überflüssig.
Benutzeravatar
fasse
Administrator
Beiträge: 5131
Registriert: 12. Nov 2005, 16:06

Re: Terminanmeldung - Email an Liste

Beitrag von fasse »

Habs jetzt angepasst. Das Problem war, dass der Seiteninhalt vorher nur in #page ersetzt wurde, damit war der alte Footer noch da.

Jetzt lade ich den Inhalt in body und es sieht gut aus. Ggf. kann man in Zukunft die Logik hier noch verbessern.
XimeX
Developer
Beiträge: 1149
Registriert: 12. Jul 2015, 13:21
Wohnort: Austria

Re: Terminanmeldung - Email an Liste

Beitrag von XimeX »

Warum überhaupt mit jQuery den Seiteninhalt ersetzen? Warum nicht ganz einfach einen normalen Link auf die andere Seite?
Benutzeravatar
fasse
Administrator
Beiträge: 5131
Registriert: 12. Nov 2005, 16:06

Re: Terminanmeldung - Email an Liste

Beitrag von fasse »

Der Code ist nicht von mir gewesen. Ich habe es jetzt erst einmal gefixed, so dass nicht mehr doppelter Inhalt angezeigt wird.

Ich denke mal, dass dies so gemacht wurde, da hier die Benutzer aus den Listen einzeln als Array übergeben werden. Dies nur über die URL zu machen, macht die URL zu lang. So werden die Daten einfach dem Request hinzugefügt.
XimeX
Developer
Beiträge: 1149
Registriert: 12. Jul 2015, 13:21
Wohnort: Austria

Re: Terminanmeldung - Email an Liste

Beitrag von XimeX »

Ich hab den code mal verbessert: https://github.com/Admidio/admidio/comm ... 0e2212f393
Jetzt haben wir auch gleich eine JS function um POST requests zu machen. Vll brauchen wir das noch an einer anderen stelle
Antworten