Alfabetisk sortering (av medlemslistan)

[tumba25]

Nu har jag tittat i memberlist.php. Här är min lösning på problemet.

Öppna memberlist.php
Leta upp

Kod: Markera allt

$order_by .= $sort_key_sql[$sort_key] . ' ' . (($sort_dir == 'a') ? 'ASC' : 'DESC');  

ersätt med

Kod: Markera allt

$order_by .= (($sort_key != 'd' && $sort_key != 'm') ? 'LOWER(' : '') . $sort_key_sql[$sort_key] . (($sort_key != 'd') ? ') ' : ' ') . (($sort_dir == 'a' && $sort_key != 'm') ? 'ASC' : 'DESC');

Det hjälper antagligen inte för egna profilfält, men du kan säkert lägga samma fix där dom letas upp.

[Michanek]

Jag tror att din lösning kommer att fungera även för profilfält. Just nu är det dock något fel på din syntax:

Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING in /forum/memberlist.php on line 1188

Peter.

[tumba25]

Inget fel med syntaxen. Jag använder exakt samma på två (snart 3) forum, utan några problem. Däremot kan det vara phpbb's php-BBCode som har problem med något. Jag ändrad mitt inlägg så det inte tolkas som php. Prova igen.

[Michanek]

Nu fungerar det precis som det ska men jag kan inte se vad du ändrade i ditt inlägg. Märkligt! Den förtydligande förgkodningen är ju borta men jag har svårt att tro att den följde med in i min kod. Nåja, det fungerar ju så jag ska väl inte gräva för djupt i det.

Fixen fungerar precis som jag trodde även för mitt profilfält. Det är implementerat på ett bra sätt med en egen sort_key och egen rad i $sort_key_sql och kommer därför in i SQL-koden via samma $order_by.

Stort tack! Du är en klippa!

Peter.

[Michanek]

Jag har inte testat men jag antar att ÅÄÖ fortfarande kommer i fel ordning och jag antar vidare att det går att åtgärda genom att ändra kollationeringen från uft8_bin till utf8_swedish_ci. Använder phpBB genomgående utf8_bin eller går detta att ställa in någonstans i ACP eller måste jag ändra direkt i databasen?

Peter.

[tumba25]

ÅÄÖ kommer i fel ordning med dom flesta teckentabeller. Det är inte något stort problem tycker jag. Men om du ska ändra så måste du ändra direkt i databasen. Tänk på att ändra headern också. Dessutom förutsätter många funktioner i phpBB att det är utf-8 så jag föreslår att du inte lägger märke till att ä råkar komma före å.

Angående ändringen i mitt inlägg så var det just den förtydligande färgkodningen som ställde till problem.

[Marcus]

ÅÄÖ ligger i fel ordning i alla teckentabeller.

Alla tecken baserade på A ligger samlade, och den ordningen tar inte hänsyn till att vi i Sverige har två tecken i en viss ordning.

/marcus

[Michanek]

Lösningen är nog att betrakta detta som en bugg.

Den är nu rapporterad som Ticket #42085.

Peter.

[tumba25]

Tack för credden.

[Michanek]

Tack för lösningen!

Peter.

[Michanek]

Jag har nu märkt att vissa fält inte mår så bra av LOWER() så man måste villkora det till att bara gälla text-fält. Av någon obegriplig anledning blev sortering på antal inlägg helt galet. Kanske returnerar LOWER() inget eller odefinierat svar om man matar den med tal.

Peter.

[tumba25]

Jag sorterar på antal inlägg och reg-datum utan problem med just denna lösning. Fält som bara innehåller siffror passerar lower() utan att ändras.

Det kan varar något med din databas? Vad har du för databas?

[Michanek]

Jag kör på MySQL 5.0.45 och hade inga problem med datumfälten men antal inlägg blev i oordning.

[tumba25]

Jag ser problemet. LOWER() returnerar ju alltid text. Alltså hamnar 18 före 2. Ska ordna en fix för det.

[Michanek]

Min egen lösning är att testa vilket fält som ska sorteras och bara använda LOWER() på de fält som behöver det.

Kod: Markera allt

if ($sort_key == 'b' or $sort_key == 'q')
{
	$order_by .= 'LOWER(' . $sort_key_sql[$sort_key] . ') ' . (($sort_dir == 'a') ? 'ASC' : 'DESC');
}
else
{
	$order_by .= $sort_key_sql[$sort_key] . ' ' . (($sort_dir == 'a') ? 'ASC' : 'DESC');
}

Fält 'q' är ett eget profilfält som är det som jag oftast vill sortera på.

Peter.