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

In der Vorschau werden Module angezeigt, die für das ausgewählte Template nicht verfügbar sind #10

Closed
dgrothaus-mc opened this issue Sep 22, 2023 · 4 comments · Fixed by #11

Comments

@dgrothaus-mc
Copy link
Contributor

Description / Beschreibung

Die Modulvorschau zeigt Module an, die für das ausgewählte Template nicht verfügbar sein sollten.

Affected versions / Verwendete Versionen

System report (REDAXO 5.15.1, PHP 8.1.22, MariaDB 10.5.21)

Nur relevanter Auszug

REDAXO
Version 5.15.1
PHP
Version 8.1.22
OPcache no
Xdebug yes
Packages
module_preview 1.3.0
structure 2.15.0
structure/content 2.15.0

Reproduktion

  1. Struktur im Backend auswählen.
  2. Beliebigen Artikel bearbeiten
  3. In der URL ist zu diesem Zeitpunkt der GET Parameter ctype noch nicht gesetzt.
  4. Die Modulvorschau prüft gegen cType 0, statt gegen cType 1.
  5. Die Methode rex_template::hasModule() des structure Addons gibt immer verfügbar zurück für unbekannte cTypes.
  6. Es werden Module angezeigt, die für das Template nicht verfügbar sind.
@eaCe
Copy link
Member

eaCe commented Sep 22, 2023

Danke für den Hinweis, werde es am Wochenende checken.

@eaCe
Copy link
Member

eaCe commented Sep 22, 2023

Oder so ;) Danke!

@dgrothaus-mc
Copy link
Contributor Author

Ich hatte nicht damit gerechnet, dass jemand so schnell antwortet, und ich somit kaum Zeit habe den PR aus zu formulieren. 😁

@eaCe eaCe closed this as completed in #11 Sep 25, 2023
@eaCe
Copy link
Member

eaCe commented Sep 25, 2023

Bis zum neuen Release möchte ich noch die Workflows updaten, das schaffe ich in dieser Woche. Nur nicht direkt

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

Successfully merging a pull request may close this issue.

2 participants