Баг сортировки по дате

Вот отловил ещё одного жука, причем в уже знакомом месте - в сортировке. Бинарно-безопасное сравнение значений не справилось с корректной сортировкой по данным вида "21/01/2006" и выдало совсем не то, что надо.

Как бороться? Если вашей целью стоит сортировка по полю DATE, а не DATE_CREATE или ID, т.е. в процессе добавления новостей у вас случается, что вы добавляете новости задним числом или же наоборот, опережая время, вам не обойтись без это "латалки".

Итак, открываем /usr/system/common_extfunctions.inc.php (для большей ясности в примере использую стандартный файл, а не тот, что я уже подправлял, как описывалось ранее в этом блоге). Далее опускаемся в низ и находим там вот такие строки:

if(!defined("cmp_indexasc")) {
function cmp_indexasc($a, $b) {
global $sapi_obj;
$index = $sapi_obj->env["index"];
return strcmp($a[$index], $b[$index]);
}
define("cmp_indexasc", 1);
}


if(!defined("cmp_indexdesc")) {
function cmp_indexdesc($a, $b) {
global $sapi_obj;
$index = $sapi_obj->env["index"];
return strcmp($a[$index], $b[$index]);
}
define("cmp_indexdesc", 1);
}

Кстати обратите внимание, тут ещё есть баг, который очевиден - и в случае ASC сортировке и в случае DESC производится одна и та же сортировка. Но сейчас мы все это поправим, а заодно не забудем про ранее описанный баг сортировки по ID. Заменяем это дело на следующее:

if(!defined("cmp_indexasc")) {
function cmp_indexasc($a, $b) {
global $sapi_obj;
$index = $sapi_obj->env["index"];
if($index=='ID')
return ($a[$index] > $b[$index]) ? -1 : 1;
else if ($index=='DATE'){
$date_a=explode('/',$a[$index]);
$date_b=explode('/',$b[$index]);
$a_index=mktime (0,0,0,$date_a[1], $date_a[0], $date_a[2]);
$b_index=mktime (0,0,0, $date_b[1], $date_b[0], $date_b[2]);
return strcmp($a_index,$b_index);
}
else
return strcmp($a[$index], $b[$index]);
}
define("cmp_indexasc", 1);
}


if(!defined("cmp_indexdesc")) {
function cmp_indexdesc($a, $b) {
global $sapi_obj;
$index = $sapi_obj->env["index"];
if($index=='ID')
return ($a[$index] > $b[$index]) ? 1 : -1;
else if ($index=='DATE'){
$date_a=explode('/',$a[$index]);
$date_b=explode('/',$b[$index]);
$a_index=mktime (0,0,0, $date_a[1],$date_a[0], $date_a[2]);
$b_index=mktime (0,0,0, $date_b[1], $date_b[0], $date_b[2]);
return strcmp($b_index,$a_index);
}
else
return strcmp($b[$index],$a[$index]);
}
define("cmp_indexdesc", 1);
}

Скачать: common_extfunctions.inc.zip
Для версии: Sapid 1.2.3 RC3

Если кто-то будет это использовать, пожалуйста, отпишитесь о результатах в комментариях.