Bug 35166 - computerroom module should not add prefix to room name
computerroom module should not add prefix to room name
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - Computer room administration
UCS@school 3.2 R2
Other Linux
: P5 normal (vote)
: UCS@school 3.2 R2 Errata
Assigned To: Dirk Wiesenthal
Florian Best
:
Depends on:
Blocks: 35618
  Show dependency treegraph
 
Reported: 2014-06-19 11:57 CEST by Ammar Najjar
Modified: 2014-08-27 17:01 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 Ammar Najjar univentionstaff 2014-06-19 11:57:23 CEST
On the UMC interface and after creating a school manually using: univention.testing.ucsschool.UCSTestSchool().create_ou(...)

 1- Adding a new computer room with a specific name is successful, but the name shown in the 'Name' field  in 'Edit room' interface is prefixed with: ("%s-" % ou_name)
    Name: ou_name-room_name
  - Desired behaviour: no prefix shown to the end user.
    Name: room_name

 2- Addding a computer to the computer room adds additional prefix ("%s-" % ou_name) to the ldap object dn and consequently to the computer room name in both 'Manage rooms and their associated computers' and 'Edit room' interfaces.
    dn: cn=ou_name-ou_name-room_name,cn=raeume,cn=groups,ou=ou_name,dc=... 
  - Desired behaviour: no prefix added to the ldap object dn and thus no prefix added to name fields for the end user.
    dn: cn=ou_name-room_name,cn=raeume,cn=groups,ou=ou_name,dc=...
Comment 1 Sönke Schwardt-Krummrich univentionstaff 2014-07-03 11:54:43 CEST
Every group membership change of a computerroom adds an additional OU prefix to the room name (which also changes the DN of that room group).
Comment 2 Sönke Schwardt-Krummrich univentionstaff 2014-08-15 15:20:57 CEST
@QA: Bug 35618 creates a ucs-test script that will trigger this bug.
Comment 3 Dirk Wiesenthal univentionstaff 2014-08-20 00:02:41 CEST
ucs-school-umc-rooms 10.0.4-1
Comment 4 Florian Best univentionstaff 2014-08-21 13:39:31 CEST
> +if self.name.startswith('%s-' % self.school):
It would be nicer to make a case insensitive comparison.
Comment 5 Dirk Wiesenthal univentionstaff 2014-08-21 23:27:12 CEST
(In reply to Florian Best from comment #4)
> > +if self.name.startswith('%s-' % self.school):
> It would be nicer to make a case insensitive comparison.

Done
Comment 6 Florian Best univentionstaff 2014-08-22 07:42:07 CEST
OK
Changelog ~OK
Comment 7 Sönke Schwardt-Krummrich univentionstaff 2014-08-27 17:01:50 CEST
UCS@school 3.2 R2 v2 has been released:
http://docs.univention.de/release-notes-ucsschool-3.2R2v2-de.html

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