From 6c1f62b1688855cc54d905bf4141645f4ab84187 Mon Sep 17 00:00:00 2001
From: Gregory Trullemans <redj@MacBook-Pro-de-Gregory.local>
Date: Thu, 13 Jan 2022 18:43:43 +0100
Subject: [PATCH] Adding Closure Table pour Educative prerequisites

---
 templates/objectives/skills/details.html      |  3 +
 templates/objectives/skills/tree.html         | 78 +++++++------------
 ultron/objective/admin.py                     | 21 ++++-
 .../management/commands/rebuild_tree.py       | 14 +---
 .../migrations/0012_prerequisiteclosure.py    | 27 +++++++
 ultron/objective/views.py                     | 20 ++++-
 6 files changed, 96 insertions(+), 67 deletions(-)
 create mode 100644 ultron/objective/migrations/0012_prerequisiteclosure.py

diff --git a/templates/objectives/skills/details.html b/templates/objectives/skills/details.html
index baf9c4325c..592933ca00 100644
--- a/templates/objectives/skills/details.html
+++ b/templates/objectives/skills/details.html
@@ -87,6 +87,9 @@
                     </div>
                 </div>
                 {% endif %}
+                <br />
+                <a href="{% url 'skill_tree' skill.id %}">Learning Line</a><br />
+                <br />
             </div>
             <div class="col-md-6 alert {% if request.session.template == 0 %}skill-info{% else %}alert-secondary{% endif %} mr-0">
                 <p>From <a href="#">{{ skill.departure }}</a>, {% if skill.rotation %} <a href="#">{{ skill.rotation }}</a> quart of <a href="#">{{ skill.get_rotation_type_display }}</a> rotation {% endif %}{% if skill.twist %} with <a href="#">{{ skill.twist }} half-twist</a> {% endif %} in a <a href="#">{{ skill.get_position_display }}</a> position, landing to <a href="#">{{ skill.landing }}</a></p>
diff --git a/templates/objectives/skills/tree.html b/templates/objectives/skills/tree.html
index 7ff575cce0..78a7d9aff4 100644
--- a/templates/objectives/skills/tree.html
+++ b/templates/objectives/skills/tree.html
@@ -1,70 +1,45 @@
 {% extends "base.html" %}
-
+{% load static %}
 {% block header %}
 <style>
     svg {
-      width: 50%;
-      height: 50%;
+      width: 25%;
+      height: 25%;
     }
   </style>
 {% endblock %}
 
 {% block content %}
 <div class="card mb-0">
-    <svg></svg>
+    <div class="card-header">
+        <h3 class="mb-0">{{ skill.short_label }}</h3>
+        <h4 class="card-title"> {{ skill.notation }}</h4>
+    </div>
+    <div class="card-body pb-0 mb-0">
+        <div class="row mr-1 ml-1 pb-0 mb-0">
+            <svg></svg>
+        </div>
+    </div>
 </div>
 {% endblock %}
 
 {% block footerscript %}
