Bug 18702 - Leersetzen von Attributen erzeugt Fehlermeldung aber funktioniert
Leersetzen von Attributen erzeugt Fehlermeldung aber funktioniert
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UDM - CLI
STABLE
Other Linux
: P5 normal (vote)
: UCS 3.0-2
Assigned To: Lukas Walter
Jascha Geerds
https://bugs.open-xchange.com/show_bu...
: interim-3
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-06-17 17:09 CEST by Tobias Scherer
Modified: 2012-07-20 15:24 CEST (History)
4 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):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tobias Scherer univentionstaff 2010-06-17 17:09:59 CEST
Berichtet an Ticket#2010061310000623:

Wenn mit dem UDM CLI ein Attribut leer gesetzt wird, wird eine Fehlermeldung ausgegeben, das Attritbut wird dann aber umgesetzt:

root@qamaster:~$ udm users/user modify --dn=uid=tester01,cn=users,dc=univention,dc=qa --set homeTelephoneNumber="123"
Object modified: uid=tester01,cn=users,dc=univention,dc=qa
root@qamaster:~$ udm users/user list --filter uid=tester01 | grep homeTelephoneNumber
  homeTelephoneNumber: 123
root@qamaster:~$ udm users/user modify --dn=uid=tester01,cn=users,dc=univention,dc=qa --set homeTelephoneNumber=""
WARNING: No attribute with name 'homeTelephoneNumber' in this module, value not set.
Object modified: uid=tester01,cn=users,dc=univention,dc=qa
root@qamaster:~$ udm users/user list --filter uid=tester01 | grep homeTelephoneNumber
  homeTelephoneNumber: None
Comment 1 Rainer Bröring 2010-12-23 13:19:51 CET
Leider auch in UCS 2.4 nicht berücksichtigt.

hier ein Korrekturvorschlag für die Datei admin.py im Verzeichnis
/usr/share/pyshared/univention/admincli, um die lästige Warnung zumindest bei einem "--set <name>=" loszuwerden. Beim --set ist das möglich, da hier das Setzen der leeren Liste im Falle eines MultiValue-Attributs in ein Löschen des entsprechenden LDAP-Attributs beim LDAP-Objekt umgesetzt wird.

Änderungen:
diff admin.py.org  admin.py
592d591
<                                                       was_set=1
595c594
<                                               was_set=1
---
>                                       was_set=1

            was_set=0
            for mod, (properties,options) in information.items():
                if properties.has_key(name):
                    if properties[name].multivalue:
                        if not input.has_key(name):
                            input[name]=[]
                        if value:
                            input[name].append(value)
                    else:
                        input[name]=value
                    was_set=1

            if not was_set:
                out.append("WARNING: No attribute with name '%s' in this module, value not set."%name)
Comment 2 Janis Meybohm univentionstaff 2011-05-16 09:33:48 CEST
Erneut berichtet über Ticket#:2010122310001986
Comment 3 Rainer Bröring 2011-09-21 11:20:00 CEST
Es nervt. 

Diesmal beobachtet beim "Leersetzen" von mailAlternativeAddress.

root@ox:/var/backups# /usr/sbin/udm users/user modify --dn=uid=juk,cn=users,dc=saacke,dc=de --set mailAlternativeAddress=y@saacke.de
Object modified: uid=juk,cn=users,dc=saacke,dc=de

root@ox:/var/backups# /usr/sbin/udm users/user modify --dn=uid=juk,cn=users,dc=saacke,dc=de --set mailAlternativeAddress=
WARNING: No attribute with name 'mailAlternativeAddress' in this module, value not set.
Object modified: uid=juk,cn=users,dc=saacke,dc=de
Comment 4 Lukas Walter univentionstaff 2012-06-11 11:47:47 CEST
Das Problem bestand darin, dass der udm-cli vergessen hat zu speichern, dass ein multivalue Attribut geändert wurde wenn es einfach nur geleert worden ist, was im weiteren Ausführungsverlauf dazu geführt hat dass er sich einbildet es gäbe das Attribut garnicht.

Ist jetzt behoben.
Comment 5 Rainer Bröring 2012-06-14 10:18:27 CEST
Die Formulierungen in Comment 4 lassen darauf schließen, dass es sich bei adm-cli um ein intelligenzbehaftetes Wesen handelt, dass zu denken und auch zu vergessen imstande ist ...
Comment 6 Alexander Kläser univentionstaff 2012-06-15 16:15:47 CEST
(In reply to comment #5)
> Die Formulierungen in Comment 4 lassen darauf schließen, dass es sich bei
> adm-cli um ein intelligenzbehaftetes Wesen handelt, dass zu denken und auch zu
> vergessen imstande ist ...

Davon kann doch durchaus ausgegangen werden! Programme sind doch auch nur Menschen :) .
Comment 7 Rainer Bröring 2012-06-15 16:42:57 CEST
(In reply to comment #6)
> (In reply to comment #5)
> > Die Formulierungen in Comment 4 lassen darauf schließen, dass es sich bei
> > adm-cli um ein intelligenzbehaftetes Wesen handelt, dass zu denken und auch zu
> > vergessen imstande ist ...
> 
> Davon kann doch durchaus ausgegangen werden! Programme sind doch auch nur
> Menschen :) .

... werden überwiegend von Menschen erstellt und die machen nun mal Fehler wie oben, wo es es in meinem Kommentar heißen müsste: "..., das zu denken ...".

Nun ist aber auch gut.
Comment 8 Alexander Kläser univentionstaff 2012-06-15 17:36:56 CEST
(In reply to comment #7)
> ...
> Nun ist aber auch gut.

Na gut.
Comment 9 Jascha Geerds univentionstaff 2012-06-25 13:45:01 CEST
Der Bug wurde gefixed; Changelog ist vorhanden.
Comment 10 Stefan Gohmann univentionstaff 2012-07-20 15:24:36 CEST
UCS 3.0-2 has been released: 
  http://forum.univention.de/viewtopic.php?f=54&t=1905

If this error occurs again, please use "Clone This Bug".