коварный location.href

[cyklop77]

уважаемые, помогите пожалуйста советом

написал следующий костыль для интернет-магазина

if(/.*\/cart.*/.test(location.href)){
console.log('cart');
}

есть сомнения по поводу правильности использования location.href . в таком виде оставлять безопасно? проблем не будет неожиданных?

запись правильная? document. не нужен?

[sigma77]

А в чем цель этого кода?

[SelenIT]

Запись правильная. AFAIK, document.location считается устаревшим, актуален window.location, а window. можно безболезненно опускать. Неожиданности могут быть, например, если в структуре сайта появится категория cartridges)

[cyklop77]

А в чем цель этого кода?

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

[sigma77]

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

Скрытие сайдбара, урл и заполнение формы не очень коррелируются между собой )

Ок. Задам вопрос по другому "cart" в url - это что? Папка, файл, запрос, часть имени папки, файла, запроса. Возможно задача решается другими способами.

[cyklop77]

cart это просто часть адреса. я до конца не разобрался в cms, но видимо существует роут, который видя адрес типа siteurl/cart/* ,загружает представление корзины

по уму конечно такие вещи нужно решать другим способом) я даже знаю как переопределить шаблон для страниц типа "корзина". но этот способ не так удобен как js-костыль потому что придётся править код в двух местах если например изменится версия jquery или придётся подключить еще скриптик

а скрываю элементы потому что думаю, что если уж пользователь решился на покупку, то его лучше уже не отвлекать ничем(типа подписки на новости или ссылкой на лучший товар месяца))

[sigma77]

Тогда это можно и без регулярки сделать - http://jsfiddle.net/cdR65/

И для поиска конкретной подстроки есть метод indexOf().

А вообще, не знаю, что за CMS, но я бы попправила в шаблоне.

[cyklop77]

Тогда это можно и без регулярки сделать - http://jsfiddle.net/cdR65/

И для поиска конкретной подстроки есть метод indexOf().

А вообще, не знаю, что за CMS, но я бы попправила в шаблоне.

спасибо. этим способом можно даже избежать проблемы с cartrige, про которую selenit писал выше

cms отличная. drupal7. и там проблема решается просто добавлением page--cart.tpl.php . но из-за моей привычки встраивать js-скрипты в конце документа, нормальный документированный способ не удобен так как придётся очень много кода переносить в head и потом еще проверять его потому что некоторые скрипты могут перестать работать. это не сложно, но так как js очень много и работаю над этим проектом фактически бесплатно, то костыль не очень отягощает мою совесть)

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

Edited August 21, 2013 by cyklop77

[Vin]

Раз отдельный шаблон создать нельзя, уместнее, по-моему, было бы выводить форму в модальном окошке (лайтбоксе).