Gelöschte Benutzer werden in Auswahllisten angezeigt.

[martin.schwier@…]

Bei einer frischen Installation von 2.6.3 bekomme ich eine Fehlermeldung beim Versuch einen Benutzer zu löschen:

UPDATE employee SET deleted = 't' WHERE login = ? (martin)
FEHLER: Spalte »deleted« von Relation »employee« existiert nicht
LINE 1: UPDATE employee SET deleted = 't' WHERE login = $1

PS: Der Benutzer ist nach der Fehlermeldung scheinbar trotzdem gelöscht.

Das kann ich bestätigen. Löscht man einen Benutzer so bleibt der Eintrag in auth.user erhalten, aber die Konfiguration wird aus auth.user_config gelöscht, und der Benutzer taucht dann auch auch nicht mehr in der Benutzerliste im Adminmenü auf.

Die Spalte "deleted" in employee gibt es gar nicht, deshalb sollte der Benutzer in der Verkäufer/Bearbeiter?-Liste also noch auftauchen (muß ja auch, wegen der history).

So einfach ist das Ganze nicht.

Mögliche Szenarien:

1. Gelöschte Bneutzer sollte in der Neuauswahl von Bearbeitern/Verkäufern? nicht mehr auftauchen. Abgesehen davon dass es das Richtige ist, ist es auch schon vorgekommen, dass ein Kunde zuviele User angelegt hatte und dann der gleiche Benutzer unter 3 verschiedenen Logins im System war.

2. Wenn ein Benutzer mal auf eine Mandantendatenbank zugegriffen hat ist er in beiden vorhanden, wird aber im Moment nur in einer auf deleted gesetzt, egal wie man es macht.

Ergo besteht der Fix aus vier Bestandteilen:

1. Das naive Löschen von Benutzern in der Administration sollte funktionieren und den Benutzer in der verlinkten mandanten-DB auf deleted setzen (was keinen SQL Fehler erzeugen sollte).

2. Es sollte eine Möglichkeit geben in der Mandanten-DB den Benutzer nachträglich auf deleted oder umgekehrt zu setzen.

3. Hat jemand mal getested was passiert, wenn man einen gelöschten Benutzer neu anlegt? Wird die alte ID neu aufgewärmt? Doppelt angelegt?

4. In allen Anzeigen von Benutzern/Verkäufern? sollten standardmässig gelöschte ausgeblendet werden, sofern Sie nicht schon ausgewählt sind.

So, employee hat mit 96fe8a6 jetzt die entsprechende Spalte.

Wenn ein Benutzer in der Administration gelöscht wird, wird übrigens der letzte User neu belebt. Das deleted Flag wird mit 1d5b112 dann auf false resetted. Damit sind Punkt 1 und 3 abgeschlossen.

Da der eigentliche Bugtitel gefixt ist bennen ich den Bug um.

War: SQL Fehler beim löschen von Benutzern
Neu: Gelöschte Benutzer werden in Auswahllisten angezeigt.

Mit der Commitserie 6bf344a1..35b746e4 werden jetzt gelöschte Benutzer nicht mehr zur Auswahl gegeben.

Damit ist Punkt 4 abgeschlossen.

Und mit der Commitserie 8f94be3..fd136da gibt es auch die Möglichkeit nachträglich benutzer zu bearbeiten.

Damit ist Punkt 2 abgeschlossen.

Ticket -> closed

Patch um Makemodel aus Erzeugnis rauszunehmen