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

Uninstall-Umfang auswählen #73

Merged
merged 2 commits into from
Sep 25, 2024
Merged

Conversation

christophboecker
Copy link
Contributor

Ich bin drüber gestolpert, dass nach dem Uninstall die Datenbanken noch immer vorhanden waren. Intuitiv erwarte ich eigentlich, dass alles abgeräumt wird. Wobei es mitunter auch sinnvoll sein kann, das eben nicht zu tun.

Wie auch immer: man kann ja wie schon beim Addon "neues" eine Lösung implementieren, bei der im einfachen Fall die Tabellen erhalten bleiben und im umfassenden Fall auch die Tabellen entfernt werden.

Löschen der Config-Einträge ist bei der Gelegenheit entfernt weil überflüssig, denn das macht Redaxo ohnehin.

Auch die Abfrage auf YForm ist überflüssig, da das Addon YForm bei der Installation voraussetzt und damit die Interdependenzen von Redaxo verwaltet werden.

analog zum Uninstall bei "neues"
@alxndr-w
Copy link
Member

Auch die Abfrage auf YForm ist überflüssig, da das Addon YForm bei der Installation voraussetzt und damit die Interdependenzen von Redaxo verwaltet werden.

Ich meine, das habe ich drin, um eine Deinstallation im Safe Mode ohne Whooops zu ermöglichen.

Hast du das mit bedacht / kannst das testen?

Safe Mode Aktionen sollten immer möglich sein.

@christophboecker
Copy link
Contributor Author

Dann hast Du aber (wenn man im Safemode ist) faktisch nur eine rudimentäre Deinstallation. Ist das im Sinne des Erfinders? Also ware die "richtige" Lösung, die Einträge aus den rex_yform_Tabellen zu löschen, das YForm-API zu ignorieren und statt dessen mit rex_sql die Einträge zum jeweiligen Tabellennamen zu entfernen.

@alxndr-w
Copy link
Member

Die rudimentäre Deinstallation im Safe Mode hätte ich inkauf genommen. Man kann ja die Tablesets immer noch anschließend manuell entfernen.

@christophboecker
Copy link
Contributor Author

Na dann: im SafeMode nur die Basis-Deinstallation. Tabellen und Tablesets bleiben.

@alxndr-w alxndr-w merged commit f7c8ef2 into alexplusde:main Sep 25, 2024
1 check failed
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