@
oxwivi
Your mistaken, this thread refers to code that appeared in osC2.2 in order to locate the area that requires alteration, it did not call for the code to be changed,
ie 2.2 has the line:
if ( (!isset($HTTP_GET_VARS['sort'])) || (!ereg('^[1-8][ad]$', $HTTP_GET_VARS['sort'])) || (substr($HTTP_GET_VARS['sort'], 0, 1) > sizeof($column_list)) ) {
which was unchanged by this mod.
but in 2.3 that line became
if ( (!isset($HTTP_GET_VARS['sort'])) || (!preg_match('/^[1-8][ad]$/', $HTTP_GET_VARS['sort'])) || (substr($HTTP_GET_VARS['sort'], 0, 1) > sizeof($column_list)) ) {
which should also be left un-changed, what you did was replace the new 2.3 code with old 2.2 code un-neccesarily
Also as the page
Bruyndoncx pointed states, you cannot simply replace ereg with preg, you must use the appropriate replacement function and modify the regex.