search

JoomGalleryfriends / JG4-dev

Development repository for JoomGallery v4.x
GNU General Public License v3.0
10 stars 6 forks source link

Fix migration errors #214

Closed MrMusic closed 2 months ago

MrMusic commented 3 months ago

Fix error 'data to long...', see https://github.com/JoomGalleryfriends/JG4-dev/issues/208 Fix error 'Return value must be of type array, bool returned', see https://github.com/JoomGalleryfriends/JG4-dev/issues/209

MrMusic commented 3 months ago

Fix wrap if a very large number of IDs are displayed, see https://github.com/JoomGalleryfriends/JG4-dev/issues/205

MrMusic commented 3 months ago

Fix script termination after three sequential errors. If the migration fails after three sequential images, currently the script stops. Thanks to @Elfangor93 for the fix.

Elfangor93 commented 3 months ago

@MrMusic What I proposed was just a check to see where the reason of the problem lies. It shouldnt be fixed like that ..

eumel1602 commented 3 months ago

Ich kann nun auch durch diesen PR eine erfolgreiche Migration von ca 11300 Bildern vermelden. die Bilder wurden bei der Migration NEU erstellt. (nicht kopiert) Wichtig war, dass eine bild zu finden, was 10 MB groß war, das ist irgendwie falsch auf die Quelle gekommen. wichtig war im BE, Fehlerberichten auf "Maximum" zu stellen und dasr meomory_limit auf seinem Server (wenn man es online testet) zu kennen und ggf. hochzuschrauben. auch wenn das im meinem fall mit über 380 m nix gebracht hat... (Ich habe das 0 MB Bild in der Quelle gelöscht) J 4.4.3 / Php 8.1. / auf dem selben Server die migration / Windows /

eumel1602 commented 3 months ago

Ein weiterer Migrationstest von einer JG 3.7rc1 diesmal zu einer J! Version 5.0.3 die PHP Version ist 8.2. Windows Rechner also Online ist POSITIV verlaufen Bilderanzahl: 11631

eumel1602 commented 2 months ago

HAb noch einen test gemacht. Diesmal mit Joomla 5.1 php 8.2. Bilder neu erstellen und Bilder kopieren. Hat ohne Probleme funktioniert.

Elfangor93 commented 2 months ago

@MrMusic can we merge this PR?

MrMusic commented 2 months ago

@MrMusic can we merge this PR?

From my point of view, there are still two small problems that should possibly be solved:

  1. If an image does not have an original, the migration of this image is skipped. Perhaps you could add an option to use the detail image of the source as the original? The problem can only be solved via image mapping if all originals are missing.

  2. Sometimes the tables of the source are renamed several times: ...old_old Perhaps you could prevent the multiple renaming?

Of course we can also address this at a later date.

DE: Aus meiner Sicht bleiben noch zwei kleine Probleme, die evtl. gelöst werden sollten:

  1. Wenn einzelne Bilder kein Original besitzen wird die Migration dieses Bildes übersprungen. Evtl. könnte man eine Option einbauen das Detailbild der Quelle als Original zu verwenden? Nur wenn alle Originale fehlen, lässt sich das Problem über das Image Mapping lösen.

  2. Manchmal werden die Tabellen der Quelle mehrfach umbenannt: ...old_old Evtl. könnte man das mehrfache umbenennen verhindern?

Man kann das natürlich auch zu einem späteren Zeitpunkt nochmal angehen. PS: Dann gibt es noch das Performance-Problem bei der Migration von sehr sehr großen Galerien. Das könnte man beim nächsten Meeting auch ansprechen.