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

Rekursives Sortieren aller JSON-Tableset-Export-Schlüssel #1473

Closed
wants to merge 4 commits into from

Conversation

alxndr-w
Copy link
Contributor

Einfachere Nachverfolgung von Änderungen, wenn das Tableset in einem GitHub-Repository aktualisiert wird (z.B. YForm-basierte Addons)

Einfachere Nachverfolgung von Änderungen, wenn das Tableset in einem GitHub-Repository aktualisiert wird (z.B. YForm-basierte Addons)
@alxndr-w alxndr-w marked this pull request as ready for review February 18, 2024 10:40
@dergel
Copy link
Member

dergel commented Jun 20, 2024

Leider funktioniert das nicht. Ich denke an mehreren Stellen müsste man nochmal prüfen was da los ist,

dann würde ich gerne keine gesonderte Methode haben wollen. Als anonyme Funktion fände ich am besten.

$rksort = function($export) use (&$rksort) { return .. };

@alxndr-w
Copy link
Contributor Author

Korrigiert

@dergel
Copy link
Member

dergel commented Aug 22, 2024

Danke. Ich habe es gerade getestet und es funktioniert. Dabei ist mir nun aufgefallen, dass dadurch die Reihenfolge der ursprünglichen Sortierung der Tabellennamen verloren geht. Also wenn ich mehrere Tabellen exportiere, möchte ich sie nachher eigentlich wieder in dieser Reihenfolge habe. Also ist es wohl sinnvoller, wenn die erste Ebene im JSON nicht sortiert wird, sondern nur die darunter.

@alxndr-w
Copy link
Contributor Author

Ok, schaue ich mir an...

@alxndr-w alxndr-w marked this pull request as draft August 22, 2024 12:36
@alxndr-w alxndr-w marked this pull request as ready for review August 30, 2024 12:10
@alxndr-w
Copy link
Contributor Author

Passt so?

@dergel dergel mentioned this pull request Sep 7, 2024
@dergel
Copy link
Member

dergel commented Sep 7, 2024

hat noch nicht ganz funktioniert, habe einen eigenen PR eingespielt . ist nun drin. #1525

@dergel dergel closed this Sep 7, 2024
@alxndr-w alxndr-w deleted the patch-3 branch September 7, 2024 13:25
@alxndr-w
Copy link
Contributor Author

alxndr-w commented Sep 7, 2024

Danke

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

Successfully merging this pull request may close these issues.

2 participants