Bug 858440

Summary: LDAP browser updates wrong entry if editing attribute and switching entry without saving
Product: [openSUSE] openSUSE 13.1 Reporter: Johannes Midgren <johannes>
Component: YaST2Assignee: Peter Varkoly <varkoly>
Status: RESOLVED WONTFIX QA Contact: Jiri Srain <jsrain>
Severity: Normal    
Priority: P5 - None    
Version: Final   
Target Milestone: ---   
Hardware: x86-64   
OS: openSUSE 13.1   
Whiteboard:
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---

Description Johannes Midgren 2014-01-12 21:02:45 UTC 
User-Agent:       Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:26.0) Gecko/20100101 Firefox/26.0

I edited details of my user groups using the LDAP browser in YAST2. At one time I forgot to hit "Save" before switching between groups. YAST2 reminded me that the entry was not saved: "There are unsaved changes in the current entry. Discard these changes?"

Reproducible: Always

Steps to Reproduce:
1. Use YAST2 LDAP browser to edit your LDAP tree
2. Select an (not previously filled) entry in the tree and double click an attribute (right pane) to edit it.
3. Enter some data in the dialogue and hit "OK".
4. Select another entry (left pane) in the LDAP tree (without saving the currently edited).
5. You are warned that the entry was not saved with a dialogue saying: "There are unsaved changes in the current entry. Discard these changes?"
6. Hit "Cancel"
7. Hit "Save"
8. Select the first entry (from 2.) again (left pane).
9. Select the second entry (from 4.) again.
Actual Results:  
- After step 6. above: In the left pane the newly selected entry is still highlighted although the right pane still contains the old unsaved values.
- After step 7: Everything seems fine although the mismatch between entries in left and right pane remains.
- After step 8: The attribute edited in step 2-3 is now again empty.
- After step 9: The attribute value entered in step 2-3 has been assigned the entry selected in step 4.

Expected Results:  
- After step 6: I would expect the actual edited entry to be selected in the left pane. As I hit "Cancel" it would still be the entry selected in step 2.
- After step 8: The entry should contain the inserted attribute value.
- After step 9: The attribute value should still be empty (assuming it was that from the beginning).

As far as I can see, other attributes of the two entries are not mixed up.

(Motivation for severity: It's not a critical issue and there are workarounds (don't forget to save and be cautious), but since it may cause hard to track errors when dealing with e.g. account information, I'd say it should be fixed...)
Comment 1 Johannes Midgren 2014-01-12 21:05:27 UTC 
(In reply to comment #0)

> 2. Select an (not previously filled) entry in the tree and double click an
> attribute (right pane) to edit it.

Step 2 should actually read:
2. Select an entry in the tree and double click an (not previously filled) attribute (right pane) to edit it.
Comment 2 Arvin Schnell 2014-01-13 08:36:24 UTC 
Reassigning to bugowner of yast2-ldap-client.
Comment 3 Peter Varkoly 2018-01-13 20:08:14 UTC 
openSUS 13.1 is out of support.