ereg to preg_match?

I have an old plugin that I am updating and I see ereg which was replaced by preg_match in php7.

Here is the code:

if ( (!isset($_GET['sort'])) || (!ereg('[1-9][ad]', $_GET['sort'])) || (substr($_GET['sort'], 0, 1) > sizeof($define_list)) ) {

Would it be changed to?

if ( (!isset($_GET['sort'])) || (!preg_match('/[1-9][ad]', $_GET['sort'])) || (substr($_GET['sort'], 0, 1) > sizeof($define_list)) ) {

Close! You'll want to add the ending delimiter, too:

if ( (!isset($_GET['sort'])) || (!preg_match('/[1-9][ad][B]/[/B]', $_GET['sort'])) || (substr($_GET['sort'], 0, 1) > sizeof($define_list)) ) {  

you are missing a slash

if ( (!isset($_GET['sort'])) || (!preg_match('/[1-9][ad]', $_GET['sort'])) || (substr($_GET['sort'], 0, 1) > sizeof($define_list)) ) { 

should be

if ( (!isset($_GET['sort'])) || (!preg_match('/[1-9][ad]/', $_GET['sort'])) || (substr($_GET['sort'], 0, 1) > sizeof($define_list)) ) { 

Great minds think alike:P By the timestamp, I note a virtual tie. We may need to go to the tape for the winner on this one.:laugh: