Django - Підручник 022. Додавання системи закладок (вибране) на сайт

Доброго времени суток Евгений. Не подскажете как сделать иконки (лайк-дизлайк) кликабельными?

Доброго времени суток Евгений. Не ясен метод:

def get_bookmark_count(self):
    return self.bookmarkarticle_set().all().count()

Не могли бы вы растолковать откуда моя модель статьи знает о этой функции?:

self.bookmarkarticle_set()

Когда применяю данный метод то выскакивает ошибка:

__call__() missing 1 required keyword-only argument: 'manager'

И второй вопрос каким образом можно подсвечивать закладку (допустим менять ее цвет когда она добавлена пользователем, ну как у вас на сайте)?

Добрый день, Игорь!

Вообще это стандартное поведение в моделях, если на данную модель какая-то иная модель имеет внешний ключ (ForeignKey)

В данном конкретном случае есть модель Article и BookmarkArticle , поскольку BookmarkArticle имеет внешний ключ на Article, то модель Article автоматически получает метод bookmarkarticle_set, который возвращает query set  всех закладок, которые имеют внешний ключ на конкретный объект статьи.

Здесь я по ходу допустил ошибку, напишите так

self.bookmarkarticle_set.all().count()

Для подсветки я использую фильтр в шаблоне, которые ищет пользователя в наборе

@register.filter
def user_in(objects, user):
    if user.is_authenticated:
        return objects.filter(user=user).exists()
    return False

а вот его применение

<span data-icon="favorite" class="mdi mdi-star mr-1 {% if obj.bookmarks.all|user_in:user %}text-success{% endif %}"></span>

self.bookmarkarticle_set.all().count()

Да, так все работает! Спасибо вам огромное.

Еще один вопрос:

Я подключил фильтр:

@register.filter
def user_in(objects, user):
    if user.is_authenticated:
        return objects.filter(user=user).exists()
    return False

в templatetags/movie_extras.py

в шаблоне:

{% load movie_extras %}
<span style="cursor: pointer" class="fa fa-bookmark-o fa-4x{% if movie.bookmarks.all|user_in:user %}fa-2x{% endif %}"></span>

При этом ошибка:

 File "/home/chunk/public/web-dev/testproj/movie/templatetags/movie_extras.py", line 8, in user_in
    return objects.filter(user=user).exists()
AttributeError: 'NoneType' object has no attribute 'filter'

что я делаю не так Евгений?

мне кажется, вы неправильно записали это

movie.bookmarks.all

Как у вас называется модель закладок для фильмов? BookmarkArticle? BookmarkMovie?

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

movie.bookmarkmovie_set.all

Или что-то типо того

Все верно.

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

self.bookmarkarticle_set.all().count()

Так все работает. Только у меня:

self.bookmarkmovie_set.all().count()

Фильтр к сожалению не работает.

так перепишите его так

{% if movie.bookmarkmovie_set.all|user_in:user %}fa-2x{% endif %}

Все работет Евгений. Спасибо вам.

Доброго времени суток Евгений. Можно поинтересоваться каким образом выводите закладки в профиле пользователя?

Добрый день, Игорь.

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

user.bookmarkmovie_set.all()

Спасибо Евгений, буду пробовать.

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

Покажите шаблон, где выводите закладки, но без контента

{% extends 'base/base.html' %}
{% block content %}
<div class="userprofile">
<div class="row">
    <div class="col-md-3">
		{% if user.avatar %}
			<img src="{{ user.avatar.url }}">
		{% else %}
			<img src="/static/img/noavatar.png">
		{% endif %}
	</div>
    <div class="col-md-9">
		{{ profile.username }}
		<ul class="nav nav-tabs" id="myTab" role="tablist">
  <li class="nav-item">
    <a class="nav-link active" id="home-tab" data-toggle="tab" href="#home" role="tab" aria-controls="home" aria-selected="true">Home</a>
  </li>
  <li class="nav-item">
    <a class="nav-link" id="profile-tab" data-toggle="tab" href="#profile" role="tab" aria-controls="profile" aria-selected="false">Закладки {{ user.bookmarkmovie_set.all.count }}</a>
  </li>
  <li class="nav-item">
    <a class="nav-link" id="contact-tab" data-toggle="tab" href="#contact" role="tab" aria-controls="contact" aria-selected="false">Contact</a>
  </li>
</ul>
<div class="tab-content" id="myTabContent">
  <div class="tab-pane fade show active" id="home" role="tabpanel" aria-labelledby="home-tab">...</div>
  <div class="tab-pane fade" id="profile" role="tabpanel" aria-labelledby="profile-tab">...</div>
  <div class="tab-pane fade" id="contact" role="tabpanel" aria-labelledby="contact-tab">...</div>
