Хочу сделать перенаправление на страницу «спасибо» Что не так в моем JS?

Question

[Вася]

Хочу сделать перенаправление на страницу "спасибо" Что не так в моем JS

Вот так работает с выводом текста на экран "Сообщение успешно отправлено"

// Отправка данных на сервер
$('#form').trigger('reset');
$(function() {
  'use strict';
  $('#form').on('submit', function(e) {
    $('.msg').removeClass('error success');
    $('input').removeClass('inputerror');
    loadanim.play();
    e.preventDefault();
    $.ajax({
      url: 'send.php',
      type: 'POST',
      contentType: false,
      processData: false,
      data: new FormData(this),
      success: function(msg) {
        console.log(msg);
        if (msg == 'ok') {
          $('#form').trigger('reset');
          $('.amount').text('Выберите файлы');
          $('label').removeClass('active');
          $('.msg').text('Сообщение успешно отправлено').addClass('success');
          showmsg.restart();loadanim.duration(0.3).reverse();
        } else {
          if (msg == 'mailerror') {
            $("#email").addClass('inputerror');
          }
          $('.msg').text('Ошибка. Сообщение не отправлено').addClass('error');
          showmsg.restart();loadanim.duration(0.3).reverse();
        }
      }
    });
  });
});

А вот так не работает с перенаправлением на страницуt: hanks.html

// Отправка данных на сервер
$('#form').trigger('reset');
$(function() {
  'use strict';
  $('#form').on('submit', function(e) {
    $('.msg').removeClass('error success');
    $('input').removeClass('inputerror');
    loadanim.play();
    e.preventDefault();
    $.ajax({
      url: 'send.php',
      type: 'POST',
      contentType: false,
      processData: false,
      data: new FormData(this),
      success: function(msg) {
        console.log(msg);
        if (msg == 'ok') {
          $('#form').trigger('reset');
          $('.amount').text('Выберите файлы');
          $('label').removeClass('active');
          $('.msg')window.location.href = "thanks.html"; .addClass('success');
          showmsg.restart();loadanim.duration(0.3).reverse();
        } else {
          if (msg == 'mailerror') {
            $("#email").addClass('inputerror');
          }
          $('.msg').text('Ошибка. Сообщение не отправлено').addClass('error');
          showmsg.restart();loadanim.duration(0.3).reverse();
        }
      }
    });
  });
});

Answer 1

[Алексей Ярков]

$('.msg')window.location.href = "thanks.html"; .addClass('success');

А вы чего ожидали от этой строчки? Разберитесь с синтаксисом JS прежде что-то на нём писать.
В вашем случае достаточно так:

if (msg == 'ok') {
    window.location.href = "thanks.html";
}

Comments

[Вася]

Вопрос куда делись эти строчки?
$('#form').trigger('reset');
$('.amount').text('Выберите файлы');
$('label').removeClass('active');

И вообще нужны ли они если перенаправление на другую страницу?

[Алексей Ярков]

Dsub92, не нужны, поэтому и убрал. Зачем манипулировать DOM, если всё равно сейчас уходить на другую страницу?

[Вася]

Посмотри пожалуйста в коде ничего больше лишнего нету?

// Код для Input file
$('.my').change(function() {
  if ($(this).val() != '') $('.amount').text('Выбрано файлов: ' + $(this)[0].files.length);
  else $('.amount').text('Выберите файлы');
});

// Для анимации label у input
$(".form-input, .form-text").focus(function() {
  $(this).siblings('label').addClass('active');
});
$(".form-input, .form-text").focusout(function() {
  if ($(this).val() != false) {
    $(this).siblings('label').addClass('active');
  } else {
    $(this).siblings('label').removeClass('active');
  }
});

// Анимация сообщения о результате
var showmsg = new TimelineMax();
showmsg.add(TweenMax.to(".msg", 0.7, {opacity: 1,y: -40,ease: Expo.easeOut}));
showmsg.add(TweenMax.to(".msg", 0.7, {opacity: 0,y: 0,ease: Expo.easeOut,delay: 2}));
showmsg.pause();

// Анимация плашки слова "подождите"
var loadanim = TweenLite.to(".loading", 1, {autoAlpha: 0.8});
loadanim.pause();

// Отправка данных на сервер
$('#form').trigger('reset');
$(function() {
  'use strict';
  $('#form').on('submit', function(e) {
    $('.msg').removeClass('error success');
    $('input').removeClass('inputerror');
    loadanim.play();
    e.preventDefault();
    $.ajax({
      url: 'send.php',
      type: 'POST',
      contentType: false,
      processData: false,
      data: new FormData(this),
      success: function(msg) {
        console.log(msg);
        if (msg == 'ok') {
          window.location.href = "thanks.html"; 
        } else {
          if (msg == 'mailerror') {
            $("#name").addClass('inputerror');
            $("#phone").addClass('inputerror');
          }
          $('.msg').text('Ошибка. Сообщение не отправлено').addClass('error');
          showmsg.restart();loadanim.duration(0.3).reverse();
        }
      }
    });
  });
});

 $(document).ready(function() {
  $("#phone").mask("[phone]");
  }); 

 $('#name').on('keypress', function() {
    var that = this;
    setTimeout(function() {
        var res = /[^А-Я а-я A-Z a-z]/g.exec(that.value);
        console.log(res);
        that.value = that.value.replace(res, '');
    }, 0);
});

[Алексей Ярков]

Вася,

var that = this;

Давно уже есть стрелочные функции, чтобы от этой дичи избавляться.

[Вася]

Это как?

[Алексей Ярков]

Вася, вот так