-<script src="{% static "js/plugins/D3-dag/d3.min.js%}"></script>
-<script src="{% static "js/plugins/D3-dag/d3-dag.0.8.2.min.js%}"></script>
+<script src="{% static "js/plugins/D3-dag/d3.min.js" %}"></script>
+<script src="{% static "js/plugins/D3-dag/d3-dag.0.8.2.min.js" %}"></script>
 <script>
     (async () => {
-        // fetch data and render
-        // const resp = await fetch(
-        //     "https://raw.githubusercontent.com/erikbrinkman/d3-dag/main/examples/grafo.json"
-        // );
-        const data = [{
-                // Rudy
-                "id": "43/",
-                "parentIds": ["-2", ".41/"]
-            },
+        const data = [
+            {% for key, value in node_dict.items %}
             {
-                // vrille
-                "id": "-2",
-                "parentIds": ["-1"]
+                "id": "{{ key.short_label }}",
+                "parentIds": [
+                    {% for prerequisite in value %}
+                        "{{ prerequisite.short_label }}",
+                    {% endfor %}
+                ]
             },
-            {
-                // Barani
-                "id": ".41/",
-                "parentIds": ["-1", ".3-/"]
-            },
-            {
-                // 1/2 vrille
-                "id": "-1",
-                "parentIds": ["|"]
-            },
-            {
-                // Chandelle
-                "id": "|",
-                "parentIds": []
-            },
-            {
-                // 3/4 Avant Tendu
-                "id": ".3-/",
-                "parentIds": [".1"]
-            },
-            {
-                // Ventre
-                "id": ".1",
-                "parentIds": ["4p"]
-            },
-            {
-                // 4 pattes
-                "id": "4p",
-                "parentIds": []
-            }];
-        // const data = await resp.json();
+            {% endfor %}
+        ]
 
         const dag = d3.dagStratify()(data);
         const nodeRadius = 20;
@@ -95,6 +70,7 @@
             //     const size = node ? base : 5;
             //     return [1.2 * size, size];
             // });
+            // .attr("r", function(d) {return d.name.length * 2.5;})
         const { width, height } = layout(dag);
 
         // --------------------------------
@@ -169,6 +145,10 @@
         nodes
             .append("text")
             .text((d) => d.data.id)
+            // .attr("font-size", "8")
+            // .attr("font-size", function(d) { return Math.min(2 * d.r, (2 * d.r - 8) / this.getComputedTextLength() * 24) + "px"; })
+            // .attr("font-size", function(d) {return (1 / d.data.id.length) * 60;})
+            .attr("font-size", function(d) {return (1 / d.data.id.replace(/ /g,'').length) * 60;})
             .attr("font-weight", "bold")
             .attr("font-family", "sans-serif")
             .attr("text-anchor", "middle")
diff --git a/ultron/objective/admin.py b/ultron/objective/admin.py
index edc3a37298..fd92b38334 100644
--- a/ultron/objective/admin.py
+++ b/ultron/objective/admin.py
@@ -8,7 +8,13 @@ from django_admin_listfilter_dropdown.filters import (
     RelatedDropdownFilter
 )
 
-from .models import TouchPosition, Skill, Routine, RoutineSkill
+from .models import (
+    TouchPosition,
+    Skill,
+    Routine,
+    RoutineSkill,
+    PrerequisiteClosure,
+)
 
 
 class TouchPositionAdmin(admin.ModelAdmin):
@@ -159,7 +165,20 @@ class RoutineSkillAdmin(admin.ModelAdmin):
     autocomplete_fields = ("routine", "skill")
 
 
+class PrerequisiteClosureAdmin(admin.ModelAdmin):
+    model = PrerequisiteClosure()
+
+    list_display = ("descendant", "ancestor", "level", "path")
+    search_fields = (
+        "descendant__long_label",
+        "descendant__short_label",
+        "ancestor__long_label",
+        "ancestor__short_label",
+    )
+
+
 admin.site.register(TouchPosition, TouchPositionAdmin)
 admin.site.register(Skill, SkillAdmin)
 admin.site.register(Routine, RoutineAdmin)
 admin.site.register(RoutineSkill, RoutineSkillAdmin)
+admin.site.register(PrerequisiteClosure, PrerequisiteClosureAdmin)
diff --git a/ultron/objective/management/commands/rebuild_tree.py b/ultron/objective/management/commands/rebuild_tree.py
index a57f34273a..c10857140d 100644
--- a/ultron/objective/management/commands/rebuild_tree.py
+++ b/ultron/objective/management/commands/rebuild_tree.py
@@ -12,33 +12,21 @@ from ultron.objective.models import Educative, PrerequisiteClosure
 
 class Command(BaseCommand):
     def handle(self, *args, **options):
-        # educative_list = Educative.objects.all()
-        educative_list = [Educative.objects.get(pk=44)]
+        educative_list = Educative.objects.all()
 
         for educative in educative_list:
-            # print('__________________________________________________________________________')
-            # print('Traitement de ' + str(educative))
-            # breadcrumb = [node for node in educative.breadcrumb()]
             breadcrumb = educative.breadcrumb()
 
             for path in range(0, len(breadcrumb)):
-                # print('  ˪ ' + str(path + 1) + 'ème chemin')
                 tree = set(PrerequisiteClosure.objects.filter(descendant=educative, path=path))
-                # print('    ' + str(tree))
     
                 for position, ancestor in enumerate(breadcrumb[path]):
-                    # print('    ˪ Traitement de ' + str(ancestor.long_label) + ' : ' + str(ancestor.long_label) + ' -> ' + str(educative.long_label) + ' | ' + str(position) + ' | ' + str(path))
                     tree_path, _ = PrerequisiteClosure.objects.get_or_create(
                         ancestor=ancestor, descendant=educative, level=position, path=path
                     )
-                    # if _:
-                        # print('      -> CREATION : ' + str(tree_path))
-                    # else:
-                        # print('      -> RECUPERATION : ' + str(tree_path))
     
                     if tree_path in tree:
                         tree.remove(tree_path)
     
                 for tree_path in tree:
-                    # print('      DELETE : ' + str(tree_path))
                     tree_path.delete()
diff --git a/ultron/objective/migrations/0012_prerequisiteclosure.py b/ultron/objective/migrations/0012_prerequisiteclosure.py
new file mode 100644
index 0000000000..331e56226b
--- /dev/null
+++ b/ultron/objective/migrations/0012_prerequisiteclosure.py
@@ -0,0 +1,27 @@
+# Generated by Django 3.2.8 on 2022-01-12 20:30
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('objective', '0011_auto_20220109_1016'),
+    ]
+
+    operations = [
+        migrations.CreateModel(
+            name='PrerequisiteClosure',
+            fields=[
+                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('level', models.PositiveIntegerField()),
+                ('path', models.PositiveIntegerField()),
+                ('ancestor', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='descendants', to='objective.educative')),
+                ('descendant', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='ancestor', to='objective.educative')),
+            ],
+            options={
+                'unique_together': {('descendant', 'ancestor', 'level', 'path')},
+            },
+        ),
+    ]
diff --git a/ultron/objective/views.py b/ultron/objective/views.py
index 8f84674d36..b24183e541 100644
--- a/ultron/objective/views.py
+++ b/ultron/objective/views.py
@@ -75,10 +75,22 @@ def skill_tree(request, skill_id):
     """
     """
     skill = get_object_or_404(Skill, pk=skill_id)
-    print(skill)
-    skill_breadcrumb = PrerequisiteClosure.objects.filter(descendant=skill).order_by("path", "level")
-    print(skill_breadcrumb)
-    context = {"skill": skill}
+    node_dict = {}
+    # node_list[skill] = list(skill.prerequisites.all())
+
+    skill_closure = PrerequisiteClosure.objects.filter(descendant=skill)
+    for closure in skill_closure:
+        node_dict[closure.ancestor] = closure.ancestor.prerequisites.all()
+        # if closure.descendant != closure.ancestor:
+        #     print(closure)
+    # print(node_dict)
+
+    # edge_list = {}
+    # for skill in node_list.values():
+    #     edge_list[skill.long_label] = list(skill.prerequisites.all())
+    # print(edge_list)
+
+    context = {"skill": skill, "node_dict": node_dict}
     return render(request, "objectives/skills/tree.html", context)