Как проверить Request.path в Django?

Question

[zelsky]

Вот код urls.py

url(r'^category/(?P<cat_id>\d+)$',  'adst.views.alboms', name='Albom'),

Надо проверить по данном урл и добавить клас к ссылке activate
Пытался вот так. Не помогло.

{% for cat in categor %}
          <li  {% if  cat.id_cat in request.path %}class="active"{% endif %} >
             <a href="{% url "Albom" cat.id_cat %}">{{ cat.Name_cat }}</a>
             <span class="sr-only">(current)</span>
              </li>
              {% endfor %}

Answer 1

[marazmiki]

А точно id_cat? По умолчанию джанга делает совсем не так. Обычно у моделей первичный ключ называется id. Или, если использовано нестандартное название, можно обратиться к нему через алиас pk.

Таким образом, если я правильно понял, что речь идёт о переборе кверисета, то условие должно выглядеть как-то так:

<li  {% if  cat.id in request.path %}class="active"{% endif %} >

Не самое, надо сказать, удачное решение. Правильнее было бы, к примеру, реализовать у модели метод get_absolute_url() и сравнивать request.path с ним в цикле.

Comments

[zelsky]

id_alb = models.AutoField(primary_key=True)

[sim3x]

BlastPy: не надо учить ОРМ жизни
убеи id_alb и используй pk

Answer 2

[Roman K]

id_alb = models.AutoField(primary_key=True) - плохая практика. С какой целью вы изменили стандартный id на точно такой же, но назвали id_alb?
В общем-то, это не единственное замечание к Вашему коду. URL у Вас называется с заглавной буквы и, вообще, называется русским транслитом (очень плохая практика). {% url 'Albom' cat.id_cat %} нужно заменить на {{ cat.get_absolute_url }} и, соответственно, в models.py добавить метод модели Cat:

# В начале:
from django.core.urlresolvers import reverse
# Метод модели:
def get_absolute_url(self):
    return reverse('name_of_your_detail_view_for_cutties', args=[self.pk])

Название поля cat.Name_cat - дублирование + заглавная буква в переменной (а нужно cat.name). в URLах регулярка должна заканчиваться /$, а не просто $. В общем, дело Ваше, но я советую почитать PEP8.
Ответ на Ваш вопрос намного проще:

$(function(){
  function stripTrailingSlash(str) {
    if(str.substr(-1) == '/') {
      return str.substr(0, str.length - 1);
    }
    return str;
  }

  var url = window.location.pathname;
  var activePage = stripTrailingSlash(url);

  $('.nav li a').each(function(){
    var currentPage = stripTrailingSlash($(this).attr('href'));

    if (activePage == currentPage) {
      $(this).parent().addClass('active');
    }
  });
});

Comments

[zelsky]

PEP8 следующий в плане.
Тоесть по моему методу зделать никак ?

[Roman K]

BlastPy: Вашим способом могут вызываться лишние срабатывания (допустим, активная вкладка с pk - 2, а URL какой-то другой - 122 или 222 или вообще что угодно, где есть двойка). Но если УЖ ОЧЕНЬ ХОЧЕТСЯ - проверьте, идёт ли в контекст request.

[zelsky]

Roman Kitaev: использую для категорий И не думаю что будет больше 10 категорий.

[zelsky]

Roman Kitaev: Roman Kitaev: использую для категорий И не думаю что будет больше 10 категорий. вопрос можно ли исправить то что не работает.

[Roman K]

BlastPy: Ещё раз говорю, проверьте, идёт ли request в контекст.

[zelsky]

Roman Kitaev: как ?

[Roman K]

BlastPy:
1) Если Вы используете обычные функции в качестве вьюх - используйте render вместо render_to_response (там первым параметром вставляете request и он идёт в контекст).
2) Если вы НИЗАЧТО не хотите использовать render - stackoverflow.com/questions/702592/django-request-...
3) Самый лучший способ - юзать CBV. Вюха должна наследоваться от TemplateView (очевидно, либо от её наследников - DetailView, ListView, UpdateView, FormView и т.д.). Тогда просто используйте {{ view.request }} вместо {{ request }} или переопределите get_context_data.

Answer 3

[Vladisus]

советую посмотреть на django-activelink