-
-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Umstellung Status "DELETED" von 2 auf -2 #110
Conversation
Könntest du hier über die Collection gehen, damit das Speichern über YOrm passiert und ggf. auch EPs korrekt getriggert werden. Collections haben eine gemeinsame setValue() und save() Methoden. Während ich das schreibe, wird mir allerdings bewusst, dass das Speichern verweigert würde (Wert nicht im Choice verfügbar). Evtl also doch per SQL und dann im Nachgang alle mit -2 nochmal mit save() behandeln. |
Dann bist Du ja wieder beim Save(), im dem immer noch die alte Klasse mit dem "falschen" choice herangezogen wird. Wie sieht denn das Risiko aus? Vermutung meinerseits: Wenn schon jemand Zusatzaktionen per EP an den Wechsel des Status auf DELETED gehängt hat, sind die durch und müssen nicht nochmal ablaufen, nur weil DELETED repräsentierende Wert nun ein anderer ist. Und falls jemand seinen Code auf "2" aufgebaut hat, muss er den eh ändern; egal ob Yorm oder rex_sql. DA würde auch ein erneuter Lauf nix ändern. |
Das URL-Profil würde bspw. noch generierte URLs haben für "2", obwohl inzwischen "-2". Die falschen URLs sollten ja auch aus der sitemap.xml verschwinden. Ein möglicher Ablauf müsste sein:
Oder
|
Ich wundere mich, wie das passeieren konnte. Da war ich schon mal dran und habe das eigentlich überprüft und korrigiert. Da muss was mit den Tablesets nicht stimmen. |
Fragt sich wie? Eine Änderung in der tableset.json findet nicht zwangsläufig den Weg in die rex_yform_field-Tabelle. |
Eigentlich über die update.php, die die install.php aufruft, die die Tablesets frisch "drüberbügelt" (wie YCom) |
Danke für´s Mergen, aber was ist mit Punk 3 Url-Cache löschen? Das fehlt noch. |
Der Status-Code für DELETED steht z.Zt. irrtümlich auf 2 statt -2 (#101 (comment)).
der PR fügt in die install.php Code ein, der beim erstmaligen Update auf eine Version 5.1 oder größer den Wert in der Datenbank auf -2 korrigiert. Analog dazu ist Entry angepasst.