Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Buchnavigation hängt sich bei Newsdetailansicht auf #8665

Closed
Olli opened this issue Mar 9, 2017 · 27 comments
Closed

Buchnavigation hängt sich bei Newsdetailansicht auf #8665

Olli opened this issue Mar 9, 2017 · 27 comments
Assignees
Labels
Milestone

Comments

@Olli
Copy link

Olli commented Mar 9, 2017

Genutzt wird ein Contao 3.5.24 mit PHP Version 7.0
Ich habe eine Buchnavigation eingebaut, Referenz ist der Startpunkt.
Wenn ich mir eine News in der Detailansicht anzeigen lassen möchte, wird der PHP Prozess nach 30 Sekunden beendet mit folgender Meldung
Fatal error: Maximum execution time of 30 seconds exceeded in /www/htdocs/***/projekte/**/system/modules/core/modules/ModuleBooknav.php on line 157

@backbone87
Copy link
Contributor

Das muss nicht zwangsläufig an der buchnavigation liegen. Die Fehlermeldung sagt nur aus, dass der request nach 30s abgebrochen wurde und zu diesem Zeitpunkt die buchnavi generiert wurde.

@fritzmg
Copy link
Contributor

fritzmg commented Mar 9, 2017

An der Stelle in ModuleBooknav.php befindet sich diese Schleife:

// Skip forward pages (see #5074)
while ($this->arrPages[$intKey]->type == 'forward' && isset($arrLookup[++$current]))
{
	$intKey = $arrLookup[($current + 1)];
}

Auf den ersten Blick erkenne ich dort aber keine Möglichkeit, dass es zu einer Endlosschleife kommen könnte.

@discordier
Copy link
Contributor

Es sei denn man hat irgendwie einen zirkulaeren Bezug in der DB gebaut, aber da hab ich auch keine Ahnung wie das gehen sollte.

@Olli
Copy link
Author

Olli commented Mar 9, 2017

Ok ein Nachtrag - es tritt nicht nur bei der Newsseite auf sondern auch noch bei einer normalen Seite.
Ich habe mal die Schleife auskommentiert - es wird nur eine Forward Seite angezeigt. Die Up und Prev Seite ist leer.

@Olli
Copy link
Author

Olli commented Mar 9, 2017

Login in das System wäre möglich falls notwendig

@Olli
Copy link
Author

Olli commented Mar 9, 2017

Noch ein Nachtrag - so sieht die Struktur aus

o1hduwd

Aktuell hängt es z.B wenn man auf "Demo Unterpunkt" geht.

@fritzmg
Copy link
Contributor

fritzmg commented Mar 9, 2017

Noch ein Nachtrag - so sieht die Struktur aus

Das scheint aber kein Screenshot der gesamten Struktur zu sein ;).

Lässt sich der Fehler evt. in der Contao Demo reproduzieren?

@Olli
Copy link
Author

Olli commented Mar 9, 2017

Ja oberhalb von Demo ist der Starpunkt. Ich muss mal schauen ob ich das morgen in meiner Demo reproduzieren kann ... wenn ich es nicht vergesse.

@fritzmg
Copy link
Contributor

fritzmg commented Mar 9, 2017

Ich meinte auf demo.contao.org

@fritzmg
Copy link
Contributor

fritzmg commented Mar 9, 2017

Welche Referenzseite ist im Modul eingestellt? Oder gar keine? Welche sonstigen Einstellungen?

@Olli
Copy link
Author

Olli commented Mar 9, 2017

Der Startpunkt ... demo.contao.org kenne ich gar nicht - muss ich mal schauen.

@Olli
Copy link
Author

Olli commented Mar 10, 2017

demo.contao.org ist ja eine 4.2er ... das nutzt mir nix in der 3.5.24er.

@Olli
Copy link
Author

Olli commented Mar 10, 2017

