search

yakamara / yconverter

👨🏻‍🍳 YConverter - REDAXO Addon
25 stars 7 forks source link

aktuelle Mindestanforderungen #21

Open tbaddade opened 7 years ago

tbaddade commented 7 years ago

Infos

gharlan commented 7 years ago

aktuell nur utf-8

Das müssen wir beachten, sollten wir die REDAXO-Mindestversion irgendwann auf unter 4.5 setzen. (Seit 4.5 gab es nur noch UTF-8.)

gharlan commented 7 years ago

Gerade mit @tbaddade über Skype bisschen überlegt, was welche REDAXO-Version bedeuten würde.

Erstes Ziel sollte sein, die Mindestversion runter zu bekommen auf 4.5. Das ist wahrscheinlich nicht allzu schwer, man muss vor allem hier im Addon auf die rückportierten Klassen wie rex_path verzichten.

Wenn man dann weiter gehen möchte, sollte das nächste Ziel die 4.3.2 sein. (Die 4.4 als Zwischenetappe ist eher unnötig.) Wir müssten dann auch ISO unterstützen. 4.3.2 und nicht 4.3.0, weil vor 4.3.2 die UTF-8-Daten falsch kodiert abgelegt wurden.

Sollte man unter 4.3.2 gehen wollen, müsste man sich mit https://github.com/redaxo/redaxo4/commit/e10559af0c18e6b7dee1ba4f7790333b8c8a75ef und eventuell den Commits kurz davor/danach beschäftigen. Bzw. mit http://www.redaxo.org/de/forum/interner-bereich-f8/utf8-4-3-x-is-broken-t15019.html

Ich denke aber, 4.3.2 wäre schon ein echt gutes Ziel und weiter sollten wir eher nicht gehen. Die Version kam Ende 2010 raus.

gharlan commented 7 years ago

Und ich fände gut, auf PHP 5.3 runter zu gehen.

hirbod commented 7 years ago

@gharlan kann das AddOn rex_path (bzw deine utilities) nicht einfach als Lib mit enthalten? if(class_exists)? Oder nen Versionscheck? Dann muss nicht so viel geändert werden. (Zumindest für 4.5)

Edit: früher hab ich deine Socket-Classes auch immer "mitgeschleppt", bis es dann im Core verfügbar war.

alxndr-w commented 7 years ago

In Slack wurde gemeldet, dass die Daten nicht mit R5.3 kompatibel sind. /cc @domkue