Seite 1 von 1

Terminanmeldung - Email an Liste

Verfasst: 15. Feb 2018, 08:22
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

Re: Terminanmeldung - Email an Liste

Verfasst: 15. Feb 2018, 21:52
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

Re: Terminanmeldung - Email an Liste

Verfasst: 7. Mär 2018, 10:39
von XimeX
Auf welcher Seite genau soll das Problem auftreten? Kann es weder in meiner v3.3 Beta, noch in der "demo" Installation nachvollziehen

Re: Terminanmeldung - Email an Liste

Verfasst: 7. Mär 2018, 22:27
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

Re: Terminanmeldung - Email an Liste

Verfasst: 7. Mär 2018, 23:10
von Offe
Ich konnte es auch nur mit folgendem Vorgehen nachstellen.

Rollenverwaltung --> Rollen Teilnahmebestätigung --> Mitglieder anzeigen --> Weitere Funktionen --> Email an Liste

Re: Terminanmeldung - Email an Liste

Verfasst: 7. Mär 2018, 23:30
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

Re: Terminanmeldung - Email an Liste

Verfasst: 8. Mär 2018, 00:38
von fasse
Hmm, das mit dem Load-Event ist eigentlich überflüssig.

Re: Terminanmeldung - Email an Liste

Verfasst: 11. Mär 2018, 18:29
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.

Re: Terminanmeldung - Email an Liste

Verfasst: 12. Mär 2018, 00:36
von XimeX
Warum überhaupt mit jQuery den Seiteninhalt ersetzen? Warum nicht ganz einfach einen normalen Link auf die andere Seite?

Re: Terminanmeldung - Email an Liste

Verfasst: 12. Mär 2018, 16:46
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.

Re: Terminanmeldung - Email an Liste

Verfasst: 12. Mär 2018, 23:14
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