Neue Features

[adrianer]

Lieber Ulrich,

ich hatte vor 2 Monaten VCMS geforkt um paar neue Features zu implementieren und auch paar Bugs zu beseitigen. Nun wäre es an der Zeit zu schauen, wie man das zurück zum Hauptzweig mergen könnte...

Changelog:

Neue Features:

• Import von Personen und Veranstaltungen mittels CSV-Dateien
• Export eines Druck-fertigen Adress-Verzeichnises als PDF-Datei
• Zusätzliche Felder bei Personen: Studium (Zwecks Unterscheidung von Studium und Beruf, wie das bei Adressverzeichnissen meiner Verbindungen bischer immer gemacht wurde), LinkedIn und Xing
• Felder für die rechtliche + Kassiereradministration (IBAN, erteilte Einzugsermächtigung, Datenschutzerklärung unterschrieben)
• Beim verschicken der Rundmal den Nutzer informieren, dass keine email verschickt wird, da 0 Personen ausgewählt wurden (passiert z.B. durch Auswahl einer unpassenden Region). Das hatte bisher zur Verwirrung geführt.
• Der header nutzt jetzt WebP statt JPEG für schnelleres laden (und bessere Bildqualität gleichzeitig). Das bisherige-JPEG-Bild muss aber von Hand konvertiert werden.
• Versionsnummern von (bearbeiteten) Modulen so geändert, dass die erste Zahl immer das Jahr ist
• robots.txt umgeschrieben, um nur Vertrauenwürdige Bots zuzulassen - und gleichzeitig das crawlen von Bildern ganz verboten (da kommen gerade viel zu viele KI-Bots die Daten fürs Training sammeln)
• Den Kassierern Zugriffe auf die gleichen Sachen gegeben, wie den Seniores und Scriptores

Fixes:

• SMTP Versand: Sicherstellung, dass die Portnummer ein integer ist (war aus irgendeinen Grund bei mir nicht)
• SMTP Versand: AutoTLS ausgeschaltet, da bei manchen Servern es nicht funktioniert hat (z.B. solche, die nur TLS 1.0 verstehen)
• composer.json ins v2 Format: konvertiert (das bisherige hat nur mit composer Version 1 funktioniert)
• Viele Pakete aktualisiert und in composer.json ergänzt - u.a. wegen CVE-2021-32610 und anderer Sicherheitslücken, die diese schliessen

Ausserdem habe ich noch eine TODO Liste erstellt mit Paketen, die noch aktualisiert werden müssen (für die der Upgrade-Pfad aber nicht ganz stright-forward ist, leider) und paar Features die ich - GDPL-konform - wieder gerne noch reinbringen würde.

Manche Sachen sind wahrscheinlich noch nicht ganz sauber - und evtl. nicht genau da, wo sie sein sollten. Da müsste ich noch durch die ganzen Sachen mal durch.

Die Verzeichnisse der Pakete, die per Composer installiert werden, könnte man evtl. aus vendor/ entfernen. Habe das (noch) nicht gemacht, da sie ja bisher alle im Repository drinnen waren.

Hast Du Wünsche, Ideen oder Sonstiges, um das Mergen zu können?

[uwol]

Hallo Adrian,

wow, Danke für den Input!

Den Diff muss ich mir einmal in Ruhe anschauen, sobald ich Zeit habe. Den Merge würde ich dann Feature-weise manuell vornehmen, auch um das mengen-mäßig etwas zu reduzieren. Bspw. mit vendor/twbs/bootstrap/docs/examples/... oder vendor/twbs/bootstrap/js/tests/ scheinen Library-Build-Artefakte im Diff zu liegen. Ich bin aber nicht auf dem neusten Stand, was Composer angeht, also ob Tests bzw. allgemein Build-Chain für Libraries bei Composer mit ins Projekt gehören -> muss ich mal recherchieren.

Bis zum Merge sollten keine Merge-Konflikte entstehen, weil ich aktuell nichts am Code ändere. Erst mit PHP 9 kommen vmtl. Änderungen, da PHP mehr Type-Checks erfordert.

[adrianer]

Ja, was die Libraries angeht, tendiere ich die, die mit Composer installiert werden, komplett aus vendor/ im git zu entfernen. Die können ja Problemlos mit Composer lokal installiert werden.

Was PHP/Composer Standards angeht muss ich ehrlich sagen, dass ich da keine Ahnung habe. Ich habe normalerweise mit PHP nichts zu tun (komme aus der Python- und Java-Ecke).