Bug 35051 - Creating work group cause error message when name contains umlauts
Creating work group cause error message when name contains umlauts
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - Classes / Teachers / Workgroup assignment
UCS@school 3.2 R2
Other Linux
: P5 enhancement (vote)
: UCS@school 3.2 R2 Errata
Assigned To: Dirk Wiesenthal
Florian Best
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-03 16:06 CEST by Michel Smidt
Modified: 2014-08-27 17:01 CEST (History)
3 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 Michel Smidt 2014-06-03 16:06:15 CEST
UCS@school 3.2 R2 v1

Chose "Add workgroup" in "Administrate workgroups" module
Set "Workgroup(*)" to "Schülerzeitung" in Properties section
Chose "Save changes" and got:
Notification
Could not fulfill the request.
Server error message:
The command has failed: Failed to create group (Invalid syntax.).

After confirming and going back to the overview the workgroup is added but no share is created.

Reproducable on UCS@school 3.2 and with other groupnames with umlauts.
Comment 1 Sönke Schwardt-Krummrich univentionstaff 2014-06-05 09:35:51 CEST
We should block invalid characters in the UMC frontend:

--- a/ucs-school-3.2r2/ucs-school-umc-groups/umc/js/schoolgroups/DetailPage.js
+++ b/ucs-school-3.2r2/ucs-school-umc-groups/umc/js/schoolgroups/DetailPage.js
 -123,6 +123,7 @@ define([
 				name: 'name',
 				label: this.moduleFlavor == 'class' ? _( 'Class' ) : _( 'Workgroup' ),
 				disabled: this.moduleFlavor != 'workgroup-admin',
+				regExp: '^[a-zA-Z0-9-]+$',
 				required: true
 			}, {
 				type: TextBox,

When fixing this, a tooltip for this widget should be added that states which characters are allowed for group names.
Comment 2 Dirk Wiesenthal univentionstaff 2014-08-15 15:55:58 CEST
I used '^[a-zA-Z0-9]([a-zA-Z0-9 _.-]*[a-zA-Z0-9])?$'
Comment 3 Dirk Wiesenthal univentionstaff 2014-08-20 00:22:48 CEST
ucs-school-umc-groups 2.0.5-3
Comment 4 Florian Best univentionstaff 2014-08-20 12:12:48 CEST
REOPEN:
(In reply to Sönke Schwardt-Krummrich from comment #1)
> When fixing this, a tooltip for this widget should be added that states
> which characters are allowed for group names.
Comment 5 Dirk Wiesenthal univentionstaff 2014-08-21 23:21:03 CEST
(In reply to Florian Best from comment #4)
> REOPEN:
> (In reply to Sönke Schwardt-Krummrich from comment #1)
> > When fixing this, a tooltip for this widget should be added that states
> > which characters are allowed for group names.

Fixed.
Comment 6 Florian Best univentionstaff 2014-08-22 07:38:32 CEST
OK, i renamed 'aufhören' into 'enden'.
Changelog: OK
Comment 7 Sönke Schwardt-Krummrich univentionstaff 2014-08-27 17:01:39 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".