Bug 26062 - UMC2: Löschen des ausgewählten Objektes
UMC2: Löschen des ausgewählten Objektes
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Domain management (Generic)
UCS 3.0
Other Linux
: P4 normal (vote)
: UCS 3.1-0-errata
Assigned To: Dirk Wiesenthal
Alexander Kläser
:
: 29609 30228 (view as bug list)
Depends on:
Blocks: 30612
  Show dependency treegraph
 
Reported: 2012-02-08 16:49 CET by Roman Asendorf
Modified: 2013-02-28 12:54 CET (History)
7 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments
LDAP-Tree: act on container below cursor and show LDAP-Path (1.54 KB, patch)
2012-12-20 21:07 CET, Dirk Wiesenthal
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Roman Asendorf univentionstaff 2012-02-08 16:49:51 CET
Es gibt ein cooles Feature zum Löschen von LDAP-Objekten und deren Unterobjekte, wenn man sich über die Navigation im LDAP-Verzeichnisbaum bewegt.

Klickt man mit der rechten Maustaste auf ein Objekt im Baum, gibt es die Kontextmenüfunktion "Löschen". Beim Auswählen erscheint der modale Dialog:

-------------------------------------------------------------------
"Bitte bestätigen Sie das Löschen des/der ausgewählten UDM-Objekt!"

[X] Zugehörige Objekte löschen.

Abbrechen    Löschen
-------------------------------------------------------------------

Es fehlt jedoch der Hinweis, um welches Objekt es sich handelt, dass da ausgewählt wurde. Markiert bzw. grün hinterlegt war in meinem Fall das Domänenobjekt (Wurzel), da ich dieses Objekt zuletzt mit der linken Maustaste ausgewählt hatte.

Im modalen Dialog gefangen, kann man jedoch die Position nicht mehr erkennen, von der aus das Kommando abgesetzt wurde.

Bricht man den Vorgang an dieser Stelle ab, erkennt man anhand einer gestrichelten Linie das aktuell ausgewählte Objekt. Gestrichelt und grün hinterlegt stellen hierbei einen semantischen Unterschied dar, der jedoch dem Benutzer (zumindest mir) nicht so leicht ersichtlich war, zumal die grün hinterlegten Objekte wesentlich deutlicher hervorstechen.

Hier sollte eine klarere Trennung vorgenommen werden, zumal sich mir der Sinn der grünen Hinterlegung von Objekten nicht mehr erschlossen hat, da es ja gerade nicht die aktuelle Auswahl darstellt.
Comment 1 Roman Asendorf univentionstaff 2012-02-08 17:07:20 CET
Und es wird noch komplizierter:

Befindet man sich in der Navigation und wählt ein Objekt (wir nennen es Objekt A) mit der linken Maustaste aus, so haben wir den Status:

Objekt A ist grün hinterlegt und der Name gestrichelt umrahmt

Klickt man nun mit der rechten Maustaste auf ein anderes Objekt (wir nennen es Objekt B, und wichtig! der Name muss mit der Maus getroffen werden) und wählt anschließend nichts aus, bzw. bricht den Vorgang ab haben wir den Status:

Objekt A ist grün hinterlegt und Objekt B ist gestrichelt umrahmt.

Nun wähle man ein weiteres Objekt (Objekt C) mit der rechten Maustaste aus, und, hierbei sind wir nicht so sicher mit der Maus, treffen wir den Namen des Objektes nicht, dann erhalten wir den Status:

Objekt A ist grün hinterlegt und Objekt B ist gestrichelt umrahmt und Objekt C, aktuelles Kontextobjekt, ist in keiner Weise hervorgehoben.

