Support for Sites contrib about django-sitetree HOT 7 CLOSED

Привет,

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

from django-sitetree.

Разные деревья в любом случае будут использоваться для разных сайтов.

Плюс этого решения в том, что ничего не надо переделывать в django-sitetree
Но минусы достаточно существенные:

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

from django-sitetree.

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

from django-sitetree.

Временно решил проблему

в sitetreeapp.py
def get_sitetree(self, alias):
alias="%s-%s" % (alias, settings.SITE_ID)
...

в models.py (необязательно, на будущее)
class Tree(models.Model):
...
site = models.ForeignKey(Site, blank=True, default=settings.SITE_ID)
objects = models.Manager()
on_site = CurrentSiteManager()

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

Под последними пунктами подразумевал то, что можно будет использовать
вместо например такого кода (в функции resolve_tree_i18_alias):
trees_count = Tree.objects.filter(alias=i18n_tree_alias).count()
такой:
trees_count = Tree.on_site.filter(alias=i18n_tree_alias).count()
Как это описано в джанго-буке.

Админка же в автомате будет показывать алиасы, привязанные только к определенному сайту при использовании общей БД (если убрать из добавленного в код модели objects = models.Manager()).

Все дополнения для мультисайтовости тоже будут работать корректно.

from django-sitetree.

Если вообще не лезть в код, то в шаблоне это выглядит так (довольно извращенно, но работает):

{% load sitetree %}
{% with site.id|slugify as id %}
{% with "mainmenu-"|add:id as menu %}
{% sitetree_menu from menu include "trunk,topmenu" template "sitetree/menu.html" %}
{% endwith %}
{% endwith %}

Будет подгружаться "mainmenu-1" для главного сайта SITE_ID=1.

from django-sitetree.

Нужно ещё подумать на тему актуальности поддержки sites contrib, потому как изменения значительные.

Касательно «если вообще не лезть в код», предпочтительнее (и краше) будет решение с самопальным обработчиком контекста.

from django-sitetree.

Закрываю. Будет решен в рамках #64.

from django-sitetree.