</div>
		<p></p>
	</div>
</div>
</div>
{% endblock %}

хм.. для вывода тогда должно быть как-то так

{% for bookmark in user.bookmarkmovie_set.all %}
    {{ bookmark .movie.title }}
{% endfor %}

Думаю, что догадаетесь как вывести нужную информацию о фильме

bookmark.movie - означает, что вы берёте поле внешнего ключа на фильм, которое должно быть в закладке. полагаю, что онго должно называться movie

Ага цикл. Спасибо за разъяснение Евгений.

здраствуйте. У меня почему-то выходит так

здраствуйте. У меня почему-то выходит так

url или не подключили, или подключили неправильно

Подскажите пожалуйста, что значит

#app_name = 'ajax'

в urls.py ??
Это надо новое приложение создать?

Да, для такого функционала лучше создать отдельное приложение. Причина предельно простая. Обычно, когда делают сайт на Django, то очень часто делают его мультиязычным. Поэтому всё, что касается ajax взаимодействия, лучше пускать мимо дистпечеризации urls с поддержкой мультиязычности. Такая проблема возникает вследствие того, что производится переадресация на url, который зависит от языка. Из-за переадресации неправильно работает js.

Тут два решения проблемы:

1. Выковыривать язык из страницы, чтобы сформировать url и биться со следующими багами
2. Пустить этот функционал в обход мультиязычности, что гораздо проще и с меньшим количеством последующих багов

ок, понял, спасибо
А в новом приложении строку app_name = 'ajax' указывать в урл?

Да, нужно будет добавить файл urls, поскольку обычно при использование команды startapp это файл не создаётся, и потом в нём обязательно добавить app_name, можете присовить своё имя для него. app_name отвечает за контекс генерирования url

подскажите еще пж, я правильно понимаю, что функцию:

def get_bookmark_count(self):
    return self.bookmarkarticle_set.all().count()

Нужно добавить в модель статьи?

И еще второй вопрос, вроде всё сделал, при клике на звездочку (кстати у меня она не отображется, просто пустое место, но при клике что-то срабатывает) ничего не происходит, в консоле: Not Found: /api/article//bookmark/

1. Да
2. У вас тут должен быть id статьи, то есть эта запись /api/article//bookmark/ должна выглядеть например так /api/article/666/bookmark/
   Так что ищите косяк где-то в js

Ок, а посмотреть добавленные закладки на каком урл можно будет после добавления?

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

Не получилось у меня сделать с ajax, по всей видимости пробелма с получением ID поста. Попробовал без ajax, по сути таже ошбика(какие-то проблемы с id). Создал новый вопрос: https://evileg.com/ru/forum/topic/1329/ , не знаю как пофиксить.

Подскажите, а дейтсвие:

data-action - действие, которое нужно совершить, в данном случае добавление в закладки

data-action="bookmark"

В какой части кода описываем?

ction to_bookmarks()
{
    var current = $(this);
    var type = current.data('type');
    var pk = current.data('id');
    var action = current.data('action'); // ЗДЕСЬ

    $.ajax({
        url : "/api/" + type + "/" + pk + "/" + action + "/",
        type : 'POST',
        data : { 'obj' : pk },

        success : function (json) {
            current.find("[data-count='" + action + "']").text(json.count);
        }
    });

    return false;
}

Вообщем решил сделать не по id, а по slug, что в итоге вышло:

urls.py

