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

MetaInfo - Media - Standardfelder erzeugt in der Migrationsdatei einen SQL Fehler #39

Open
tbaddade opened this issue Jul 31, 2019 · 5 comments
Labels

Comments

@tbaddade
Copy link
Member

tbaddade commented Jul 31, 2019

Generierte Migration

rex_sql_table::get('rex_media')
        ->ensureColumn(new rex_sql_column('med_description', 'text', true, '\'\'', null), 'med_focuspoint')
        ->ensureColumn(new rex_sql_column('med_copyright', 'text', true, '\'\'', null), 'med_description')
        ->alter();

Dies Part '\'\'' schlägt fehl

Lösung: Migrationsdatei direkt anpassen und nochmals deployen.

@gharlan
Copy link
Member

gharlan commented Feb 12, 2020

Das Grundproblem wurde im metainfo-Addon behoben: redaxo/redaxo#3270

Wenn das Problem bei schon bestehenden Seiten auftritt:

  • Default-Wert in den Spalten entfernen, bzw. auf NULL setzen
  • Nochmal Diffen
  • So sollte es sich dann auf allen Instanzen normalisieren und angleichen (egal ob MySQL oder MariaDB)

@tbaddade
Copy link
Member Author

Wir hatten hier eben dieses Problem

Via YForm hatte Robert ein Feld angelegt. Nach dem Diff erschien das in der Migration:

 ->ensureColumn(new rex_sql_column('statement', 'text', false, '\'\'', null), 'search')

Nach meiner Migration und einem weitere Diff erschien das in einer neuen MIgrationsdatei:

->ensureColumn(new rex_sql_column('statement', 'text', false, '\'\\\'\\\'\'', null))

@gharlan
Copy link
Member

gharlan commented Mar 20, 2022

Ich habe mir das nochmal angeschaut. Es ist denke ich doch ein Bug im Core in rex_sql_table, aber gleichzeitig leider schwer zu fixen ohne BC-Probleme.

Bin mir noch nicht sicher, was ich da mache.

Solange das Problem besteht, gilt sinngemäß was ich oben für die Metafelder geschrieben habe. Also am besten den Default-Wert von den TEXT-Feldern entfernen.

@alxndr-w
Copy link
Contributor

alxndr-w commented Apr 9, 2023

Mir ist das in der letzten Zeit nicht mehr begegnet, dir @tyrant88 ? Lag das an den unterschiedlichen MySQL/MariaDB-Versionen die wir inzwischen wieder angeglichen haben?

@tyrant88
Copy link
Contributor

tyrant88 commented Apr 9, 2023

Es passierte ja bei uns auch, weil du mit MariaDB neue Felder angelegt hattest. Das hast du wohl nichit mehr getan seitdem. Oder eben kein MariaDB mehr benutzt in deinen Projekten. Oder immer mit Maria auch bei deinem Hoster.

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

No branches or pull requests

4 participants