Petsmacker Posted February 9, 2008 Share Posted February 9, 2008 How can this be written properly? if ((isset($_GET['do'])=='read') && (isset(ereg("^[0-9]+$", $_GET['id'])))){ The error it gives: Parse error: syntax error, unexpected T_STRING, expecting T_VARIABLE or '$' in /home/tbhgweb/public_html/mail.php on line 133 Its the ereg part it doesn't like. Quote Link to comment Share on other sites More sharing options...
kenrbnsn Posted February 9, 2008 Share Posted February 9, 2008 What are you trying to do? You can't use an ereg withing an isset. Ken Quote Link to comment Share on other sites More sharing options...
Petsmacker Posted February 9, 2008 Author Share Posted February 9, 2008 I'm trying to get it so that it only allows a certain section of the page to be shown when: mysite.com/page.php?do=read&id=4 When id is an integer. Quote Link to comment Share on other sites More sharing options...
kenrbnsn Posted February 9, 2008 Share Posted February 9, 2008 Try <?php if (isset($_GET['do']) && $_GET['do'] == 'read' && isset($_GET['id']) && ereg("^[0-9]+$", $_GET['id'])) ?> You have to test each condition separately. Ken Quote Link to comment Share on other sites More sharing options...
Petsmacker Posted February 9, 2008 Author Share Posted February 9, 2008 Magic, it works. If theres any other way of shortening it please let me know. I used to only have it like this: if (($_GET['do']=='read') && (ereg("^[0-9]+$", $_GET['id']))){ And it was all fine, but then I asked to have diplay_errors on and my site went slightly mental. But if what you suggested is the best, I shall go with it, cheers! Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.