Jefferson49 / ExtendedImportExport

A webtrees 2.1 custom module for advanced GEDCOM import, export ,and filter operations. The module also supports remote downloads/uploads via URL requests.
GNU General Public License v3.0
5 stars 0 forks source link

Der Export kann nicht auf dem Server erfolgen #17

Closed WeT-Klb closed 3 months ago

WeT-Klb commented 3 months ago

Ich hatte bislang das "alte" Modul am laufen. Heute habe ich nun die neue Version installiert, kann aber seitdem die Sicherung nicht mehr am Server ablegen. Wenn ich z.B.

https://XXXXXXXXXX/webtrees/index.php?route=/webtrees/ExtendedImportExport&tree=tree1&key=xxxxxxxxx

aufrufe, wird die GEDCOM-Datei heruntergeladen und nicht wie bisher unter

data/media

abgelegt. Die Einstellungen wurden vom alten Modul eingelesen und übernommen. Ich habe auch alle 4 Checkboxen für die Remote-Abfragen über URL angeklickt und eine gelbe und eine rote Warnung deshalb aktiv.

Hat sich der Aufruf zum serverbasierten Speichern geändert? Wenn ja, wie kann ich angeben, daß der Export nach "data/media" erfolgen soll?

I had the ‘old’ module running until now. Today I installed the new version, but since then I can no longer store the backup on the server. For example, if I use

https://XXXXXXXXXX/webtrees/index.php?route=/webtrees/ExtendedImportExport&tree=tree1&key=xxxxxxxxx

for example, the GEDCOM file is downloaded and not saved under

data/media

as before. The settings were read in from the old module and applied. I have also ticked all 4 checkboxes for the remote queries via URL and a yellow and a red warning are therefore active.

Has the call for server-based saving changed? If so, how can I specify that the export should be to ‘data/media’?

Screen 1

Screen 2

Screen 3

Screen 4

Screen 5

Jefferson49 commented 3 months ago

Hallo Werner,

danke, für die schnellen Rückmeldungen zur neuen Version.

Habe mir gerade den Code zu der Thematik "Speichern auf dem Server" angesehen. Früher gab es hier eine Default-Einstellung im Control Panel. Ich vermute, dass Du bisher "Save" als Default eingestellt hattest. Dann wurde "&action=save'" angenommen, selbst wenn in der URL nichts angegeben ist.

setting_action

Im neuen Modul gibt es nun noch viel mehr Aktionen, u.a. import und convert. Ich habe hier Sorgen, dass man durch einfache Verwechslungen einen Import auslösen kann; dieser ist ja möglicherweise folgenreich und überschreibt Daten. Daher habe ich die Einstellung für die Default-Action jetzt herausgenommen. Generell sollte jetzt immer im URL-Aufruf die Action explizit angegeben werden. Wenn nicht, wird jetzt immer "download" als Default genommen; weil unkritisch.

Für Deinen Fall musst Du also nun "&action=save" an Deine Aufrufe anhängen. Dann sollte es funktionieren.

Eigentlich wollte ich ja für Kompatibilität der alten URLs sorgen. Aber hier ist es jetzt unglücklich. Schreib mir gerne Deine Meinung, was Du ggfs. vorschlagen würdest. Ich muss nochmal drüber nachdenken.

WeT-Klb commented 3 months ago

Hallo Jefferson,

auf jeden Fall funktioniert es jetzt. Eigentlich ist deine Überlegung ja richtig, aber vielleicht sollte man unter "Update" nocheinmal darauf hinweisen, daß sich der Aufruf geändert hat. In meinem Fall hatte ich ursprünglich weder action=save noch tree=tree1 angeben müssen - und alles hat funktioniert. Jetzt musste ich den Aufrug halt entsprechend anpassen, was ja kein Problem ist, wenn man es weiß.

