Open
Conversation
Проблема была решена благодаря внедрению глобальной карты (node_map) в методе _build_tree_data. Теперь, при повторном появлении узла, его поддерево извлекается из node_map, что позволяет корректно выводить потомков для всех родителей.
MikasaAl4naya
changed the title
sergeybv26
reviewed
Jun 8, 2025
| self.show_complex = show_complex | ||
| complex_tz_names = ('Таблица', 'Тест') | ||
| self.complex_tz = Tz.objects.filter(name__in=complex_tz_names).values_list('pk', flat=True) | ||
| self.relations_info = {} # {(<parent_id>, <child_id>): {name: <str>, status: <str>, author: <int>}, } |
Collaborator
There was a problem hiding this comment.
Думаю комментарии лучше оставить. Так как на проекте бывают участники с разным уровнем подготовки, то с комментариями им будет легче понять как что работает и что хранится в ловарях
RomanBorovskiy
requested changes
Jun 9, 2025
sergeybv26
approved these changes
Jun 13, 2025
RomanBorovskiy
requested changes
Jun 14, 2025
drevo/relations_tree.py
Outdated
Comment on lines
+149
to
+159
| # Получаем детей по ярлыкам (через KnowledgeLabel) | ||
| qs_label = Znanie.published.filter( | ||
| knowledgelabel__parent_knowledge=knowledge | ||
| ) | ||
|
|
||
| # Объединяем оба QuerySet. При этом применяем distinct(), чтобы избежать дублей. | ||
| combined_qs = (qs_relation | qs_label).distinct() | ||
|
|
||
| # Если нужно задать порядок – можно, например, отсортировать по pk или другому полю. | ||
| # В связи с тем, что ярлыковые связи могут не иметь поля order, часто разумно выбрать универсальное упорядочивание. | ||
| return combined_qs.order_by('pk') |
Collaborator
There was a problem hiding this comment.
Объясни - для чего этот код? Выглядит как генерация нейросети. Вроде поля такого нет в модели. В рамках задачи этот код вообще используется?
Author
There was a problem hiding this comment.
Извиняюсь, случайно в коммит этот код вставил. Пытался сначала через ярлыки сделать
Collaborator
There was a problem hiding this comment.
Тогда drevo/relations_tree.py надо убрать из pr. Может быть проще создать новый pr только с нужными изменениями?
Comment on lines
-25
to
-52
| """ | ||
| Тег для построения дерева знаний \n | ||
| tree_num: номер дерева (на случай если необходимо на одной странице создать несколько деревьев); \n | ||
|
|
||
| show_searchbar: отображать поле поиска по дереву; \n | ||
|
|
||
| empty_tree_message: если дерево по какой либо причине нельзя построить, то будет выводиться сообщение указанное | ||
| в данном параметре; \n | ||
|
|
||
| show_only: принимает объект вида связи, если передан данный параметр, то будут отображаться только связи | ||
| данного вида для переданных знаний (используется если у одного знания из queryset есть несколько связей разных | ||
| видов и необходимо отобразить связи только определённого вида); \n | ||
|
|
||
| hidden_author: принимает объект автора, около знаний данного автора он не указывается; \n | ||
|
|
||
| show_complex: если данный параметр имеет значение True, то на дереве будут отображаться сложные знания. | ||
| В настоящее время для отображения на дереве существует 2 вида сложных знаний: "Таблица", "Тест" | ||
|
|
||
| edit_widgets: список виджетов для редактирования дерева. Допустимые значения: \n | ||
| create - создать новую ветвь (связь) | ||
| delete - удалить ветвь (связь) | ||
| update - удалить ветвь (связь) | ||
|
|
||
| empty_categories: если данный параметр имеет значение True, то на дереве будут отображаться категории, которые | ||
| не имеют знаний. | ||
|
|
||
| is_constructor_type: является ли данное дерево конструктором | ||
| """ |
Collaborator
There was a problem hiding this comment.
Предлагаю вернуть все комментарии и описания
RomanBorovskiy
requested changes
Jun 18, 2025
| return Znanie.published.filter(related__bz=knowledge, | ||
| related__is_published=True | ||
| ).order_by('related__order') | ||
| else: |
Collaborator
There was a problem hiding this comment.
Тогда изменения drevo/relations_tree.py надо убрать из pr. Может быть проще создать новый pr только с нужными изменениями?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Исправлено объединение узлов с несколькими родителями. Проблема была решена благодаря внедрению глобальной карты (node_map) в методе _build_tree_data. Теперь, при повторном появлении узла, его поддерево извлекается из node_map, что позволяет корректно выводить потомков для всех родителей.