app_name = 'ajax'
urlpatterns = [ url('<slug:slug>/', login_required(views.BookmarkView.as_view(model=BookmarkArticle)), name='article_bookmark'),

в .js

function to_bookmarks()
{
    var current = $(this);
    var type = current.data('type');
    var pk = current.data('id');
    var action = current.data('action');

    $.ajax({
        url :   "/" + pk + "/" ,
        type : 'POST',
        data : { 'obj' : pk },

        success : function (json) {
            current.find("[data-count='" + action + "']").text(json.count);
        }
    });

    return false;
}

// Подключение обработчика
$(function() {
    $('[data-action="bookmark"]').click(to_bookmarks);
});

В шаблоне:

<div data-id="{{ post.slug }}" data-type="post" data-action="bookmark" title="Избранное">
    <span class="far fa-heart"></span>
    <span data-count="bookmark">{{ post.get_bookmark_count }}</span>
</div>

В итоге при клике на иконку в консоле получаю:

[30/Apr/2020 16:21:00] "POST /novayastatya/ HTTP/1.1" 200 7405

Счётчик при клике на иконку никак не обновляется 0 всегда, также в БД таблица bookmark_article пустая. Может быть с урлами что-то?

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

[30/Apr/2020 16:21:00] "POST /novayastatya/ HTTP/1.1" 200 7405

То есть @login_required не срабатывает

Евгений, можете подсказать почему может быть так?

А вы js код где подключаете?

файл js с to_bookmarks можно загрузить в head теге, а вот этот код

$(function() {
    $('[data-action="bookmark"]').click(to_bookmarks);
});

должен вызываться в конце страницы, когда html загружен

Загрузил в конце страницы - ситуация аналогичная. Клик нажимается, в консоле

 "POST /novayastatya/ HTTP/1.1" 200 7405

Но счетчик не обновляется и в базе нет записи. У меня подозрения, что не срабатаывает url.

urls.py

app_name = 'ajax'

urlpatterns = [
    url('/api/favourites/', login_required(views.BookmarkView.as_view(model=BookmarkArticle)),
        name='article_bookmark'),
]

Урл нигде не перподключал, создал новое app "ajax" в нём добавил urls.py и в него добавил код выше. Может быть надо где-то include url вставить?

И еще немного симптомов, если я убираю во views.py (ajax app) класс BookmarkView(View) , то ничего не меняется (никаких ошибок нет). Но в коносле POST запрос ("POST /novayastatya/ HTTP/1.1" 200 7405) при клике всё равно пишет. Получается урл не срабатывает или view, а может и с моделями что-то...

А вы эти urls вообще подключили в самом главном файле urls, который обычно находится рядом с файлом settings.py?

нет, а как?)

также, как подключили сами статьи в файле urls.py, который лежит рядом с settings.py. Статьи же у вас работают...

я не понимаю, как нужно подкллючить url из ajax app. Сатьи у меня лежат в главном приложении

Статьи я подключил вот так:

path('<slug:slug>/', views.post_detail, name='post_detail')

А как подключить url'ы приложения ajax app ?

боюсь, что вам это слабо поможет, но обычно url из других app подключают подобным образом

urlpatterns = [
    path('api/', include('ecore.urls')),
]

Постарайтесь разобраться с этим, а то у меня возникает ощущение, что вас за руку водить нужно. Поэтому я склоняюсь к тому, что вы рано начали решать свою задачу. Вам нужно поднабраться опыта с Django. Поэтому переключитесь на какие-нибудь другие задачи, а добавление в избранное реализуете позже.

За неделю я разобрался и нашел где была ошибка, сейчас закладки добавляются успешно.

Сейчас новый вопрос, как можно их вывывести?
Пытаюсь вывести в шаблоне:

{% for bookmark in user.bookmarkarticle_set.all %} 
{{ bookmark.post.title }} #модель статьи Post
{% endfor %}

Ничего не выводится. Вьюху не создавал, не имею понятия как. Подскажите как вывести закладки юзера?

Добрый день.
То, что вы добавили как статью, следовало добавить как вопрос на форуме. ЭТо не статья, вы описали, что сделали и спрашивайте, что делать дальше. Поэтому задайте именно вопрос на форуме, в противном случае я не могу пропустить ту статью, чтобы другие пользователи имели к ней доступ. Она не соответсвует наименованию статьи.

Но суть вашей проблемы в том случае заключается в том, что вы не забираете список постов в соответствии с пользователем.
Поскольку никак не передаёте информацию о пользователе во view

ок, немного промазал при создании)
создал вопрос к статье: https://evileg.com/ru/forum/topic/1345/

> Но суть вашей проблемы в том случае заключается в том, что вы не забираете список постов в соответствии с пользователем.
Поскольку никак не передаёте информацию о пользователе во view

Вы можете показать пример как это сделать?

всё, разобрался

Оставлю для тех, у кого возникнет подобный вопрос. Посты выводим циклом:

{% for bookmark in user.bookmarkarticle_set.all %} 
{{ bookmark.obj.title }} 
{% endfor %}

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

{{ bookmark.obj.title }} 

Как я могу вывести добавленные посты в обратном порядке? сейчас выводятся новодобавленные в конец. Пыатюсь добавить order_by('-created_on') :

views.py

    object_list =  User.bookmarkarticle_set.all().order_by('-created_on')

Ошибка:

Cannot resolve keyword 'created_on' into field.

Может так?

object_list =  User.bookmarkarticle_set.all().order_by('-obj__created_on')

да, спасибо)