Re: [RC] [3.1] [3.2] [3.3] MySQLDumper im ACP und Index
Verfasst: 13.11.2022 23:52
Es klappt weder mit Firefox (Standartbrowser) noch mit Opera und Google Chrome.
Hab die Extension komplett auf controller umgestellt und auf twig getrimmt.
Wenn du mich per mail anschreibst kann ich dir die auch gern so zukommen lassen. Die kannst du gern auch benutzen wenn dir die änderungen zusagen. Aber die hat die mindestanforderung phpbb 3.2.6 und php 7.0 oder beides höher.
Folgendes hab ich gemacht:
- Kompletten Code auf controller basis umgestellt inklusive acp Modul
- php Code auf short arrays umgestellt
- Html Code auf neue Twig syntax getrimmt (twig3)
Code: Alles auswählen
<!-- IF not SQL_DUMPER_LINK_ENABLE -->
<!-- IF not SQL_DUMPER_USERMENU_ENABLE -->
<!-- IF not SQL_DUMPER_SHORTCUT_ENABLE -->
<!-- IF not SQL_DUMPER_TEAMLINK_ENABLE -->
<div class="errorbox">{L_MESSAGE_SQL_DUMPER_DISABLE}</div>
<!-- ENDIF -->
<!-- ENDIF -->
<!-- ENDIF -->
<!-- ENDIF -->
Code: Alles auswählen
{% if not SQL_DUMPER_LINK_ENABLE && not SQL_DUMPER_USERMENU_ENABLE && not SQL_DUMPER_SHORTCUT_ENABLE && not SQL_DUMPER_TEAMLINK_ENABLE %}
<div class="errorbox">{{ lang('MESSAGE_SQL_DUMPER_DISABLE') }}</div>
{% endif %}
Code: Alles auswählen
<!-- IF not SQL_DUMPER_LINK_ENABLE and not SQL_DUMPER_USERMENU_ENABLE and not SQL_DUMPER_SHORTCUT_ENABLE and not SQL_DUMPER_TEAMLINK_ENABLE -->
<div class="errorbox">{L_MESSAGE_SQL_DUMPER_DISABLE}</div>
<!-- ENDIF -->
Desweiteren hab ich auch ein paar fehler gefunden die ich beseitigt habe:
Du hast z.B. das hier benutzt um den Dumper per iframe einzubinden:
Code: Alles auswählen
<iframe src="{SQL_DUMPER_OUTPUT}" width="100%" height="800" a href="{SQL_DUMPER_OUTPUT}"></a></iframe>Hier hast du z.B. einen denkfehler oder ich verstehs nicht. Du lädst die quelle also die dumper seite mittels source parameter:
src="{SQL_DUMPER_OUTPUT}"Dann hast dud as hier drin stehen was vollkommen überflüssig und ohne wirkung ist:
a href="{SQL_DUMPER_OUTPUT}"und hast dann nach dem öffnenden und vor dem schliessenden iframe
</a> Stehen aber kein öffnendes <a>Korrekt wäre eigentlich dies hier:
Code: Alles auswählen
<iframe src="{SQL_DUMPER_OUTPUT}" width="100%" height="800" ></iframe>src="..." ja die Quelle die zu laden ist vorgibt. Daher ist das zweite falsch. Des weiteren hab ich dann die breite und Höhe auch aus dem Quellcode raus in css ausgelagert. Da ich das ganze auch optisch etwas angepasst habe:
So sähe das im ACP aus:
https://prnt.sc/O9sG04wNi2fb
und so in der Forenansicht:
https://prnt.sc/43bYYZh5WBVy
Des weiteren würde ich dir empfehlen diese Option aus dem ACP zu entfernen:
https://prnt.sc/CCUQS0PvceSX
Dies wäre besser aufgehoben in den Sprachdateien. Diese hab ich dir auch angepasst. Weil man brauch nicht die Sprachvariablen bei allen acp Seiten vom dumper zu laden. Das Problem ist die Sprachvariablen in den info_... Dateien werden auch geladen wenn man anderswo im ACP Seiten aufruft. Daher hab ich mal die inf_.. Dateien auf das Minimum reduziert und die Sprachvariablen in eigene Dateien ausgelagert:
https://prnt.sc/nEJTFUj5BR6H
So könntest du die Extension wahrscheinlich auch auf phpbb.com einreichen wenn du das wolltest.
Ich hab mal alles das was ich bei meinen Extension welche ich teilweise ja auch auf phpbb.com eingereicht habe angewendet.
-wie gesagt wenn du mir sagst wie ich dir das zukommen lassen kann würde ich dir das geben. Ich kann das auch auf meinen Server hochladen und dir einen Download Link schicken. Aber nicht hier im Beitrag sonst gibts evtl. Probleme mit. Wenn du es gar nicht willst ist das auch ok. Für mich war das nur ein netter Zeitvertreib.