register_globals = off, и это правильно - подробности можно посмотреть здесь . Твой код должен был выглядеть так:
<?php
Error_Reporting(E_ALL);

print "Добро пожаловать <b>$_POST[user]</b><p>\n\n";
print "Ваш адрес: <p>\n\n<B>$_POST[address]</b>";
print "Your product choices are:<p>\n\n";
print "<ul>\n\n";
foreach ($_POST['products'] as $value){
print "<li>$value<br>\n";
}
print "</ul>";

Надо заметить, что этот код будет работать всегда, не зависимо от состояния register_globals

SDN , при чём тут foreach, когда "проблема" в том, что переменные передаются в массивах $_POST и $_GET?

SDN , ошибки надо исправлять последовательно. Вышеуказанный автором постинг явно говорил о том, что не видно любых переменных, независимо от того, являются ли они массивом или нет и как к ним происходит доступ - первоначально необходимо было устранить именно это. Фраза же "всё проще" может рассматриваться ни как иначе, как панацея, т.е. полное решение проблем, что неверно. То, что указали вы, не решит проблему до конца - если не будет выделен ни один элемент - предупреждение останется.