[laimas]

Отвечаю, что не так в структуре:

1) В самом начале запускается вызов функции на клиенте - scroll(), откуда ей взяться и что она должна выполнить, если сама страница выводится далее?

2) Ладно, пусть бы это было сделано грамотно и не было бы проблем, но почему вызов этой функции в блоке, который проверяет данные пришедшей формы?

3) Ранее говорилось о завершении операции входа, то есть действии сервера, и у вас определен переход на "саму себя" - location. Можно и так, не проблема, но что после такого перехода увидит пользователь, если логика вашего кода кроме вывода формы не подразумевает ничего?

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

В начале также определяются значения полей формы - если форма получена, значит они равны соответствующим значениям полям формы, иначе NULL. Эти значения (кроме пароля) подставляются в форму.

Далее идет проверка формы, если она была принята, и если ОК, значит пишется в сессию признак, передается заголовок осуществляющий переход, и выдох из скрипта. Если ошибки, значит формируется сообщение в определенной переменной.

Далее вывод непосредственно html-кода страницы включая и клиентских сценариев, если нужно, с подключением по условию "вход/нет входа" шаблонов, а также вывод ошибок.

***********

У вас же просто каша - серверным сценарием вы пытаетесь управлять клиентом, не фактически, а по логике. При этом "проверки и т.д." и "обработчик по клику кнопки", что являющееся вообще-то одной задачей, у вас расколота на части с непонятным по логике определением места им в структуре кода - часть в начале, часть в самом конце.