search

yakamara / yconverter

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

#25 + #26 fix YCom Anpassungen #28

Closed TobiasKrais closed 6 years ago

TobiasKrais commented 6 years ago

Erste Lösung war doch voreilig. Neue Lösung ist gestestet und funktioniert

TobiasKrais commented 6 years ago

Jetzt müssten die YCom Änderungen vollständig sein.

TobiasKrais commented 6 years ago

So, alles jetzt nochmals durchlaufen lassen, getestet und funktioniert. Bitte Mergen.

TobiasKrais commented 6 years ago

OK. Wie gewünscht angepasst und alles einzeln getestet. Funktioniert alles.

TobiasKrais commented 6 years ago

Die Zeile ['password\|password\|' => 'ycom_auth_password|password|'], muss nach der Zeile mit dem com_auth_password_hash eingefügt werden. Das sollte lieber so bleiben.

Auch würde ich die Zeile nicht in den XForm Teil rein nehmen, da es im XForm ja auch das password Feld gibt und hier nur die Ersetzung des Beispielcodes verwendet wird um möglichst viele Fälle abzudecken.

Den anderen Vorschlag von dir habe ich commited.

tbaddade commented 6 years ago

Die Zeile ['password|password|' => 'ycom_auth_password|password|'], muss nach der Zeile mit dem com_auth_password_hash eingefügt werden. Das sollte lieber so bleiben.

Warum muss das genau so bleiben?

Auch würde ich die Zeile nicht in den XForm Teil rein nehmen, da es im XForm ja auch das password Feld gibt und hier nur die Ersetzung des Beispielcodes verwendet wird um möglichst viele Fälle abzudecken.

Verstehe nicht ganz. Welchen Beispielcode? Kannst du aus deiner 4er Instanz die Zeilen die du im Builder hast posten. Vielleicht wird es dann klarer.

TobiasKrais commented 6 years ago

OK. Optionales ? ist mit drin. Gute Idee. Danke für deinen Support. Wie du merkst bin ich kein sonderlicher Experte für Regular Expressions...

Hier ein Beispiel für das Passwort ändern Formular in Redaxo 4. Es is abgesehen von den XOutputFilter Platzhaltern das vorgeschlagene Original:

password|password|###passwort_neu###
password|password_2|###passwort_wiederholen###||no_db
validate|empty|password|###passwort_fehler_leer###
validate|empty|password_2|###passwort_fehler_leer###
validate|compare|password|password_2|!=|###passwort_fehler_ungleich###
action|showtext||<div class="xform"><ul class="form_info"><li>###passwort_neu_aktiv###</li></ul></div>||1
com_auth_password_hash|password|password|
action|com_auth_db

Hier siehst du auch schon, warum die Zeile com_auth_password_hash|password|password| vor password|password| ersetzt werden sollte, da sonst die com_auth_password_hash|password|password| Zeile verhagelt wird.

Die XForm Doku sagt zum Passwort Feld folgendes: password -> Beispiel: password|name|label|default_value|[no_db]

Die YForm Doku in Redaxo 5 sagt zum Passwort Feld genau das Gleiche, nur dass es von YCom ein nun eigenes Passwort Feld gibt. Hier die Doku: ycom_auth_password -> Beispiel: ycom_auth_password|name|label|[password-rules-as-json]|message|[script 0/1] Jetzt soll ein ehemaliges XForm Passwort Feld entweder so bleiben wie es ist - wenn es für ein normales YForm weiterverwendet wird. Oder es soll durch das neue Passwort Feld von YCom ersetzt werden. Woher kann man wissen, um welche Verwendung es sich handelt? Die Chance liegt am höchsten, wenn ich die Ersetzung des exakten Comunity Addon Beispielcodes vornehme. Weißt du, wie ich meine?

TobiasKrais commented 6 years ago

Hi Thomas, wie sieht's aus? Möchtest du noch Änderungen oder kann der Code gemerged werden?

tbaddade commented 6 years ago

Der PR ist jetzt hier mit eingeflossen https://github.com/yakamara/yconverter/commit/ce334b251b8c25cf7d699a9f348a493a3214a897 https://github.com/yakamara/yconverter/commit/7f10e0c9660ddf5deb0a97fd67335a568a9ecc6a

TobiasKrais commented 6 years ago

Danke!