PHP-Fusion-GERMANY / PHP-Fusion

Vom deutschen Team weiterentwickelter Fork
http://phpfusion-deutschland.de
6 stars 3 forks source link

Modernisierung Core #14

Closed iPaat closed 2 years ago

iPaat commented 8 years ago

Hier würde ich gerne alle Ideen zur Modernisierung sammeln. Bevor wir nämlich Dinge umschreiben müssen wir erst einmal wissen was wir modernisieren wollen.

Vorschläge

Webmeteor commented 8 years ago

Als erstes brauchen wir überarbeitete Datenbankfunktionen. Dann muss das URL-Rewriting fertig sein. Funktionen zum Überprüfen von Usereingaben.

Das sind glaube ich die Hauptsächlichen Dinge, bevor wir uns um anderes Kümmern können / sollten. Aus diesen 3 Dingen ergeben sich garantiert auch noch andere Dinge zwangsläufig, die geändert werden müssen.

iPaat commented 8 years ago

Sollen wir Bootstrap fest integrieren?

Ich finde schon richtig dass wir auf ein CSS Framework zurückgreifen. Vielleicht laden wir nicht immer automatisch die CSS und Scripte dazu aber! wenigstens die Klassen sollten wir in den Core übernehmen damit man mehr gestalten kann und wenn man will kann man sich dann auch Designs kaufen die dann auf PHP-Fusion laufen bzw. nicht so schwer zu portieren sind.

Webmeteor commented 8 years ago

Da mittlerweile fast alle Designer auf Bootstrap setzen, sollte es integriert werden. Und da Bootstrap für Anfänger ähnlich einfach ist, wie Tabellenlayouts. Würde dabei aber eher auf MaxCDN setzen.

FontAwesome sollte man vielleicht auch als Option integrieren.

Formularsystem, wie weit würdest du da gehen, ähnlich wie in der V9 der defender?

iPaat commented 8 years ago

Also ich finde die IDEE hinter Defender echt nicht schlecht. Nur wir sollten aus den Fehlern die, die Engländer gemacht haben lernen.

Webmeteor commented 8 years ago

Die Idee ist super, nur die Ausführung schlecht, bzw. hat ab und an unerklärliche Nebenwirkungen.

Die Formularfelder haben auch noch einen ungünstigen Nebeneffekt, will man alle Eventualitäten berücksichtigen, ist entweder der Code aufgebläht oder der Seitenquelltext.

Eine einfache Überprüfungsfunktion, wie sie damals im RapidCMS Projekt (sollten ja fast noch alle kennen, ansonsten das Codefragment habe ich noch) eingebaut war, würde glaube ich schon reichen. Was die UK's schön gelöst haben ist der Multiupload. Damit haben nun auch Anfänger es etwas leichter vor allem den Bilderupload zu integrieren.

Rolly8-HL commented 8 years ago

Wo kann ich mir das laden zum testen, was da schon geändert wurde. Kann da so nichts zu beitragen, arbeite gerade mit V7 PHP7 und veränderter DB_SUBMISSIONS und DB_SUBMISSIONS_ADD um auch anderen Codern die Möglichkeit zu geben über Einsenden.

Webmeteor commented 8 years ago

Noch ist nichts wirklich geändert. Ansonsten ist oben Rechts ein Button für den Download hier auf Github.

Rolly8-HL commented 8 years ago

Button für den Download

Habe ich ja gemacht aber die läuft nicht unter PHP7. Arbeite mit der (Nicht offizielle PHP Fusion Version 7.02.07 PDO PHP7) leicht abgeändert, mit automatischer Erkennung für PDO. Kann unter PHPv.5.xx installieren und danach auf PHPv.7 den Server umstellen ohne Probleme.

globeFrEak commented 8 years ago

Wie oder wo kann ich helfen?! Wie machen wir weiter?

iPaat commented 8 years ago

Du könntest PDO einbauen ( #8 ). In #8 sind schon mal die Funktionen was fehlt habe ich schon mal kommentiert. Du kannst dann einen PR gegen webmeteor:patch-2 machen. Dann können wir da schon einmal weiter machen und müssen nicht warten. Natürlich habe ich den Code noch nicht getestet.

globeFrEak commented 8 years ago

OK, ich wusste jetzt nicht genau wie ich es mit Github anstelle. Ich habe mir einen neuen Branch in meinem Repo gemacht? https://github.com/globeFrEak/PHP-Fusion/tree/Enhancement/PDO

zum Thema PHPDoc und PDO Exceptions habe ich keine große Erfahrung

iPaat commented 8 years ago

Vielen Dank, dann machst du einfach einen PR von da aus. Das sollte auch okay sein.


PHPDoc einfach einfügen:

PDO Ecxeptions kannst du erst mal so lassen. Wir würden dann am besten später die PDO Exceptions mit in das Error Log schreiben.