Fehler bei Wiederherstellung von Backup, egw 17.1.007, mysql -> egw 19.1.20191220, postgres

[GKlimm]

Welche Version (aus Admin/Änderungslog und Versionen)?
19.1.20191220

Installationsart (Paketinstallation, Archivinstallation oder git)?
Dockerimage (egroupware:latest)

Welches Modul?
Setup

Tritt der Fehler nach einem Update auf?
Ist eine Neuinstallation, Wiederherstellen eines Backups im Setup (Schritt 5).

Sind Hinweise im Web-Server-Error-Log zu finden?
Postgres-DB:

2020-01-12 10:40:04.599 UTC [4209] ERROR: date/time field value out of range: “0000-00-00 00:00:00” at character 204
2020-01-12 10:40:04.599 UTC [4209] STATEMENT: INSERT INTO egw_cal_user (cal_id,cal_recur_date,cal_user_type,cal_user_id,cal_status,cal_quantity,cal_
role,cal_user_modified,cal_user_auto,cal_user_attendee) VALUES (46,0,‘u’,‘5’,‘A’,1,‘REQ-PARTICIPANT’,‘0000-00-00 00:00:00’,48,NULL)

EGW:

2020-01-12 10:40:04.656 UTC [4209] ERROR: current transaction is aborted, commands ignored until end of transaction block

Welche Datenbank wird verwendet (MySQL/MariaDB oder PostgreSQL)?
Postgresql 10.11

Welcher Webserver wird verwendet (Apache oder nginx)?
nginx (dockerimage, nginx:latest)

Welche PHP-Version wird verwendet?
die im egroupware:latest

Scheint so, dass die backups nicht über DB-Typen kompatibel sind. Bei postgres ist ‘0000-00-00 00:00:00’ nicht erlaubt, dafür müsste NULL eingefügt werden.
Das Ersetzen im Backup habe ich schon probiert, auch das Kopieren mit pgloader (dann kam im Setup die Meldung, dass die egw-version in der DB auf 17.1 ist und versucht wird, sie zu updaten, hat dann Fehler geschmissen, dass iwelche. Spalten nicht existierten, log habe ich dazu leider nicht mehr, wenn Bedarf besteht, kann ich es aber nochmal probieren und das Ergebnis posten).

Wie kriege ich also das Backup einer mysql DB in eine egw Instanz mit postgres rein?

MfG, David

[RalfBecker]

Der folgende Patch sollte helfen:

RalfsMac:egroupware ralf$ git diff api/src/Db.php
diff --git a/api/src/Db.php b/api/src/Db.php
index 74616531b1..cf2ed6312f 100644
--- a/api/src/Db.php
+++ b/api/src/Db.php
@@ -1468,6 +1468,11 @@ class Db
                                }
                                break;  // handled like strings
                        case 'date':
+                               // insert valid NULL instead of old MySQL value (eg. in Db\Backup::restore)
+                               if (is_string($value) && substr($value, 10) === '0000-00-00')
+                               {
+                                       return NULL;
+                               }
                                // if DateTime object given, convert it (NOT converting the timezone!)
                                if (is_object($value) && ($value instanceof \DateTime))
                                {

Gib’ kurz Rückmeldung, dann commite ich das.

Ralf

[GKlimm]

Danke für die schnelle Antwort!

Funktioniert leider nicht , es kommt immernoch der gleiche Fehler (ja, ich habs in der egroupware-sources geändert, da die /usr/share/egroupware durch den entrypoint bei jedem restart zurückgesetzt wird).
Hier mal ein Ausschnitt aus dem Backup:

46,1301155200,“u”,“5”,“A”,1,“REQ-PARTICIPANT”,0000-00-00 00:00:00,1425,NULL

scheint mir so, als ob dass nicht als string im backup gespeichert wird?

[RalfBecker]

Sorry, versuche das mal:

diff --git a/api/src/Db.php b/api/src/Db.php
index 74616531b1..b68d2133ff 100644
--- a/api/src/Db.php
+++ b/api/src/Db.php
@@ -1468,6 +1468,11 @@ class Db
                                }
                                break;  // handled like strings
                        case 'date':
+                               // insert valid NULL instead of old MySQL value (eg. in Db\Backup::restore)
+                               if ($value === '0000-00-00')
+                               {
+                                       return 'NULL';
+                               }
                                // if DateTime object given, convert it (NOT converting the timezone!)
                                if (is_object($value) && ($value instanceof \DateTime))
                                {
@@ -1475,6 +1480,11 @@ class Db
                                }
                                return $this->Link_ID->DBDate($value);
                        case 'timestamp':
+                               // insert valid NULL instead of old MySQL value (eg. in Db\Backup::restore)
+                               if ($value === '0000-00-00 00:00:00')
+                               {
+                                       return 'NULL';
+                               }
                                // if DateTime object given, convert it (NOT converting the timezone!)
                                if (is_object($value) && ($value instanceof \DateTime))
                                {

Ralf

[GKlimm]

Hallo Ralf!

Also wenn ich das ‘NULL’ nehme, dann wird noch wegen dem not null contraint gemeckert.
Aber nehme ich den default, also ‘CURRENT_TIMESTAMP’, geht es natürlich.

Dürfte ja egal sein, ob da jetzt ein stamp drin ist, oder?