id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	blockedby	blocking
1428	Fehler in /admin/countries.php	noRiddle	somebody	"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
"	Bug/Fehler	closed	normal	modified-shop-2.0.5.0	Admin	2.0.3.0	fixed