Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mod_password & mod_change_password #704

Closed
jedoCodes opened this issue Feb 12, 2017 · 10 comments
Closed

mod_password & mod_change_password #704

jedoCodes opened this issue Feb 12, 2017 · 10 comments
Assignees
Labels
Milestone

Comments

@jedoCodes
Copy link

jedoCodes commented Feb 12, 2017

customTPL is not working on Contao 4.3.4.

the dropdown is empty

screenshot at feb 12 12-15-29

@leofeyer
Copy link
Member

How do I reproduce this?

@jedoCodes
Copy link
Author

jedoCodes commented Feb 17, 2017

  • install contao 4.3.4
  • add a password or change_password module
  • select a custom template (it's empty)

  • contao installieren
  • theme und layout erstellen
  • module hinzufügen
  • individuelles Template auswählen (hier ist die Auswahl bereits leer)

@jedoCodes
Copy link
Author

Die Templates müssten heißen (in den klammern ist die aktuelle Bezeichnung):

mod_lostPassword.html5 (mod_password.html5)
mod_changePassword.html5 (mod_change_password.html5)

@leofeyer
Copy link
Member

Hast Du denn ein eigenes Template angelegt? Und wenn ja, wie hast Du es benannt?

@jedoCodes
Copy link
Author

jedoCodes commented Mar 15, 2017

nein ich habe kein eigenes angelegt, da diese nicht abgefragt werden. in meinen letzten post stehen in den klammern die Bezeichnungen, die in den Modulen hinterlegt sind.

diese werden aber nicht geladen, auch nicht wenn man eigene nutzt.

Das steht ind er Funktion:
return $this->getTemplateGroup('mod_' . $dc->activeRecord->type);

und das in der DB im field "type"

lostPassword
changePassword

@leofeyer
Copy link
Member

Beim Testen ist mir aufgefallen, dass das Problem auch mit den Newsletter-Modulen besteht. Wir sollten also einmal alle Module und Inhaltselemente prüfen.

@leofeyer leofeyer added this to the 4.3.6 milestone Mar 15, 2017
@leofeyer leofeyer self-assigned this Mar 17, 2017
@leofeyer
Copy link
Member

Folgendes müssen wir ändern:

Custom Navigation

Die individuelle Navigation nutzt aktuell das Template mod_navigation, sollte aber mod_customnav nutzen, damit das Template ausgewählt und überschrieben werden kann.

Change Password

Das Modul "Passwort ändern" nutzt das Template mod_change_password, sollte aber mod_changePassword nutzen.

Lost Password

Das Modul "Passwort vergessen" nutzt das Template mod_password, sollte aber mod_lostPassword nutzen.

Close Account

Das Modul "Konto schließen" nutzt das Template mod_password, sollte aber mod_closeAccount nutzen.

Module Comments

Das Kommentarmodul nutzt das Template ce_comments, sollte aber mod_comments nutzen.

Ich würde gerne alle Änderungen bis zum nächsten Feature-Release (Contao 4.4) aufschieben, da es zwar grundsätzlich Bugfixes sind, aber sie ggf. Auswirkungen auf angepasste Templates haben.

@leofeyer leofeyer modified the milestones: 4.4.0, 4.3.6 Mar 17, 2017
leofeyer added a commit to contao/newsletter-bundle that referenced this issue Mar 17, 2017
@Toflar
Copy link
Member

Toflar commented Mar 20, 2017

Ich würde gerne alle Änderungen bis zum nächsten Feature-Release (Contao 4.4) aufschieben, da es zwar grundsätzlich Bugfixes sind, aber sie ggf. Auswirkungen auf angepasste Templates haben.

Approved.

@jedoCodes
Copy link
Author

ich konnte kein Template anpassen, da dieses nicht geladen bzw. zur Auswahl steht. Das bezieht sich aber nur auf die beiden Passwort-Module, die anderen nutze ich selber nicht.

@leofeyer
Copy link
Member

Geändert in a4e7ab8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants