search

FriendsOfREDAXO / focuspoint

Erweitert den Medienpool um die Fähigkeit, den Fokuspunkt eines Bildes zu bestimmen.
MIT License
58 stars 7 forks source link

Code-Verbesserungen nach Analyse mit rexstan #112

Closed christophboecker closed 2 years ago

christophboecker commented 2 years ago

Der PR bassiert auf den mit rexstan (phpstan) durchgeführten Code-Analysen bis Stufe 6.

Benachrichtigungen, die nicht in Focuspoint behoben werden können, sind auf @phpstan-ignore-next-line gesetzt. Drei weitere, deren Ursache mir nicht klar ist, sind weiterhin unbearbeitet.

Eine weiterer Anlauf, den Cache-Bug aus #110 zu lösen, ist ebenfalls enhalten (positiv getestet mit ritzfritz).

Alten Code nur für PHP 5.6 entfernt, da Focuspoint mittlerweile PHP 7.3 voraussetzt.

Und leider ist auch das Update der help.php hier mit reingelaufen statt in einen separaten PR.

staabm commented 2 years ago

Falls du am redaxo tag in mainz bist: ggf. Macht es sinn dort mit ein paar addon entwicklern eine session zu machen um gemeinsam mal rexstan fehler meldungen und mögliche lösungen durchzugehen

aber prinzipiell hast du hier schon gut was abgearbeitet - klasse!

christophboecker commented 2 years ago

Falls du am redaxo tag in mainz bist: ggf. Macht es sinn dort mit ein paar addon entwicklern eine session zu machen um gemeinsam mal rexstan fehler meldungen und mögliche lösungen durchzugehen

gerne, bin dabei.