feedback zu convert_frontend_users.php

[knb]

Vorher
Zeile 123

Code: Alles auswählen

		foreach ($aClients as $client => $aClient) {
			echo "<br><br>".i18n(sprintf("Conversion: Client-ID: %s", $client))."\n";
			echo "<br>".i18n("Conversion: Groups: Getting groups with frontend access...")."\n";

Das Script gibt nur die Client id aus, nicht den Mandanten-Namen. Wenn man mehr als zwei Clients hat, benötigt man diesen Namen aber, um das Ergebnis besser beurteilen zu können. Daher
schlage ich folgende Änderung vor.

Nachher

Code: Alles auswählen

		foreach ($aClients as $client => $aClient) {
			// TODO: here we need the client name, too
			/* Lookup client name  (Hack! Hack! Hack!) */
	    $sql = "SELECT name, idclient from " . $cfg["tab"]["clients"] . " WHERE (idclient=".$client.")";
	 
	    $oDB->query($sql); 
	    while ($oDB->next_record())
	    {
			if ($oDB->f("idclient")==$client) {
				$client_name=trim($oDB->f("name"));
		
				}
    	}
			echo "<br><br>".i18n(sprintf("Conversion: Client %s (ID: %s)", $client_name, $client))."\n";

Ausgabe
vorher

Code: Alles auswählen

Conversion: Client ID 25 

nachher

Code: Alles auswählen

 Conversion: Client Kunde (ID: 42) 

[knb]

Ausserdem bin ich der Meinung dass der Output der convert_remove Option überarbeitet werden sollte ...
das Script macht anscheinend alles richtig nur finde ich die Ausgabe nicht so verständlich.
Wie gesagt, es sind nur Feinheiten. Aber das Script macht nun mal wichtige Änderungen am Gesamtsystem, die so klar wie möglich beschrieben sein sollten.



Grundsätzlich: Man denkt es beim ersten Lesen es handelt sich bei convert_remove um die zu convert komplementäre Operation (= Rückgängig machen). Es steht zwar im Text was convert_remove macht aber selbsterklärend ist es noch nicht.
Dabei ist es eine zusätzliche Option ("convert_and_remove")

Der Output ist etwas unklar :

Code: Alles auswählen

Conversion: Groups: Trying to convert account 'unterh (H. Unterhxxx)': Account already exists, just adding to frontend user group. Successfully finished. Account planned for removement.

Was wird hier vom Script vorgemerkt/gespeichert?
Das Script erzeugt was, merkt sich was es gerade gemacht hat und merkt es sich trotzdem um es hinterher wieder zu löschen?
Alternativ-Vorschlag:

Code: Alles auswählen

Conversion: Groups: Trying to convert account 'unterh (H. Unterhxxx)': Frontend User Account already exists, just adding it to frontend user group. Successfully finished. Account will be removed from backend. See end of script output / bottom of this page.

Code: Alles auswählen

Conversion: Client xxx (ID: 19)
Conversion: Groups: Getting groups with frontend access...
Conversion: Groups: No groups with frontend access found.
Conversion: Users: Getting users with frontend access...

im Output hier fehlt eine entsprechende Zeile

Code: Alles auswählen

Conversion: Users: No users with frontend access found.

Code: Alles auswählen

Clean Up: Removing users... (groups have to be removed manually) 

Dies ist auch irreführend. Man denkt der Administrator muss noch was tun... dabei wird dieser Löschvorgang auch vom Script erledigt .

Code: Alles auswählen

Clean Up: Removing backend users...  

Code: Alles auswählen

Clean Up: Backend account db509bf6c2bf6f9c0cf814c57fe7672cunterh (H.  Unterhxxx) removed 

Da fehlt ein blank nach dem md5 hash. Wie reinkriegen?

Ändern removement => removal

Möglicherweise hinzufügen Auswahlmöglichkeit für einzelne Clients?

[HerrB]

Account planned for removement

Der gleiche Account kann in mehreren Gruppen Mitglied sein. Wird er sofort gelöscht, geht diese Information verloren. Zu diesem Zeitpunkt wird nichts gelöscht. Die Aussage ist korrekt (ggf. removement -> removal).

Clean Up: Removing users... (groups have to be removed manually)

Backend-Gruppen werden NICHT gelöscht, da nicht abschätzbar ist, was da noch dran hängt. Die Aussage ist korrekt.

Rest ok, sehe ich mir bei Gelegenheit an.

Möglicherweise hinzufügen Auswahlmöglichkeit für einzelne Clients?

Wird es von mir nicht geben, da a) mir ehrlich gesagt der Aufwand zu hoch ist und b) die Frontend-User-Verwaltung für das ganze System und nicht nur für einzelne Mandanten mit Contenido V4.6.x umgestellt wurde.

Gruß
HerrB

[knb]

Gut, alles klar, sonst ja funktioniert alles,

ich wollte mich nur mal zur Usability äußern.

Das Script ist wohl auch eine gute Vorlage für eigene Contenido-API-scripts.


Das mit den Backendgroups habe ich nach dem Posting auch realisiert.