search

terminal42 / contao-easy_themes

Contao extension "easy_themes"
https://unmaintained.tech/
MIT License
13 stars 11 forks source link

easy_themes stört install.php #4

Closed jantheofel closed 11 years ago

jantheofel commented 11 years ago

Besteht keine Datenbankverbindung, stört easy_themes das Install-Tool. (Mit dem man neue Daten eintragen könnte.) Er versucht auf die Datenbank zuzugreifen, vermutlich um Usereinstellungen zu suchen. Das führt zur Nichtbenutzbarkeit des Install-Tools, weil einem das immer einen PHP-Datenbankfehler wirft.

Ich kann mir vorstellen, dass das noch mehr krude Seiteneffekte hat. Daher würde ich bitten eine Prüfung einzubauen, die easy_themes beim Laufen der install.php bis auf die DCA komplett deaktiviert.

aschempp commented 11 years ago

Kann ich bestätigen.

jantheofel commented 11 years ago

Die Frage wäre: Ist der Fehler easy_themes "anzulasten" oder ist das eher ein Core-Thema. Aus meiner Sicht sollte install.php von den Extensions nur dir DCA auswerten um die nötigen Felder zu erkennen und den Rest (wie etwa diesen DB-Zugriff) gar nicht ausführen.

Toflar commented 11 years ago

Ist der Fehler easy_themes "anzulasten" oder ist das eher ein Core-Thema.

Ich schau es mir mal für ET an. Aber du kannst ja trotzdem noch ein Ticket bei Contao anlegen und schauen, was Leo dazu meint? :-)

Toflar commented 11 years ago

Hey @jantheofel,

Könntest du testen, damit ich es im ER veröffentlichen kann? https://github.com/terminal42/contao-easy_themes/commit/9ccdacda46e859b181cb2fbaf662672ce83a7be9

Danke :-)

jantheofel commented 11 years ago

Test war erfolgreich. Danke!

ghost commented 11 years ago

Ebenfalls erfolgreich getestet vorher/nachher, einschließlich #5. Wobei ich mit Jan der Meinung bin, dass das hier nur ein Symptom ist - das Problem liegt im core. install.php sollte weder direkt (core) noch indirekt (extensions) zu DB Zugriffen führen, bevor es selbst erfolgreich bis zur DB durchgedrungen ist (d.h. ggf. die Zugangsdaten gesetzt/korrigiert werden konnten). Selbst extension DCAs sollten bis dahin nicht angefasst werden. Andernfalls kann install.php nicht ganz das online Reparatur-Tool sein, dass es in vielen Fällen schon erfolgreich ist.

Toflar commented 11 years ago

Joa, das gehört aber dann ich ein Contao-Core Ticket :) Ist übrigens veröffentlicht :)