BTW: Gelöscht wird übrigens wirklich Objekt C also greift die Positionierung korrekt.
Comment 2 Alexander Kläser univentionstaff 2012-03-30 12:12:06 CEST
Stimmt der Name sollte noch einmal mit ausgegeben werden, das kann ansonsten verwirren, gerade bei Löschoperationen.
Comment 3 Stefan Gohmann univentionstaff 2012-12-07 09:25:17 CET
(In reply to comment #2)
> Stimmt der Name sollte noch einmal mit ausgegeben werden, das kann ansonsten
> verwirren, gerade bei Löschoperationen.

Gerade vermutlich bei den Tests von Philipp nochmal aufgetreten, es wurde die komplette LDAP Basis gelöscht.

Zum einen sollte der Name angezeigt werden, es sollte aber auch das Objekt verwendet werden, auf das geklickt wird. Und nicht das markierte.
Comment 4 Philipp Hahn univentionstaff 2012-12-07 09:27:32 CET
*** Bug 29609 has been marked as a duplicate of this bug. ***
Comment 5 Dirk Wiesenthal univentionstaff 2012-12-20 21:07:48 CET
Created attachment 4943 [details]
LDAP-Tree: act on container below cursor and show LDAP-Path

Patch für Errata-Update
Comment 6 Dirk Wiesenthal univentionstaff 2012-12-21 09:56:57 CET
Das ContextMenu des Baums hat einen neuen Eintrag: Der erste Eintrag ist ein Button, immer disabled, der dynamisch auf den LDAP-Pfad des aktuellen Objektes gesetzt wird. Damit ist vor der Auswahl klar, auf welchen Container im Baum das folgende "Löschen" beziehen wird (nämlich das unter der Maus, nicht das ausgewählte, das für das Grid rechts davon verantwortlich ist). Die Dialoge selbst wurden nicht angefasst.

Das hätte auch Auswirkungen auf alle Grids und ich weiß nicht, ob wir die im Zuge von Bug#29596 vielleicht ohnehin umstrukturieren.
Comment 7 Alexander Kläser univentionstaff 2013-01-08 18:06:31 CET
Änderung → FAILED

Die Idee finde ich gut, den Namen mit in das Kontextmenü zu integrieren. Zwei Verbesserungsvorschläge:
* statt die DN komplett als "cn=testb,..." anzugeben könnte, könnte über umc/tools:ldapDn2Path() ein Pfad ausgegen werden
* die Darstellung als Menüeintrag empfinde ich als verwirrend, man kann allerdings ein beliebiges HTML-Element (bspw. umc/widgets/Text) übergeben und entsprechend formatieren (bspw. fett).

YAML-Eintrag → OK
Übernahme der Änderungen nach 3.1-1 → FAILED
Changelog 3.1-1 → OK
Comment 8 Dirk Wiesenthal univentionstaff 2013-01-10 11:11:40 CET
(In reply to comment #7)
> Änderung → FAILED
> 
> Die Idee finde ich gut, den Namen mit in das Kontextmenü zu integrieren. Zwei
> Verbesserungsvorschläge:
> * statt die DN komplett als "cn=testb,..." anzugeben könnte, könnte über
> umc/tools:ldapDn2Path() ein Pfad ausgegen werden
> * die Darstellung als Menüeintrag empfinde ich als verwirrend, man kann
> allerdings ein beliebiges HTML-Element (bspw. umc/widgets/Text) übergeben und
> entsprechend formatieren (bspw. fett).
> 
> YAML-Eintrag → OK
> Übernahme der Änderungen nach 3.1-1 → FAILED
> Changelog 3.1-1 → OK

Einfach einen Text übergeben geht nicht, denn es handelt sich bei einem Menu HTML-seitig um eine Tabelle und bei den MenuItems um <tr>s. Neue Herangehensweise: Wenn nur ein Objekt gelöscht oder verschoben werden soll, wird es noch einmal direkt im Dialog angezeigt.

Änderungen wurden jetzt übernommen und YAML entsprechend angepasst.
Comment 9 Alexander Kläser univentionstaff 2013-01-11 17:42:29 CET
Ja, so ist das prima!

Änderung → OK
YAML-Datei → OK
Übernahme 3.1-1 → FAILED
Changelog 3.1-1 → OK
Comment 10 Alexander Kläser univentionstaff 2013-01-11 19:38:35 CET
(In reply to comment #9)
> Ja, so ist das prima!
> 
> Änderung → OK
> YAML-Datei → OK
> Übernahme 3.1-1 → FAILED
> Changelog 3.1-1 → OK

Übernahme → OK
Comment 11 Alexander Kläser univentionstaff 2013-01-11 19:38:45 CET
YAML-Datei: 2013-01-04-univention-management-console-module-udm.yaml
Comment 12 Moritz Muehlenhoff univentionstaff 2013-01-15 15:17:49 CET
http://errata.univention.de/3.1-errata14.html
Comment 13 Jan Christoph Ebersbach univentionstaff 2013-01-31 14:24:31 CET
*** Bug 30228 has been marked as a duplicate of this bug. ***