Default-Sortierung bei Activity-Mod

[Sujao]

Bei der Ansicht der Spiele kann man auswählen nach welchen Kriterien man die Spiele sortieren kann. Ich habe diese Sortier-Aufklapp-Leisten in der activity_body.tpl entfernt. Jetzt würde ich gerne die activity.php so verändern, dass die Liste automatisch nach der Game-ID sortiert wird und zwar aufsteigend, also 1,2,3,4,5... Ich hab mich in der Datei ein bisschen umgeguckt und die Variable default in Zeile 150 gefunden und vermute das man diese Einstellen müsste.

Im Prinzip brauche ich die ganzen Sortieralgorithmen überhaupt nicht, also kann man sie auch ruhig komplett entfernen. Was bleiben soll ist nur:

Sortier immer nach Game-ID und zwar aufsteigend !

Leider fehler mir die PHP-Kenntnisse um das umzusetzen. Kann jemand bitte helfen ?

[makue]

Im Script wird die Sortierung in $order_by festgelegt. Wenn Du nur diese eine Sortierung brauchst, kannst Du den ganzen Block mit switch( $mode ) weglassen und stattdessen nur die eine Zeile:

Code: Alles auswählen

$order_by = "game_id $sort_order LIMIT $start, " . $board_config['games_per_page'];

dafür benutzen. Ich habe den Code jetzt nicht komplett analysiert, aber so könnte es gehen. Ach ja, das $sort_order in dieser Zeile vielleicht auch noch weglassen, da das entscheidet, ob auf- oder absteigend sortiert wird.

[Sujao]

Aah, ausgezeichnet! Es hat geklappt. Vielen Dank!

Hab da nochwas: Es ist zwar nicht ganz so wichtig, aber könntest du dir vielleicht mal Zeile 90-130 kurz durschauen und mir sagen ob man das ebenfalls entfernen kann ? Die Verzögerungen durch diesen Code sind zwar bestimmt minimal,a ber es ist ein gutes Gefühl keinen Müll in der Datei zu haben

[makue]

Also nach diesem switch( $mode ) Block kann man auch alles im Block
if( $mode != "" ) löschen (zeile 155-285), muss dann aber das else in Zeile 285 mit rauswerfen, ich würde das Klammernparr in Zeile 186 und 465 aber mit rauswerfen. Der Klammernblock {...} vom Else dürfte aber eigentlich nicht stören. Mit Zeile 90-130 kann ich leider nichts anfangen, Zeile 90 ist mittem im Code. Jednfalls bei der von Dir geposteten Datei. Du kannst aber auch noch folgende Blöcke löschen:

Code: Alles auswählen

if (isset($HTTP_GET_VARS['mode']) || isset($HTTP_POST_VARS['mode']))
{
	$mode = (isset($HTTP_GET_VARS['mode'])) ? $HTTP_GET_VARS['mode'] : $HTTP_POST_VARS['mode'];
}
else 
{
	if( isset($HTTP_POST_VARS['game']) )
	{
		$mode = "game";
	}
	else if( isset($HTTP_POST_VARS['stats']) )
	{
		$mode = "stats";
	}
	else
	{
		$mode = "";
	}
}

Code: Alles auswählen

if(isset($HTTP_POST_VARS['order']))
{
	$sort_order = ($HTTP_POST_VARS['order'] == 'DESC') ? 'DESC' : 'ASC';
}
else if(isset($HTTP_GET_VARS['order']))
{
	$sort_order = ($HTTP_GET_VARS['order'] == 'DESC') ? 'DESC' : 'ASC';
}
else
{
	$sort_order = 'DESC';
}

Natürlich lässt sich an weiteren Stellen noch was weglassen aber viel bringt das wirklich nicht.

[Sujao]

Wie genau meinst du das beim Klammernpaar ? Zeiel 186 wird ja sowieso gelöscht wenn ich die erste Löschung (155-285) vornehme. Und wie hängt 465 mit 186 zusammen ?

[makue]

Der else-Zweig sieht ja etwa so aus:

Code: Alles auswählen

else
{
...anderer Code
}

Wenn man das else wegmacht, dann besser auch die { und die }. In 186 und 465 sollten eben diese Klammern stehen.

[Sujao]

ach so ! ok verstanden