ycom icon indicating copy to clipboard operation
ycom copied to clipboard

ycom_auth_load_user ist zerhackt ;-)

Open marcohanke opened this issue 3 years ago • 1 comments

Wenn ich ycom_auth_load_user mit Pipes aufrufe werden die Daten zwar geladen, dumpe ich aber die Klasse, sind die abgefragen values als Label gespeichert uns das label als Klasse Bildschirmfoto 2021-09-08 um 11 53 21

marcohanke avatar Sep 08 '21 09:09 marcohanke

Ersteres ist wohl bei allen yform-values so, die den zweiten Parameter nicht als label nutzen. Weil das hier https://github.com/yakamara/redaxo_yform/blob/master/lib/yform/value/abstract.php#L200 gesetzt wird. Zweiteres kann ich nicht nachvollziehen.

tyrant88 avatar Dec 01 '21 18:12 tyrant88

Ich sehe hier auch kein Label als Klasse.

Hier ist nix zerhackt, nur ein Fleißbienchen-Stempel gäbe es vermutlich dafür, wenn nicht der 2. Parameter, sondern der 3. Parameter hier ausgelesen würde.

https://github.com/yakamara/redaxo_ycom/blob/d5b10b1b155038589cc6d3a7ec7defd7e2a20fc5/plugins/auth/lib/yform/value/ycom_auth_load_user.php#L8-L13

Das wäre dann nicht rückwärtskompatibel und man müsste einen leeren Pipe-Parameter hinzufügen - ich wüsste nicht, ob das ein Gewinn wäre.

M.E. kann man hier zumachen.

alxndr-w avatar Sep 01 '22 06:09 alxndr-w

sieht für mich auch ok aus.

dergel avatar Sep 27 '22 13:09 dergel