Частичная работа JavaScript

Question

[Никита Склюев]

Здравствуйте уважаемые хабрачеловеки.
Столкнулся с такой неприятностью, сделал 1 из клиентов новое оформление админпанели самописного движка. Для этого использовал Twitter Bootstrap. Проверил во всех браузерах, все отлично работает, но звонит клиент и говорит что не работает ряд функций, я естественно быстро за проверку, у меня сново все работает, клиент присылает видео, где действительно не работают описанные им функции.
А не работает переключение табов (табы из коробки Bottstrap) а так же не работает следующая конструкция кода:

$("input[name=cat]").live("click", function(){
if($(this).is(":checked")){
$("select[name=cat]").attr({"disabled":"disabled"});
}else{
$("select[name=cat]").removeAttr("disabled");
}
});


Причем ошибок осел не выкидывает вообще никаких.

Проверял на аналогичных браузерах у себя на компе, все работает на ура. А модальные окна и Bootstrap работают нормально у клиента. Получается что половина функционала куда то пропадает.
Видимо у клиента стоят какие то ограничения, или что то урезает JS.

Сейчас нужно поехать к клиенту, что бы посмотреть на его компе в чем дело, но хотелось бы поехать с конкретными мыслями как это можно поправить.

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

P.S.ОС Windows 7, IE9, Антивирус Касперского!

Answer 1

[lashtal]

кривые правила adblock съедают что-то?

Answer 2

[Silver_Clash]

Попробуйте $('select[name=«cat»]').
Включите вывод ошибок js и посмотрите что пишет браузер.

Comments

[Silver_Clash]

Attribute Equals Selector

Answer 3

[Никита Склюев]

Просто этот код работает в аналогичном осле, только на другом компе. ладно, если тут еще можно поэксперементировать, то как быть с табами bootstrip?

Comments

[Silver_Clash]

Сложно посоветовать что то конкретное. Включаем вывод сообщений об ошибках и проверяем. Если ошибок нет, то внимательно проверяем работу селекторов, может где то в код страницы вкралась ошибка.

Answer 4

[Silver_Clash]

А вообще вместо input[name=cat] лучше обращаться к элементу по id (надеюсь он у Вас один).

Еще очень интересна эта конструкция: .attr({«disabled»:«disabled»});
Я почему то всегда считал что должно быть: .attr(«disabled»,«disabled») (jQuery .attr).

Comments

[ngreduce]

map он тоже принимает.

[Homakov]

disabled: true куда наглядней

[ngreduce]

а потом какой-то умник напишет false и ничерта не наглядно.

Answer 5

[Никита Склюев]

эллемент не 1 поэтому использовать id не пойдет.
а attr({«disabled»:«disabled»}); использую по привычке, когда добавляю много атребутов.

Comments

[Никита Склюев]

и проблема не в этом участке, при снятии галочки селект должен активироваться, а он не реагирует

Answer 6

[altrosilka]

Была похожая проблема. Мне помогло когда-то, не помню сути, если честно.
$('select[name=cat]').attr({'disabled':'disabled'}); => $('select[name=cat]').attr({'disabled':''});

Answer 7

[Никита Склюев]

Меня сейчас не столько конструкция $('select[name=cat]').attr({'disabled':'disabled'}); волнует сколько нерабочие табы. Селект я еще как то могу обойти, но табы это катастрофа!

Answer 8

[MT]

Я бы попробовал временно отключить Антивирус Касперского.

К элементам форм, кстати, имеет смысл обращаться так:
form.elements['cat']

Answer 9

[Dmitry Tallmange]

Ваш js код (конкретно бутстрапа) минифицируется? Возможно, собирается в один js файл с другими скриптами?

Comments

[Dmitry Tallmange]

Сразу по сути: у нас при конкатенации бутстрапа (что файлов версии 1.х, что скрипта уже версии 2.0) компилер глотал некоторые символы. А все из-за того, что разработчики не пользуются прекрасным символом ";" в конце строки.

[Никита Склюев]

Нет, помимо bootstrap на странице подключается jQuery, и код отправки ajax запроса, больше ничего нету!

[Dmitry Tallmange]

Скрипты нужно немного править, либо исключать из минификации/конкатенации

[Dmitry Tallmange]

Версия бутстрапа?

Answer 10

[Dmitry Tallmange]

Посмотреть на проблему вживую, конечно же, невозможно?

Comments

[Никита Склюев]

К сожалению проблема в недрах админ панели 1 сервиса!

Answer 11

[Никита Склюев]

Версия bootstrap v1.4.0

Answer 12

[Никита Склюев]

Вот весь js используемый на странице:




Answer 13

[Никита Склюев]

$(document).ready(function(){

$('.tabs').tabs();

$("form.editorV").live("submit", function(){
var _id=$(this).attr("id");
$("#"+_id+" button").button('loading');
$.ajax({
type: 'POST',
action: "http://payservice.uz/admin.php?cont=vauch",
url: "http://payservice.uz/admin.php?cont=vauch",
dataType: 'html',
async: true,
timeout: 15000,
data: {
ajax: 'update_vaucher',
cat: ($("#"+_id+" .catterc").is(':checked')?$("#"+_id+" .catterc").val():$("#"+_id+" .catter option:selected").attr("value")),
id: $("#"+_id+" input[name=vauchid]").val(),
pin: $("#"+_id+" textarea[name=pin]").val(),
numinal: $("#"+_id+" input[name=numinal]").val(),
usd: $("#"+_id+" input[name=usd]").val(),
descr: $("#"+_id+" textarea[name=descr]").val(),
status: $("#"+_id+" select[name=status]").val(),
date: $("#"+_id+" input[name=date]").val()
},
success: function(data){
$("#"+_id+" button").button('complete');
$("#"+_id+" input[name=cat]").val(($("#"+_id+" input[name=cat]").is(':checked')?$("#"+_id+" input[name=cat]").val():$("#"+_id+" select[name=cat] option:selected").attr("value")));
$("#"+_id+" .cata").text(($("#"+_id+" input[name=cat]").is(':checked')?$("#"+_id+" input[name=cat]").val():$("#"+_id+" select[name=cat] option:selected").attr("value")));
}
});
return false;
});

$(".modal").on("hidden",function(){
var _idsM=$(this).attr("id");
$("#"+_idsM+" button").button('reset');
$("input[name=cat]").attr({"checked":"checked"});
});
$("select[name=cat] option:first").attr({"selected":"selected"});

$(".modal").on("show", function(){
var mID=$(this).attr("id");
$('#'+mID+" .catterc").live("click", function(){
if($('#'+mID+" .catterc").is(":checked")){
$('#'+mID+" .catter").attr({"disabled":"disabled"});
}else{
$('#'+mID+" .catter").removeAttr("disabled");
}
});
});

});


Answer 14

[Никита Склюев]

Перенос строк в коде сделал хабрапарсер!

Answer 15

[Dmitry Tallmange]

По вашему коду: как вариант заменить .attr() на .prop()
По поводу клиента. Непременно запустить код на другом браузере, чтобы исключить или подтвердить участие системы/антивируса в этом злодеянии

Comments

[Никита Склюев]

Да, вот этим завтра с утра и собираюсь заняться на машине клиента.
что касается attr, то это не критичный момент, самое критичное это табы, так как много разнообразной информации в табах!

[Dmitry Tallmange]

В представленном коде, на мой беглый взгляд, ничего критичного нет

Answer 16

[Никита Склюев]

Так и я о том, из всего не работает так как должно только: $('.tabs').tabs();

Comments

[Dmitry Tallmange]

Ну значит у заказчика открываете js-код, отвечающий за табы и дебажите строку за строкой, определяя где именно он перестает работать и что происходит.

[Dmitry Tallmange]

Заодно, при удачном исходе, заявите заказчику: Я же говорил, что все работает! Это у вас тут система/антивирус/что-то кривое)))

[Dmitry Tallmange]

И еще один совет, возможно прислушаетесь: посмотрите в сторону Backbone или чего-то подобного. Вход не очень прост, зато профит после вникания колоссальный.

[Dmitry Tallmange]

И это не из соображений поумничать. Код и отладка упрощаются в разы, ибо инкапсуляция как-никак.

[Никита Склюев]

Спасибо за совет, но мне jQuery пока хватает вдоволь! Тем более насколько я бегло просмотрел в нем нет готовых методов анимации, и все равно советую использовать вместе с jQuery. Все таки предназначение у них разное. Сомневаюсь что в нем таких проблем не возникнет, если антивирус начнет резать код!

Answer 17

[Никита Склюев]

Как оказалось все это творит KIS. Пришлось сайт добавить в исключение

Comments

[Никита Склюев]

Или кто то знаетболее изящный выход из этой ситуации?