wer den Quelltext vor der Nase hat:
$current ist bei mir false
denn array_search($objPage->id, $arrLookup) (https://github.com/contao/core/blob/master/system/modules/core/modules/ModuleBooknav.php#L118) findet die ID nicht.
Deswegen wird die Schleife vermutl. auch endlos laufen.

@Olli
Copy link
Author

Olli commented Mar 10, 2017

So und nun kommen wir zu des Pudels Kern: Wie man meinem Screener entnehmen kann ist "Demo" in der Navigation nicht sichtbar aka versteckt. Ich habe aber in der Buchnavi angehakt, dass versteckte Seiten nicht angezeigt werden sollen.
Mache ich einen Haken bei "Versteckte Seiten anzeigen" funktioniert die die Buchnavi.

@Olli
Copy link
Author

Olli commented Mar 10, 2017

Ich kann es auf demo.contao.org nicht nachstellen.

@leofeyer leofeyer added this to the 3.5.25 milestone Mar 14, 2017
@leofeyer
Copy link
Member

Wir sollten den Fall $intCurrent === false eventuell abfangen.

@leofeyer leofeyer removed this from the 3.5.25 milestone Mar 16, 2017
@leofeyer leofeyer added invalid and removed defect labels Mar 16, 2017
@leofeyer
Copy link
Member

Ich kann es jedoch bei mir auch nicht reproduzieren.

@Olli
Copy link
Author

Olli commented Mar 17, 2017

Ok ich konnte es tatsächlich reproduzieren.
Ich hatte was vergessen - eine bzw mehrere Umleitungen (Asche auf mein Haupt @fritzmg ). Ich habe es noch mal mit einer frischen Install getestet - da tritt der Fehler beim Aufruf von "1. unterpunkt" auf.

Frische Install komplette Struktur

@discordier
Copy link
Contributor

@leofeyer den false Fall myssen wir also wirklich abfangen oder aber die forwards hier direkt aufloesen. Letzteres geht vermutlich nur rudimentaer, da ja eigene Page types moeglich waeren, welche dann hier nicht behandelt werden wyrden.

@Olli
Copy link
Author

Olli commented Mar 31, 2017

@leofeyer hat sich in dem Issue noch mal was ergeben? In der History sehe ich nichts und da das Issue auch "invalid" ist befürchte ich, dass das jetzt übersehen wird.

@leofeyer
Copy link
Member

Keine Sorge.

@Olli
Copy link
Author

Olli commented Apr 3, 2017

Ein weiterer Fall wo die Booknav spinnt ist wenn kein Haken bei "Geschützte Seiten anzeigen" gesetzt ist aber eine Seite in der Ebene liegt wo angehakt ist "nur Gästen anzeigen"

@leofeyer
Copy link
Member

@Olli Kannst Du mir nochmal erklären, wie ich das Problem reproduziere? Auf welche Seite leitet die Seite "demoweiterleitung" weiter?

@Olli
Copy link
Author

Olli commented Apr 18, 2017

Die leitet auf die Seite 'demo' weiter.
Bei der Buchnavigation muss die Referenzseite der Startpunkt sein.

@leofeyer
Copy link
Member

Ich kann das Problem jetzt reproduzieren:

Allerdings verstehe ich nicht, wie Du die Seite "1. Unterpunkt" bzw. bei mir "Link elements" überhaupt aufrufen konntest, da die Seite bei mir weder in der Haupt- noch in der Buchnavigation angezeigt wird:

Hast Du die URL manuell in die Adresszeile des Browsers eingetippt?

@Olli
Copy link
Author

Olli commented Apr 19, 2017

Na klar manuell eingetippt. Sowas kann ja auch über irgend eine Individuelle Nav aufgerufen werden oder der Besucher kommt über die Suchmaschine auf so eine Unterseite.

@leofeyer leofeyer added defect and removed invalid labels Apr 19, 2017
@leofeyer leofeyer self-assigned this Apr 19, 2017
@leofeyer leofeyer added this to the 3.5.26 milestone Apr 19, 2017
@leofeyer
Copy link
Member

Behoben in 26059a9.

jsonn pushed a commit to jsonn/pkgsrc that referenced this issue Apr 25, 2017
# Contao calendar bundle change log

### 4.3.8 (2017-04-24)

 * Correctly use the en dash in the calendar modules (see contao/core#8690).
 * Correctly support 64 character template names everywhere (see contao/core#6819).

# Contao comments bundle change log

### 4.3.8 (2017-04-24)

 * Correctly support 64 character template names everywhere (see contao/core#6819).

# Contao core bundle change log

### 4.3.9 (2017-04-25)

 * Revert the Punycode library changes (see contao/core#8693).

### 4.3.8 (2017-04-24)

 * Inline small images in protected folders in the file manager (see #636).
 * Correctly encode the URL in the DataContainer::switchToEdit() method (see #762).
 * Fix the parent view drag and drop in Firefox (see #666).
 * Correctly display the search results in the extended tree view (see #739).
 * Update the Punycode library to version 2 (see #748).
 * Fix the "delete file" button for non-admin users (see #764).
 * Prevent endless loops in the book navigation module (see contao/core#8665).
 * Limit the maximum size of dimensionless SVGs in the back end (see contao/core#8684).
 * Correctly support 64 character template names everywhere (see contao/core#6819).
 * Remove the UTF-8 BOM when combining files (see contao/core#8689).
 * Correctly move folders with an "@" in their name (see contao/core#8674).
 * Correctly redirect to the last page visited upon login (see contao/core#8632).

### 4.3.7 (2017-03-23)

 * Check the database connection in the WebsiteRootsConfigProvider class.
 * Fix the %2B conversion in the Controller::addToUrl() method.

# Contao listing bundle change log

### 4.3.8 (2017-04-24)

 * Correctly support 64 character template names everywhere (see contao/core#6819).

# Contao news bundle change log

### 4.3.8 (2017-04-24)

 * Correctly support 64 character template names everywhere (see contao/core#6819).

# Contao newsletter bundle change log

### 4.3.8 (2017-04-24)

 * Correctly support 64 character template names everywhere (see contao/core#6819).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants