Neues Modul löst ständige Datenbank-Updates aus

[joeker]

Hallo zusammen,

und schön hier zu sein - in euerer Community

Jetzt zu meinem Problem:
Habe ein Modul kreiert, das anhand der DCA-Config die tl_member-Tabelle um einige Felder erweitern und eine weitere Tabelle erzeugen soll.
Jetzt merke ich, dass diese "ALTER table..." queries immerwieder unter "Erweiterungsverwaltung > Datenbank aktualisieren" auftauchen.
Das selbe gilt auch im Install-Tool. Die Felder sind bereits generiert, aber Contao will immer wieder diese "CHANGEN". Beispiel:
"ALTER TABLE `tl_member` CHANGE `contact_person` `contact_person` VARCHAR(255) NULL DEFAULT NULL;"

Was stimmt da nicht oder was muss ich da noch beachten?

Danke schon mal!

[the_scrat]

Deine sql definition im dca ist falsch. Schau einfach wie andere dca Felder ein varchar angeben. Dann sollte das Db update durchgehen


Gesendet von meinem iPhone mit Tapatalk

[joeker]

Die Query, die ich als Beispiel^^ angegeben habe, ist ein Auszug aus Contao. Mein DCA sieht u.a. so aus:

$GLOBALS['TL_DCA']['tl_member']['fields']['contact_person'] = array(
'label' => &$GLOBALS['TL_LANG']['tl_member']['contact_person'],
'exclude' => true,
'search' => true,
'inputType' => 'text',
'eval' => array('feEditable' => true,'feViewable' => true, 'mandatory' => true, 'tl_class'=>'w50'),
'sql' => 'varchar(255) null'
);

DIes hab ich aus aus der tl_member_DCA rauskopiert und umgeschrieben

[the_scrat]

'sql' => 'varchar(255) null'

Ich glaube aber nicht, dass das so in Contoa irgendwo vorkommt. Also kannst du das nicht kopiert haben!

Ändere das ab in:

'sql'                     => "varchar(255) NOT NULL default ''" 

[joeker]

Ja aber, ich möchte dieses Feld nicht als Pflichtfeld erzeugen.
Kann es sein, dass es an der externen SQL-Datei liegt, denn manche Module besitzen von Haus aus im Config-Ordner eine SQL-Datei mit allen DB-Anpassungen, die ebenso in der DCA-Config vorhanden sind?

[the_scrat]

lösch die .sql Datei, das ist eine technik, die in Contao2 verwendet wurde. In Contao 3 wird das SQL nur noch im DCA verwendet.

Und ob ein Feld ein Pflichtfeld ist, oder nicht, entscheidet nicht das sql, sondern der Wert im eval()

[gri]

ohne default value kommt das Update

[the_scrat]

ohne default value kommt das Update

???????????????

[joeker]

Jetzt hab ich das DCA für das eine Feld umgestellt und die SQL-Datei gelöscht:

$GLOBALS['TL_DCA']['tl_member']['fields']['contact_person'] = array(
'label' => &$GLOBALS['TL_LANG']['tl_member']['contact_person'],
'exclude' => true,
'search' => true,
'inputType' => 'text',
'eval' => array('feEditable' => true,'feViewable' => true, 'mandatory' => true, 'tl_class'=>'w50'),
'sql' => 'varchar(255) NOT NULL default ""'
);

Aber das Install-Tool will es immernoch ständig updaten.
Und sorry, es soll doch ein Pflichtfeld sein - die darunter sind es nicht ('mandatory' => false)

[the_scrat]

Sagmal was machst du da? Kopier doch jetzt einfach mal den String von oben!!!!

'sql'                     => "varchar(255) NOT NULL default ''" 

Wieso ist in deinem die Anführungszeichen vertauscht. Außen sind einfache ' und der Defaultwert ist mit doppelten angegeben!

[joeker]

Achso, ich dachte es spielt keine Rolle - zumindest nicht PHP-technisch.
Ich wusste doch nicht, dass Contao das unterscheidet.

Aber was mache ich dann für TEXT-Typen wie hier:

$GLOBALS['TL_DCA']['tl_member']['fields']['company_description'] = array(
'label' => &$GLOBALS['TL_LANG']['tl_member']['company_description'],
'exclude' => true,
'search' => true,
'inputType' => 'textarea',
'eval' => array('rte' => 'tinyMCE'),
'sql' => "TEXT NOT NULL"
);

...wo es keine default-Werte gibt?
...oder bei INT-Typen?

[the_scrat]

Stimmt, PHP technisch ist es egal. Aber in einem CMS gibt es immer bestimmt Vorgaben die eingehalten werden müssen.

Orientiere dich was das sql angeht am besten an den vorhandenen Erweiterungen bzw. dem Core
Du findest die DCA vom Core in /system/modules/core/dca
Such dort einfach nach 'textarea' dann findest du ganz schnell einen Beispielbereich und wie das sql dort angegeben wurde. Halte dich daran, so vermeidest du Fehler.

Hier die Lösung.

'sql'                     => "text NULL" 

[gri]

@ scrat

ich meinte damit wenn beim SQLstring

'sql'                     => "varchar(255) NOT NULL default ''" 

der Defaultwert fehlt......wie im geposteten Beispiel des Threaderstellers.

[joeker]

Ok danke dir vielmals.

Nur trotzdem wundert es mich, wenn ältere Module (die mit 3.5 kompatibel sind) ohne der vorhandenen SQL-Datei auch so einen Loop erzeugen und sich immer wieder kreieren wollen.
Naja, muss man wohl nicht verstehen :P


Danke nochmals

[Spooky]

varchar(255) NULL ist eigentlich korrekt, aber es kann sein dass Contao dies falsch erkennt. Hast du's schon mal mit groß geschriebenem NULL probiert?

[joeker]

Sorry, dass ich so spät erst antworte.

Das Problem waren die großgeschriebennen SQL-Kommandos - hast du also ganz recht.
Conao ist da etwas pingelig - und wie es aussieht case-sensitive was die queries anbetrifft

[Spooky]

Die case sensitivity hängt vom MySQL Server bzw. dessen Einstellungen ab.

[joeker]

Ja aber wieso funktionieren bei mir gewöhnliche Queries mit PHP?
Ich hatte sonst nie Probleme mit Groß- und Kleinschreibung.

[BugBuster]

Weil Contao in dem Fall folgendes tut:
1: er besitzt ein DCA Extractor der die SQL Definitionen da raus nimmt.
2. er liest die Struktur aus der bestehenden Tabelle aus.

Das wird dann verglichen und wenn nicht identisch, dann wird er mit "alter table..." reagieren.

Das war damals bei der database.sql ähnlich, auch da kam es drauf an was groß- und was klein-geschrieben wurde.

[Spooky]

Ja aber wieso funktionieren bei mir gewöhnliche Queries mit PHP?
Ich hatte sonst nie Probleme mit Groß- und Kleinschreibung.

Hm, beschreibe mal ein Beispiel.

[joeker]

@BugBuster:
Danke für die Aufklärung. Ich merk schon - wenn ich immer länger mit Contao mich befasse, lerne ich wie Contao zu denken

@Spooky:
Ein Beispiel wäre zu Beispiel:

\Database::getInstance()->query("ALTER TABLE 'so-und-so' CHANGE COLUMN dings INT(1) NULL DEFAULT 'bums' ");

Da würde Contao schon meckern, dass "default" groß geschrieben ist.

[Spooky]

Ok, eine ALTER TABLE query habe ich his jetzt noch nie über das Datenbank Interface von Contao abgesetzt, wusste nicht, dass dort Contao auch noch irgendwas überprüft.

Was für ein Fehler kommt da genau?

[webstar]

@BugBuster:
Danke für die Aufklärung. Ich merk schon - wenn ich immer länger mit Contao mich befasse, lerne ich wie Contao zu denken

@Spooky:
Ein Beispiel wäre zu Beispiel:

\Database::getInstance()->query("ALTER TABLE 'so-und-so' CHANGE COLUMN dings INT(1) NULL DEFAULT 'bums' ");

Da würde Contao schon meckern, dass "default" groß geschrieben ist.

Nein, würde es nicht. Nur beim Abgleich zwischen den vorhandenen Datenbankdefinitionen und den erforderlichen ist Contao pingelig. Da werden alle Informationen in zwei Arrays geschrieben und miteinander verglichen. Wenn sich etwas unterscheidet, geht Contao davon aus, dass es geändert werden sollte. Es "normalisiert" dabei nicht die Informationen, die der Entwickler im DCA/database.sql hinterlegt.

[joeker]

Wenn wir schon bei Datenbanken sind - wie sage ich Contao anhand meines Models in meinem Custom-Modul, dass auf andere Tabellen beziehen soll?
Oder wird es ausschließlich mit DCA realisiert - und wenn ja wie?

[zinky]

Per DCA; So: http://de.contaowiki.org/Models#Rela..._Fremdtabellen / https://docs.contao.org/books/cookbo...atens%C3%A4tze

[joeker]

Ja das habe ich mir schon gedacht und ausprobiert.
Bei 1:n Beziehungen geht es wudnerbar - doch bei m:n wird es etwas tricky und leicht undurchsichtig, da man immer eine "pid"-Spalte in jeder Tabelle haben muss und Contao keine "direkte" SQL-ForeignKey Beziehung aufbaut.

Da musste ich mich behelfen und Custom-Methoden in mein Model einbauen, um diese per InnerJoin zu koppeln.

[zinky]

Schau dir dazu am Besten mal die Extension Haste an. Die unterstützt auch m:n-Beziehungen

[joeker]

Ok danke. Ich wusste gar nicht, dass Haste so etwas kann. Und das, obwohl ich das Ding bei eingerichtet habe :P

[s3lf]

Nein, würde es nicht. Nur beim Abgleich zwischen den vorhandenen Datenbankdefinitionen und den erforderlichen ist Contao pingelig. Da werden alle Informationen in zwei Arrays geschrieben und miteinander verglichen. Wenn sich etwas unterscheidet, geht Contao davon aus, dass es geändert werden sollte. Es "normalisiert" dabei nicht die Informationen, die der Entwickler im DCA/database.sql hinterlegt.

Tipp: In \Contao\Database\Installer::compileCommands die Variable $sql_current ausgeben nachdem man das DB Update gemacht hat - da kann man ersehen, welches Format Contao erwartet.