переменная $_POST и JS

[voral]

<form method="post">
	<input name="login" type="text">
	<input name="submit" type="submit" class='btn' id="submit"></form>
<?php
if(empty ($_POST['login'])) {?>	

<script>
  window.onload = function() {
    swal("Password or Login incorrect!");
  }
</script>
<?php 
} else {echo 'login correct';}
?>

[arimanecro]

почти то что надо, огромное спасибо

но сразу появляется проблема, как только я захожу на страницу, скрипт автоматом срабатывает(выскакивает окно), видимо поэтому и нужно было onclick = function(). Кажись тупиковая ситуация

[voral]

Нет
Ну "онклик" же явно говорит, что функция срабатывает на клик. А вам это не надо. Соответсвенно ваша ошибка в php.
Вы проверяете пустой или нет $_POST['login'], а вам нужно проверить есть ли он вообще. if (isset($_POST['login']))..... и вы этом случае не выводить ни чего. В настоящий момент при этой ситуации php скрипт попадает в ваше условие заставляющее отобразить окно с ошибкой

<?php
  if (isset($_POST['login'])) {
    if(empty ($_POST['login'])) {?> 
       <script>
          window.onload = function() {
             swal("Password or Login incorrect!");
          }
       </script>
<?php
    }
    else {
       echo 'login correct';
   }
}
?>

[arimanecro]

voral,
да, Вы совершенно правы, я по своей невнимательности, забыл сначала проверить на isset.
Благодарю ещё раз, за столь быстрые и правильные ответы
В карму смог добавить только один плюсик, больше не получается, из-за того, что требуется ещё кому-нибудь поставить (

[danik.js]

<input name="login" type="text" required> и фиг ты отправишь пустое поле, по крайней мере в современных браузерах)
Но это не отменяет необходимость серверной проверки.

[kostyanet]

Так не делают, это уродство. Когда страница перезагружается то все сообщения валятся прямо в html, а когда страница не перезагружается, то все сообщения валятся в js-скрипт, который затем валит их в html.

В принципе можно и алерты втюхивать, onload тут вообще не при делах. Как только парсер дошел до алерта - показал. Потому что все уже решено до того как.

[kostyanet]

Вот так сделайте, как людей:

В процессе валидации просто собираете ерроры в массив, например

if(!validate($var))
$error[]='Нет такого юзера, или юзер ввел пароль неправильно';

Затем в рендере у вас, например

<div class="error_message">
 ?>
  if(count($errors))
     echo '<ul><li>', join('</li><li>',$errors), '</li></ul>';
 <?php
</div>

и все. Если ошибки были - они в хтмле проявятся. А если не было, этот div вообще не будет виден, ибо это div, а пустой div занимает 0 целых Х десятых места.

Более того, если вы захотите запросить данные с того же скрипта, то попросту выделяете этот кусок в отдельный файл (или метод, или функцию) и вызываете его (его, ее) единственно когда случились ошибки. Тогда по XMLHttpRequest скрипт получит точно такой же кусок хтмля и вставит его в то же самое место. - Это для случая без перезагрузки.

[danik.js]

ибо это div, а пустой div занимает 0 целых Х десятых места.

А если ему назначен padding и иконка/бэкграунд?

[voral]

А если не было, этот div вообще не будет виден, ибо это div, а пустой div занимает 0 целых Х десятых места.

Ну уж если все "вылизывать" то зачем вообще этот div выводить? Это сейчас он пустой, а завтра обновят на сайте стили....

На вопрос "зачем". Тут все от общей задачи зависит. Так просто "это не правильно" говорить не надо. Возможно окно строится общим для сайта скриптом. И используется скрипт в различных ситуациях. А может шаблон окна уже есть в html.