Eventuell solltest Du auch bei den Beispielen der Aufrufe noch ein bisschen nachschärfen (https://github.com/Jefferson49/ExtendedImportExport?tab=readme-ov-file#example-urls-for-downloadsaving): Wäre vielleicht nicht verkehrt, da Beispiele für Sicherung auf dem Server (action=save) anzugeben. Die Beispiele haben mich z.B. eher verwirrt.

Was auch anders ist: Das alte Modul tauchte rechts unter "Andere" auf - jetzt versteckt es sich nur noch oben linke unter "Alle Module". Läßt sich das ändern?

Screen

Jefferson49 commented 3 months ago

vielleicht sollte man unter "Update" nocheinmal darauf hinweisen, daß sich der Aufruf geändert hat

Guter Vorschlag! Habe jetzt noch ein entsprechendes Kapitel ergänzt: Upgrading remote URL requests from the former DownloadGedcomWithURL module

Wäre vielleicht nicht verkehrt, da Beispiele für Sicherung auf dem Server (action=save) anzugeben.

Habe ich jetzt ebenfalls entsprechend abgeändert: Example URLs for Remote Requests

Das alte Modul tauchte rechts unter "Andere" auf - jetzt versteckt es sich nur noch oben linke unter "Alle Module". Läßt sich das ändern?

Nein leider nicht. Habe eben überprüft, wieso es sich geändert hat. Der Grund ist, dass ExtendedImportExport nun im Listen-Menü einen Menü-Eintrag erhalten hat, siehe Screenshot unten. Im Control Panel zeigt webtrees unter "Sonstige" nur diejenigen Module an, die sonst nirgends im User Interface zu sehen sind.

Das neue Menü könnte aber ggfs. auch für Deine Zwecke passen. Man kann dort alle Filter-Operationen ausführen, jedoch nicht die Modul-Grundeinstellungen ändern; Letzteres nur als Admin im - nun tiefer versteckten - Control Panel.

screenshot_open_user_views_from_list_menu

Jefferson49 commented 3 months ago

Das alte Modul tauchte rechts unter "Andere" auf - jetzt versteckt es sich nur noch oben linke unter "Alle Module". Läßt sich das ändern?

Mir ist jetzt noch eine Möglichkeit eingefallen: Habe nun einen Link zu den Einstellungen (bzw. Control Panel) bei der Auswahl der Filtermöglichkeiten ergänzt. Ist dann in einer nächsten Version enthalten.

preferencdes

Man kann dann mit 2 Clicks zu den Einstellungen navigieren:

WeT-Klb commented 2 months ago

Hi,

das ist ne gute Idee!

Von: Jefferson49 @.*** Gesendet: Mittwoch, 28. August 2024 22:32 An: Jefferson49/ExtendedImportExport Cc: Werner; Author Betreff: Re: [Jefferson49/ExtendedImportExport] Der Export kann nicht auf dem Server erfolgen (Issue #17)

Das alte Modul tauchte rechts unter "Andere" auf - jetzt versteckt es sich nur noch oben linke unter "Alle Module". Läßt sich das ändern?

Mir ist jetzt noch eine Möglichkeit eingefallen: Habe nun einen Link zu den Einstellungen (bzw. Control Panel) bei der Auswahl der Filtermöglichkeiten ergänzt. Ist dann in einer nächsten Version enthalten.

preferencdes.jpg (view on web) https://github.com/user-attachments/assets/0435db1a-f8a7-4964-aade-fafb57e0 add7

Man kann dann mit 2 Clicks zu den Einstellungen navigieren:

— Reply to this email directly, view it on GitHub https://github.com/Jefferson49/ExtendedImportExport/issues/17#issuecomment- 2316198979 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AE5FMFQG6NSK5O4SIVGAXMLZT YXTJAVCNFSM6AAAAABNBY4NZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJWGE4TQ OJXHE . You are receiving this because you authored the thread. https://github.com/notifications/beacon/AE5FMFWF4ND6INSGVROUFQDZTYXTJA5CNFS M6AAAAABNBY4NZGWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTU KBZSEG.gif Message ID: @.***>