Skip to content
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

Merged
merged 2 commits into from
Sep 10, 2024
Merged

Umstellung Status "DELETED" von 2 auf -2 #110

merged 2 commits into from
Sep 10, 2024

Conversation

christophboecker
Copy link
Member

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.

@alxndr-w
Copy link
Member

alxndr-w commented Sep 9, 2024

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.

@christophboecker
Copy link
Member Author

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.

@alxndr-w
Copy link
Member

alxndr-w commented Sep 9, 2024

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:

  1. Tableset aktualisieren
  2. Datensätze mit 2 auf -2 ändern
  3. Dabei EPs triggern.

Oder

  1. Datensätze mit 2 auf -2 ändern per SQL
  2. Tableset aktualisieren
  3. Url-Cache löschen

@christophboecker
Copy link
Member Author

Lass uns hier mal stoppen, bitte. Da müssen wohl noch mehr Sahen in den Felddefinitionen angegangen werden. Z.B sehe ich grade dies hier:
grafik
Gleicher Zweck andere Feldtypen und andere DB-Feldtypen (text für nen int-Wert ist etwas overdone; passiert aber schon mal). So was müsste man alles auf einen Schlag in einem Release erledigen.

@alxndr-w
Copy link
Member

alxndr-w commented Sep 9, 2024

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.

@christophboecker
Copy link
Member Author

Fragt sich wie? Eine Änderung in der tableset.json findet nicht zwangsläufig den Weg in die rex_yform_field-Tabelle.

@alxndr-w
Copy link
Member

Eigentlich über die update.php, die die install.php aufruft, die die Tablesets frisch "drüberbügelt" (wie YCom)

@alxndr-w alxndr-w merged commit 87cf430 into main Sep 10, 2024
@alxndr-w alxndr-w deleted the cb-16 branch September 10, 2024 11:03
@christophboecker
Copy link
Member Author

Danke für´s Mergen, aber was ist mit Punk 3 Url-Cache löschen? Das fehlt noch.

install.php Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants