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

Kein Einsatzbericht mehr mit Avada-Theme und Layouts #336

Open
dany9328 opened this issue Sep 9, 2024 · 4 comments
Open

Kein Einsatzbericht mehr mit Avada-Theme und Layouts #336

dany9328 opened this issue Sep 9, 2024 · 4 comments
Assignees
Labels
Type: Compatibility Playing nice with themes and other plugins Type: Possible Bug

Comments

@dany9328
Copy link

dany9328 commented Sep 9, 2024

Vor der Änderung der 1.11.2 wurde der Inhalt mehrfach angezeigt. Inzwischen wird wenn ich in Avada Layouts verwende, gar kein Inhalt mehr angezeigt.
Avada 7.11.10.
Layout mit Condition "All Einsatzberichte". In der Content-Section des Layouts habe ich das "Content" Element eingefügt. Auf der Website wird an der Stelle aber nichts ausgegeben...

@abrain abrain added Type: Possible Bug Type: Compatibility Playing nice with themes and other plugins labels Sep 12, 2024
@abrain abrain self-assigned this Sep 12, 2024
@abrain
Copy link
Owner

abrain commented Sep 12, 2024

Auf der Website wird an der Stelle aber nichts ausgegeben...

Also nicht einmal der Berichtstext? Es dürfte maximal der Header (Alarmzeit usw.) fehlen. Oder wird auch innerhalb der Einsatzverwaltung das Template-Feature genutzt?

@dany9328
Copy link
Author

Es wird garnichts angezeigt.
Im Avada Layout verwende ich das "Content"-Element. In der Einsatzverwaltung habe ich das Template aktuell auf "Überall verwenden". Es macht aber absolut kein Unterschied. Wenn es auf nicht verwenden stelle, gehts auch nicht.
Wenn ich deine 2 Zeilen die du in der 1.11.3 hinzugefügt hast (Avada fix) auskommentiere, wird der Inhalt mehrfach angezeigt - das geht also dann. Wenn ich die Zeilen aber wieder reinnehme, kommt gar nichts mehr.
image
Wenn ich keine Avada Layouts verwende, also die Einsatzseite "default" nutze, gehts. Das ist aber blöd, weil dann die Einsatzberichtsseite ganz anders aussieht wie der Rest der Website. Der Layout-Support wäre schon gut.

@traubst
Copy link

traubst commented Nov 18, 2024

Bin grade am gleichen Thema dran.
Folgenden Sachverhalt haben wir jetzt:

  1. wie von @dany9328 erwähnt wird das "Content"-Element verwendet (ohne Anpassung)
  2. Das Plugin wurde auf die aktuellste Version aktualisiert und es wird jetzt (bei neuen) Einsätzen der Einsatz-Text angezeigt. Aber keine Meta-Informationen.
  3. Wir verwenden ein Template (wie auch oben erwähnt - "Überall anwenden") in der folgenden Definition:
<div class="einsatz">
    <h1 class="fusion-title-heading title-heading-left" style="margin:0">Einsatz %number% - %incidentType%</h1>
    <div class="time">Datum: %date% | Alarmierungszeit: %time% Uhr</div>
    <div class="einsatz-meta">Einsatzort: %location%</div>
    <div class="einsatz-meta">Alarmierungsart: %typesOfAlerting%</div>
    <div class="einsatz-meta">Mannschaftsstärke: %workforce%</div>
    <div class="einsatz-meta">Fahrzeuge: %vehicles%</div>
    <div class="einsatz-meta">Weitere Kräfte: %additionalForces%</div>
    <h2 class="fusion-title-heading title-heading-left" style="margin:0;margin-top: 30px">Einsatzbericht:</h2>
    <div class="einsatz-text">%content%</div>
</div>
  1. Ein Auskommentieren der
    add_action('awb_remove_third_party_the_content_changes', function () {
    hat zur Folge, dass der Einsatzbericht zwar dargestellt wird aber doppelt.

Demnach gehe ich davon aus, dass das Problem beim Filtern des Contents entsteht (4.).

Gerne würden wir das Problem mal analysieren und ggf. ein PR für einen eventuellen Fix erstellen.

Folgende Fragen hätte ich:

  1. Können wir das Pulugin irgendwie debuggen?
  2. Könntest du kurz beschreiben, was der Fix bzw. Filter bewirken soll?
  3. Hat jemand ein Avada mit dem Plugin am Laufen? Wenn ja, mit welchen Settings?

Würde mich freuen!
Danke für die Arbeit!!

@traubst
Copy link

traubst commented Nov 18, 2024

Also ich hab jetzt den Workaround gemacht und ein Flag für die

public function renderContent(string $content): string
gesetzt.

Es scheint die Funktion wird zu oft aufgerufen was zur Duplizierung des Content führt (ob das nur für das Avada gilt weiß ich nicht).
Mein Fix setzte einfach ein Flag via

    static $content_modified = false;
    if ($content_modified) return $content;
    $content_modified = true;

am Anfang der Function renderContent. Die oben geanannten Filter sind auskommentiert.

Ich bin ein WP Experte, aber ich vermute, dass es eine schönere Lösung gibt.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Compatibility Playing nice with themes and other plugins Type: Possible Bug
Projects
None yet
Development

No branches or pull requests

3 participants