Bug 27949 - Hinweise bei der Installation über UMC System Setup
Hinweise bei der Installation über UMC System Setup
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Basic settings
UCS 3.0
Other Linux
: P4 minor (vote)
: UCS 3.1
Assigned To: Lukas Walter
Dirk Wiesenthal
: interim-1
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-17 10:27 CEST by Felix Botner
Modified: 2012-12-12 21:10 CET (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): Usability
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Botner univentionstaff 2012-07-17 10:27:24 CEST
Ich habe einen ungejointen 3.0-2 Slave mit univention-samba. Wenn man nun über UMC System Setup z.B. noch den Desktop auswählt, bekommt man folgende Hinweise.

Hinweis: Es ist nicht möglich NT- und Active-Directory-kompatible Domaincontroller gleichzeitig zu verwenden. Stellen sie sicher, dass die existierende UCS-Domäne NT-kompatibel (Samba 3) ist.

Hinweis: Die Installation oder das Entfernen von Softwarekomponenten kann Neustart oder Anhalten von diversen Diensten zur Folge haben. Dies kann ernsthafte Auswirkungen haben, sollte sich das System im Produktiveinsatz befinden.

Der erste Hinweis scheint mit überflüssig, das samba ja bereits installiert ist.
Comment 1 Lukas Walter univentionstaff 2012-08-01 10:49:38 CEST
Die Softwareauswahl wurde so angepasst, dass dieser Hinweis nurnoch dann angezeigt wird wenn eine nicht bereits installierte Version von Samba ausgewählt wird.


univention-system-setup (6.0.6-1) unstable; urgency=low

  * only display samba3/samba4 combination warning
    when a samba component is newly selected that is
    not already installed (Bug #27949)
Comment 2 Alexander Kläser univentionstaff 2012-08-03 11:01:34 CEST
Bitte die Änderung noch einmal überprüfen. Bei mir funktioniert sie auch in Chrome, aber laut W3C sollte RegExp.test() nur einen String und keinen Array von Strings als Parameter erwarten, das kann ggf. bei anderen Browsern fehlschlagen:

http://www.w3schools.com/jsref/jsref_regexp_test.asp

Änderungsvorschlag:

> this.connect(this._form.getWidget('components'), 'onChange', function(newVal) {
> 	dojo.forEach(['samba', 'samba4'], function(ikey) {
> 		var r = new RegExp('univention-' + ikey + '\\b');
> 		dojo.forEach(this._getInstalledComponents(), function(icomponent) {
> 			if (r.test(icomponent)) {
> 				// only show the note when the samba 3 or 4 package is selected
> 				this._showNote(ikey);
> 				return false; // break foreach loop
> 			}
> 		}, this);
> 	});
> }, this);
Comment 3 Lukas Walter univentionstaff 2012-08-10 11:27:22 CEST
Änderungsvorschlag übernommen.


univention-system-setup (6.0.12-1) unstable; urgency=low

  * modified changes for Bug #27949 in order to
    fit W3C specifications

svn34774
Comment 4 Dirk Wiesenthal univentionstaff 2012-09-13 15:09:40 CEST
Die Anpassungen an Dojo 1.8 sind nicht komplett. Es fehlt ein this.own um den watch. Und außerdem ist das return false nicht funktional. array.forEach schert sich nicht um return-values. Damit bricht man die Schleife nicht ab. Wenn das wirklich gewollt ist (ist ja durchaus vernünftig), gibt es wohl array.some()
Comment 5 Lukas Walter univentionstaff 2012-09-17 15:53:08 CEST
Das ist nicht Teil dieses Bugs.

Das gehört zu Bug #28551 (ich habe es dort vermerkt).
Comment 6 Dirk Wiesenthal univentionstaff 2012-09-17 17:32:10 CEST
So sei es
Comment 7 Stefan Gohmann univentionstaff 2012-12-12 21:10:29 CET
UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

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