Fehler in /admin/countries.php

[noRiddle]

In der /admin/countries.php gibt es einen Fehler bei dem Javascript-Link auf die Zeile eines markierten Landes nachdem man nach Status gefiltert hat.
Vorgehensweise:

• Im Backend auswählen Land / Steuer > Land
• Filtern nach z.B. "Status aktiv"
• Klick auf ein Land zum Markieren
• Erneuter Klick zum Bearbeiten

Der Grund ist, daß in der URL des onclick-Events der $_GET[status] fehlt welcher durch das Filtern gesetzt ist.

Lösung:
In Zeile 183 anstatt

$tr_attributes = 'class="dataTableRowSelected" onmouseover="this.style.cursor=\'pointer\'" onclick="document.location.href=\'' . xtc_href_link(FILENAME_COUNTRIES, 'page=' . $_GET['page'] . '&cID=' . $cInfo->countries_id . '&action=edit') .'\'"';

analog zum Link des "Bearbeiten"-Buttons in Zeile 251 mittels xtc_get_all_get_params() das

$tr_attributes = 'class="dataTableRowSelected" onmouseover="this.style.cursor=\'pointer\'" onclick="document.location.href=\'' . xtc_href_link(FILENAME_COUNTRIES, xtc_get_all_get_params(array('page', 'action', 'cID')).'page=' . $_GET['page'] . '&cID=' . $cInfo->countries_id . '&action=edit') .'\'"';

Desgleichen funktioniert nach Filtern auch der onclick-Link zum Markieren eines Landes (um es anschließend bearbeiten zu können) nicht richtig.
Dazu müsste in Zeile 185 anstatt

$tr_attributes = 'class="dataTableRow" onmouseover="this.className=\'dataTableRowOver\';this.style.cursor=\'pointer\'" onmouseout="this.className=\'dataTableRow\'" onclick="document.location.href=\'' . xtc_href_link(FILENAME_COUNTRIES, 'page=' . $_GET['page'] . '&cID=' . $countries['countries_id']) .'\'"';

dies stehen

$tr_attributes = 'class="dataTableRow" onmouseover="this.className=\'dataTableRowOver\';this.style.cursor=\'pointer\'" onmouseout="this.className=\'dataTableRow\'" onclick="document.location.href=\'' . xtc_href_link(FILENAME_COUNTRIES, xtc_get_all_get_params(array('page', 'action', 'cID')).'page=' . $_GET['page'] . '&cID=' . $countries['countries_id']) .'\'"';

Beides ist getestet.

Die von mir aufgeführten Fixes sind analog zum sonstigen Vorgehen in der Datei aufgebaut.
Eigtl. würde es ausreichen in der Funktion xtc_get_all_get_params() lediglich action aufzuführen.
Man schließt nämlich momentan $_GET[page] und $_GET[cID] aus, um sie anschließend dem Link wieder hinzuzufügen.

Gruß,
noRiddle

[Gerhard Waldemair]

In 11227:

fix #1428