search

KarelZe / DHBW_MWI

MWI Projekt :mortar_board:
0 stars 0 forks source link

Spiel initialisieren - Mehrere Fehler #11

Closed MrApplebyte closed 5 years ago

MrApplebyte commented 5 years ago
  1. Unternehmen: Leere Unternehmensnamen werden nicht abgefangen (Fehler: Abort due to constraint violation (NOT NULL constraint failed: Wertpapier.unternehmen_id))
  2. Spiel kann ohne Aktien bzw. Anleihen initialisiert werden
  3. Es können mehrere Aktien/Anleihen für ein Unternehmen angelegt werden
  4. Beim Klicken auf den "-"-Button (Lösch-Button) wird nicht die entsprechende Zeile sondern die oberste Zeile entfernt
  5. Fehler bei Anlegen der Aktien/Anleihen (wahrscheinlich durch gleichzeitiges Anlegen): " The database file is locked (database is locked)"
KarelZe commented 5 years ago
  1. Ist noch zu implementieren (Wie besprochen).
  2. @jackderblaue
  3. Ist für mich kein Fehler, da nicht spezifiziert. Hier Aufwand für Beschränkungen zu implementieren, halte ich nicht für notwendig.
  4. Kann ich nicht nachstellen bei mir. Es wird immer das richtige Wertpapier gelöscht.
  5. Siehe Issue #6 :bug:
MrApplebyte commented 5 years ago
  1. Vielleicht hilft das kurze GIF weiter... So verhält es sich bei mir: ezgif-4-277be6578a66
KarelZe commented 5 years ago

Danke. Sehe was du meinst. Hatte immer zwischengespeichert, da tritt der Fehler nicht auf. Werde mich dem annehmen.

KarelZe commented 5 years ago

@MrApplebyte Kannst du das bitte nachtesten, ob es nun die richtigen Elemente selektiert (Punkt 4). Bei mir war das der Fall 🤗.

MrApplebyte commented 5 years ago

@KarelZe ja passt. 4. ist gefixt

KarelZe commented 5 years ago

@MrApplebyte Kannst du bitte nachtesten, ob nun die Fehlerprüfung bei Unternehmensanlage vollständig ist (Punkt 1)? Bei mir war das der Fall.

Da m. E. Punkt 3 kein Fehler ist und Punkt und Punkt 1, 4, und 5 geschlossen wurden, übergebe ich den Issue an @jackderblaue.

jackderblaueaffe commented 5 years ago
  1. ist behoben: es muss mindestens 1 Unternehmen, 1 Anleihe und 1 Aktie erfasst werden, um fortfahren zu können @MrApplebyte solltest du keine weiteren Fehler finden, kann die Issue geschlossen werden
MrApplebyte commented 5 years ago

@KarelZe Punkt 1 ist gefixt