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

Templates mit "-" im Dateinamen nicht mehr erkannt/auswählbar #725

Closed
tuxenberger opened this issue Sep 6, 2019 · 9 comments

Comments

@tuxenberger
Copy link

commented Sep 6, 2019

Betrifft Contao ab 4.8.1

Laut Spooky werden nur noch Templatenamen mit "_" erkannt, TEmplates mit "-" werden im Backend in der Liste nicht mehr angezeigt und sind damit nach einem Update und der Bearbeitung eines Elements oder Artikels auf das Default Template zurückgeworfen.

Da einige meiner Projekte ein "-" im Templatenamen haben und auch einige Kauftemplates diese Namensvergabe haben ist das ein ziemliches Problem, was nur mit einigem Arbeitsaufwand wieder gerade zu kriegen ist (Projekt mit mehr als 5.000 Elementen)

Welchen Grund hat der Wegfall der Unterstützung des Bindestrichs?

Siehe auch https://community.contao.org/de/showthread.php?75906-Templates-nach-Update-nicht-mehr-ausw%C3%A4hlbar

@aschempp

This comment has been minimized.

Copy link
Contributor

commented Sep 6, 2019

Kannst du bitte testen ob das Problem mit dem 4.8-branch behoben ist?

@tuxenberger

This comment has been minimized.

Copy link
Author

commented Sep 6, 2019

Bin leider etwas unwissen was das angeht. Sorry :-/

@exscorp

This comment has been minimized.

Copy link

commented Sep 6, 2019

Contao 4.8.2

das "Problem" wurde nicht behoben.
Kann es an dem "Picker" liegen, den mir werden zwei verschiedene Selectfelder angeboten.

In dem Beispiel wurde das Navigationsmenü genommen.
In meinem Templateordner liegen folgende Dateien:

  • nav_default.html5

  • nav_default_test.html5

  • nav_default-test.html5

  • mod_navigation.html5

  • mod_navigation_test.html5

  • mod_navigation-test.html5

Beim Navigationstemplate bekomme ich ein normales Selectfeld und kann alle Templates auswählen.

Beim Individuellen Template bekomme ich ein chosen Selectfeld und mir wird nur das "mod_navigation_test.html5" Template angezeigt. Der globale Standardtemplate wird auch nicht angezeigt.

template-auswahl

@tuxenberger

This comment has been minimized.

Copy link
Author

commented Sep 6, 2019

Das ist die aktuelle Version die ich habe - 4.8.2 ist installiert.

@tuxenberger

This comment has been minimized.

Copy link
Author

commented Sep 6, 2019

Ich bearbeite einen Artikel und möchte diesen mit einem Template versehen. Im Ordner liegt sowohl ein Template mit der Bezeichnung "mod_article_hero.html5" wie auch "mod_article-hero.html5". Es wird nur "mod_article_hero.html5" in der Liste angezeigt.

Bildschirmfoto 2019-09-06 um 10 21 52
Bildschirmfoto 2019-09-06 um 10 22 27

@fritzmg

This comment has been minimized.

Copy link
Collaborator

commented Sep 6, 2019

@leofeyer imho we should revert the changes to the template grouping. I don't think that there can be a solution that works for every use case.

@leofeyer

This comment has been minimized.

Copy link
Member

commented Sep 6, 2019

I disagree. The new logic works fine, we have just changed how the method is called:

Contao 4.4

public function getArticleTemplates()
{
return $this->getTemplateGroup('mod_article');
}

Contao 4.8

public function getArticleTemplates()
{
return $this->getTemplateGroup('mod_article_');
}

This is actually a bugfix, because the "custom template" drop-down menu is not supposed to show the original template.

Contao 4.4

Contao 4.8

We could adjust the glob pattern to also support hyphens, however, using hyphens in custom templates only worked accidentally, because the glob pattern happened to return them. We have, however, never recommended or documented using hyphens in template names.

und auch einige Kauftemplates diese Namensvergabe haben

@tuxenberger What themes exactly?

leofeyer added a commit that referenced this issue Sep 6, 2019
@tuxenberger

This comment has been minimized.

@leofeyer

This comment has been minimized.

Copy link
Member

commented Sep 6, 2019

See #731 for a possible fix.

@leofeyer leofeyer closed this Sep 10, 2019

leofeyer added a commit that referenced this issue Sep 10, 2019
Support hyphens in custom template names (see #731)
Description
-----------

This PR fixes #725, however, since the underlying problem is caused by a wrong template naming scheme, I am not sure whether we should merge this.

@contao/developers WDYT?

Commits
-------

14fc66c Support hyphens in custom template names (see #725)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.