[cpo+] : Modifikationen 
« zurückFalsche Navigation bei "Benutzer wiederherstellen"
| Kategorie: | Bugfix |
| Beschreibung: | Beim Arbeiten mit der Benutzerverwaltung ist mir aufgefallen, dass es Probleme gibt, wenn die Liste der gelöschten User über mehrere Seiten geht, also die Navigation benötigt wird.
Das liegt daran, dass der verwendete Parameter defaultAction nur bei Änderung der Anzahl pro Seite korrekt ausgewertet wird, ansonsten aber ignoriert wird. |
| Erstellt von: | Markus Rösel |
| Geprüft von: | |
| Veröffentlicht von: | |
| CPO+ Version: | |
| Kurze Erläuterung der Codeanpassung: | in _admin/users.php |
Code vorher:
$defaultAnzahl = 10;
$anzahl = $defaultAnzahl; // FIXME system settings?
if (!empty($_GET['anzahl'])) $anzahl = (int)$_GET['anzahl'];
if ($_REQUEST['action']=='setPerPage') {
if (!$antiCSRF->checkToken()) {
$message[] = ___('Security check failed. Please try again.');
logaction('users setPerPage', 'CSRF check failed.');
} else {
if ($anzahl!=$_POST['newPerPage'])
$_SESSION['SID_pagers']['users'] = 0;
$anzahl = $_POST['newPerPage'];
if (!empty($_REQUEST['defaultAction']))
$_REQUEST['action'] = $_REQUEST['defaultAction'];
}
}
$actions = array();
if (isset($_REQUEST['action']))
is_array($_REQUEST['action'])
? $actions = $_REQUEST['action']
: $actions = array($_REQUEST['action']);
Code nachher:
$defaultAnzahl = 10;
$anzahl = $defaultAnzahl; // FIXME system settings?
if (!empty($_GET['anzahl'])) $anzahl = (int)$_GET['anzahl'];
if ($_REQUEST['action']=='setPerPage') {
if (!$antiCSRF->checkToken()) {
$message[] = ___('Security check failed. Please try again.');
logaction('users setPerPage', 'CSRF check failed.');
} else {
if ($anzahl!=$_POST['newPerPage'])
$_SESSION['SID_pagers']['users'] = 0;
$anzahl = $_POST['newPerPage'];
if (!empty($_REQUEST['defaultAction']))
$_REQUEST['action'] = $_REQUEST['defaultAction'];
}
}
if (!empty($_REQUEST['defaultAction']))
$_REQUEST['action'] = $_REQUEST['defaultAction'];
$actions = array();
if (isset($_REQUEST['action']))
is_array($_REQUEST['action'])
? $actions = $_REQUEST['action']
: $actions = array($_REQUEST['action']);
« zurück
