search

mittwald / feature-requests

Sammlung von Feature-Ideen.
https://www.mittwald.de/roadmap
14 stars 0 forks source link

mStudio: App löschen ≠ verknüpfte DB löschen #168

Open paul-hph opened 2 weeks ago

paul-hph commented 2 weeks ago

Welches Problem möchtest du lösen? Wann tritt es auf? Wenn zwei Apps mit der gleichen Datenbank verknüpft sind und ich ein der Apps lösche, erhalte ich den Hinweis:

Folgende Verknüpfte Konponenten werden automatisch mit gelöscht: Datenbanken

Problem 1: Ich sehe nicht, welche DB mitgelöscht wird. Problem 2: Wenn mehre Apps mit der DB verknüpft wird, wird die DB tatsächlich nicht gelöscht. Das kann zu Probleme/Verwirrungen führen, je nachdem was man denn will.

Welche Lösungsideen hast du?

patrickhilker commented 2 weeks ago

Hey Paul, kannst du mir ein paar Hintergründe erklären:

Inwiefern ist es irritierend, welche Datenbank gelöscht wird? In der Regel haben die üblicherweise genutzten CMS ja genau eine Datenbank. Da hätte ich erwartet, dass der Name beim Löschen egal sein dürfte. Oder stecken da noch relevante Infos drin? Brauchst du noch mehr, als den Namen bzw. die Beschreibung?

Datenbanken, die noch mit anderen Apps verknüpft sind, löschen wir bewusst nicht - sonst wäre die andere App ja mit ziemlicher Sicherheit kaputt. Darum finde ich das auch eher weniger verwirrend.

Die Datenbanken generell nur optional zu löschen finde ich nachvollziehbar. Eventuell möchte man die ja aus Archiv-Gründen o.ä. noch behalten.

paul-hph commented 1 week ago

Inwiefern ist es irritierend, welche Datenbank gelöscht wird? In der Regel haben die üblicherweise genutzten CMS ja genau eine Datenbank. Da hätte ich erwartet, dass der Name beim Löschen egal sein dürfte. Oder stecken da noch relevante Infos drin? Brauchst du noch mehr, als den Namen bzw. die Beschreibung?

Du hast recht, in der Regel ist immer eine App mit einer DB verknüpf. Ich weiß nicht wie es bei anderen ist, aber bei uns gibt es auch viele Ausnahmen. Wahrscheinlich, weil auch unsauber gearbeitet wird bei Updates/Kopien/Duplikaten. Gerne gibt es mal verschiedene Datenbanken und Versionen für das gleiche Projekt. Beispiele:

Beschreibung:TYPO3 CMS DB Beschreibung:TYPO3 CMS DB BACKUP Beschreibung:TYPO3 CMS DB 01.04.203 Beschreibung:TYPO3 CMS DB V11 Beschreibung:TYPO3 CMS DB V9 Beschreibung:TYPO3 CMS DB Master Beschreibung:TYPO3 CMS DB Live Beschreibung:TYPO3 CMS DB asdasdas test 12

Ich denke, wenn ihr die ID und die Beschreibung der verknüpften DB beim App löschen mit anzeigt, reicht das schon aus.

Konkretes Beispiel, dass ich gestern hatte. Ich habe eine App in einem Projekt aktualisieren wollen:

  1. Ich habe dazu einen neue App mit dem Update angelegt
  2. Dann die Live-DB mit der neuen App verbunden zu testen, ob alles läuft.
  3. Anschließend habe ich die neue App zur Live-App gemacht und die alte App gelöscht. Zum Glück wurde dadurch die verbundene Live-DB nicht mitgelöscht.

Datenbanken, die noch mit anderen Apps verknüpft sind, löschen wir bewusst nicht - sonst wäre die andere App ja mit ziemlicher Sicherheit kaputt. Darum finde ich das auch eher weniger verwirrend.

Das finde ich auch richtig und wichtig so. Dann sollte aber der Hinweis in mStudio dann auch entsprechend so sein. Aktuell steht da ja, dass die DB gelöscht wird.

Die Datenbanken generell nur optional zu löschen, finde ich nachvollziehbar. Eventuell möchte man die ja aus Archiv-Gründen o.ä. noch behalten.

Auch in bin ein Freund der optionalen Löschung. Siehe auch https://github.com/mittwald/feature-requests/issues/164 :-D