Seite 3 von 4
Re: [3.2] [3.3] Collapse Quote
Verfasst: 24.02.2024 20:50
von Kirk
Thorsten
Es funktioniert bei einem Zitat im Zitat leider immer noch nicht, weder bei der Vorschau noch nach dem speichern des Beitrages. Ich hatte die 1.3.1 neu installiert, es spielt auch keine Rolle welche Einstellungen im ucp bei "Sichtbare Textzeilen" eingestellt sind. Bei nur einem Zitat funktioniert es bestens.
Re: [3.2] [3.3] Collapse Quote
Verfasst: 24.02.2024 21:28
von IMC
Hi Udo,
ich habe eben nochmal ein Update mit dem ZIP von GitHub in meinem Testboard gemacht.
Es Funktioniert. Mit 2-fach und 3-fach verschachtelten Zitaten getestet. Hast du nach dem Update alle Cache Speicher gelöscht. Eine andere Erklärung habe ich auf die schnelle nicht.
Re: [3.2] [3.3] Collapse Quote
Verfasst: 24.02.2024 21:45
von Kirk
Es war kein Update sondern eine neu Installation. Beim aktivieren einer EXT wird eigentlich automatisch der Cache gelöscht, hatte es dennoch durchgeführt. Selbst in einem TB ohne irgendeine EXT funktioniert es nur mit einem Zitat, im Quelltext wird deine js auch korrekt geladen.
Getestet mit phpBB 3.3.11 und PHP 8.2 sowie mit den Browsern FF und Opera. Es wird auch im Entwicklertool kein Fehler angezeigt.
Re: [3.2] [3.3] Collapse Quote
Verfasst: 24.02.2024 21:51
von IMC
Kannst du mir einen Link zu den Beitrag senden, bei den der Fehler auftritt?
Re: [3.2] [3.3] Collapse Quote
Verfasst: 24.02.2024 22:03
von Kirk
Leider nein, weil meine TBs mit einem Verzeichnisschutz versehen sind.
Edit:
Das was sich machen kann in meinem LB dafür einen Bereich (Passwort geschützt) zu erstellen, dies kann ich dir dann per PN schicken. Aber erst morgen da ist die NI am frischesten
Re: [3.2] [3.3] Collapse Quote
Verfasst: 24.02.2024 22:16
von IMC
Wenn du den Button zum aufklappen untersuchst muss das DIV diese Klassen haben class="imcger-quote-togglebutton imcger-visible".
Wenn nicht ist irgend etwas schräg. Die Klasse imcger-visible ist in der Version 1.3.1 dazu gekommen.
Re: [3.2] [3.3] Collapse Quote
Verfasst: 25.02.2024 09:10
von Kirk
Nach mehreren Tests ist mir folgendes aufgefallen, zitiert man einen Beitrag der genauso viele bzw. weniger Zeilen hat als im ucp eingestellt, bekommt man im Entwicklertool folgende Meldung:
Code: Alles auswählen
Uncaught TypeError: quoteElement.querySelectorAll(...)[0] is undefined
imcgerQuoteBox domain.de/ext/imcger/collapsequote/styles/all/template/quote.js?assets_version=3263:171
<anonymous> domain.de/ext/imcger/collapsequote/styles/all/template/quote.js?assets_version=3263:207
es erscheint auch dieser Button zum aufklappen nicht.
Wenn das Zitat mehr Zeilen hat als im ucp eingestellt, funktioniert alles.
Re: [3.2] [3.3] Collapse Quote
Verfasst: 25.02.2024 10:42
von IMC
Danke Udo.
Mit dieser Information konnte ich den Fehler reproduzieren.
Edit:
Ich habe jetzt die fehlerbehafteten Releases gelöscht und eine fehlerbereinigte Version 1.3.2 als beta hochgeladen.
Eventuell pflege ich noch Änderungen ein bevor ich ein neues Release erstelle.
Re: [3.2] [3.3] Collapse Quote
Verfasst: 25.02.2024 11:59
von Kirk
Mit der V 1.3.2-b1 funktioniert es ohne Prob. auch mit mehreren Zitaten, egal was man im ucp eingestellt hat.
Re: [3.2] [3.3] Collapse Quote
Verfasst: 25.02.2024 12:33
von IMC
Vielen Dank für die Rückmeldung.
Da hatte ich mal wieder einen Fehler eingebaut den ich "regelmäßig" bei JS begehe. Nach dieser Peinlichkeit werde ich mit Sicherheit mehr dafür tun um dies zu vermeiden.