Bug 46117 - Add a "Address book entry" object type
Add a "Address book entry" object type
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UDM (Generic)
UCS 4.3
Other Linux
: P5 normal (vote)
: UCS 4.3
Assigned To: Johannes Keiser
Stefan Gohmann
: interim-3
Depends on: 46357
Blocks: 46473
  Show dependency treegraph
 
Reported: 2018-01-23 14:55 CET by Florian Best
Modified: 2018-03-14 14:37 CET (History)
4 users (show)

See Also:
What kind of report is it?: Feature Request
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 Florian Best univentionstaff 2018-01-23 14:55:11 CET
There might be the need for an "Adress book entry" object type.
Comment 1 Stefan Gohmann univentionstaff 2018-02-21 07:24:55 CET
As discussed via mail, I think we shouldn't migrate person only users to full users. The problem is, that these users will be synchronized to Active Directory. This could led to problems.

Johannes, please add a UDM module for these address book entry. My suggestion is users/contact. The attributes should be only address book values.

I'll create another bug for the migration script.
Comment 2 Johannes Keiser univentionstaff 2018-03-02 16:16:33 CET
f0e9f20 Bug #46117: Add users/contact object type
64e9dfc Bug #46117: Add debian changelog entries

univention-ldap (14.0.2-13) unstable; urgency=medium

  * Bug #46117: Add temporary container for locking dn(s)

univention-directory-manager-modules (13.0.21-4) unstable; urgency=medium

  * Bug #46117: Add users/contact object type

univention-management-console-module-udm (8.0.5-3) unstable; urgency=medium

  * Bug #46117: Add users/contact udm module to frontend


Added users/contact object type
The dn for a contact is generated like 'firstname lastname number'
or 'lastname number' if no firstname is given.
The number is counted up for every contact with the same name.
Comment 3 Stefan Gohmann univentionstaff 2018-03-02 17:04:44 CET
(In reply to Johannes Keiser from comment #2)
> f0e9f20 Bug #46117: Add users/contact object type
> 64e9dfc Bug #46117: Add debian changelog entries
> 
> univention-ldap (14.0.2-13) unstable; urgency=medium
> 
>   * Bug #46117: Add temporary container for locking dn(s)

As discussed, we don't need these locking objects since we don't need a global uniqueness of these entries.
Comment 4 Johannes Keiser univentionstaff 2018-03-02 20:22:24 CET
0755911 Bug #46117: Remove locking of dn
9be7389 Bug #46117: Add debian changelog entries
679fbb0 Bug #46117: Changelog entry
a4a612d Bug #46117: Fix typo and enable move operation
6c947c9 fixup! Bug #46117: Fix typo and enable move operation # Debian changelog
1d5f0bb Bug #46117: Version bump
Comment 5 Stefan Gohmann univentionstaff 2018-03-04 21:00:35 CET
Changelog: OK

Tests: Fail

1. The icon is not available:
The requested URL /univention/js/dijit/themes/umc/icons/16x16/udm-users-contact.png was not found on this server.

2. The birthday can't be set:
root@master431:~# udm users/contact create  --set lastname=Test2 --set birthday="2017-01-01"
LDAP Error: Object class violation: attribute 'univentionBirthday' not allowed
root@master431:~# 

Everything else works.
Comment 6 Johannes Keiser univentionstaff 2018-03-05 12:20:50 CET
(In reply to Stefan Gohmann from comment #5)
> Changelog: OK
> 
> Tests: Fail
> 
> 1. The icon is not available:
> The requested URL
> /univention/js/dijit/themes/umc/icons/16x16/udm-users-contact.png was not
> found on this server.
> 
> 2. The birthday can't be set:
> root@master431:~# udm users/contact create  --set lastname=Test2 --set
> birthday="2017-01-01"
> LDAP Error: Object class violation: attribute 'univentionBirthday' not
> allowed
> root@master431:~# 
> 
> Everything else works.

448de96 Bug #46117: Fix saving birthday and add icon for grid
Comment 7 Stefan Gohmann univentionstaff 2018-03-05 14:30:19 CET
(In reply to Johannes Keiser from comment #6)
> 448de96 Bug #46117: Fix saving birthday and add icon for grid


OK, works now.

Two things we could change later:
1. The icon for users and contacts is currently the same.
2. Contacts use cn=users as default container.
Comment 8 Arvid Requate univentionstaff 2018-03-05 17:20:52 CET
An all contacts now get a number suffix by default, even if there is no conflict. That might be considered ugly:

# Arvid Surname 1, ar41i1.qa
dn: cn=Arvid Surname 1,dc=ar41i1,dc=qa
objectClass: person
objectClass: univentionPWHistory
objectClass: univentionObject
objectClass: organizationalPerson
objectClass: top
objectClass: inetOrgPerson
univentionObjectType: users/contact
displayName: Arvid Surname
cn: Arvid Surname 1
sn: Surname
givenName: Arvid
Comment 9 Stefan Gohmann univentionstaff 2018-03-14 14:37:58 CET
UCS 4.3 has been released:
 https://docs.software-univention.de/release-notes-4.3-0-en.html
 https://docs.software-univention.de/release-notes-4.3-0-de.html

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