Angela, the variable $current_page does not exist. so $curent_page does not equal $saved_page. Also the ! in front of the entire statement means that all of this is false. Since one items is true and not true = false "Don't save" is echoed out. If you are looking to save the results based on the above sample it would look like if(($page === $saved_page) && ($current_ip === $saved_ip) && ($current_dt < ($saved_dt +3600)) { //save the results } else { //don't save } I would also suggest keeping with your original statement to return early and return often. Its best to exit out of your functions sooner than later. Specially if its a large function. On Mon, Mar 11, 2013 at 6:03 PM, Angela Barone <angela@xxxxxxxxxxxxxxxxxxxx>wrote: > On Mar 11, 2013, at 3:47 PM, Ashley Sheridan wrote: > > if ( !( ($current_page == $saved_page) and ($current_ip == $saved_ip) > and ($current_dt < ($saved_dt + 3600)) ) ) > > Hello Ash, > > This makes sense to me, but I can't get it to work, so I'm either > not understanding it or I'm asking the wrong question. Here's a complete > scriptlet: > > <?php > $saved_page = 'ddd'; > $page = 'ddd'; > $saved_ip = '1.1.1.1'; > $ip = '1.1.1.1'; > $saved_dt = '2013-03-11 11:11:11'; > $current_dt = '2013-03-11 11:22:11'; > > if ( !( ($current_page == $saved_page) and ($current_ip == $saved_ip) and > ($current_dt < ($saved_dt + 3600)) ) ) { > echo 'Save results.'; > } else { > echo "Don't save."; > } > ?> > > Using the supplied data, the result should be "Don't save." Can > you see what's wrong? > > Angela > -- > PHP General Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php > >