From b43132e292e6efac11fa3cb6232e4ccee85583c0 Mon Sep 17 00:00:00 2001
From: Étienne Loks <etienne.loks@iggdrasil.net>
Date: Tue, 9 Jul 2019 19:02:09 +0200
Subject: Graph generation: manage link - display raw SVG - scale SVG - manage
 reopening in a specific tab

---
 .../ishtar/blocks/sheet_relation_image.html        | 10 ++++-
 ishtar_common/templates/ishtar/sheet.html          |  8 +++-
 ishtar_common/templatetags/ishtar_helpers.py       | 11 ++++--
 ishtar_common/utils.py                             | 46 ++++++++++++++++++----
 4 files changed, 62 insertions(+), 13 deletions(-)

(limited to 'ishtar_common')

diff --git a/ishtar_common/templates/ishtar/blocks/sheet_relation_image.html b/ishtar_common/templates/ishtar/blocks/sheet_relation_image.html
index 552dc5adc..f636f0b50 100644
--- a/ishtar_common/templates/ishtar/blocks/sheet_relation_image.html
+++ b/ishtar_common/templates/ishtar/blocks/sheet_relation_image.html
@@ -1,9 +1,17 @@
-{% load i18n %}{% if relation_image %}
+{% load i18n ishtar_helpers %}{% if relation_image %}
+<div class="svg-image">
+{{relation_image|file_content|safe}}
+</div>
+{% comment %}
+<object class="svg-image" data="{{relation_image.url}}"  class="img-fluid img-thumbnail" type="image/svg+xml"> 
+    <img src="{{relation_image.url}}" class="img-fluid img-thumbnail">
+</object>
 <div id="lightgallery-{{window_id}}-relation-image{{type}}" class="text-center">
     {% if output != "ODT" %}<a href="{{relation_image.url}}">{% endif %}
     <img src="{{relation_image.url}}" class="img-fluid img-thumbnail">
     {% if output != "ODT" %}</a>{% endif %}
 </div>
+{% endcomment %}
 {% endif %}
 
 {% if DOT_GENERATION and output != "ODT" and output != "PDF" %}
diff --git a/ishtar_common/templates/ishtar/sheet.html b/ishtar_common/templates/ishtar/sheet.html
index 6b9c97948..0dda760b8 100644
--- a/ishtar_common/templates/ishtar/sheet.html
+++ b/ishtar_common/templates/ishtar/sheet.html
@@ -73,7 +73,7 @@
       {% block toolbar %}{% endblock %}
       {% endif %}
       <div class="card-body">
-        {% if output != "ODT" and output != "PDF"%}
+        {% if output != "ODT" and output != "PDF" %}
         {% block head_sheet %}
         <script type="text/javascript">
         var last_window='{{window_id}}';
@@ -108,6 +108,12 @@
             {% if item.relation_image %}
             lightGallery(document.getElementById('lightgallery-{{window_id}}-relation-image'));
             {% endif%}
+
+            var hash = window.location.hash.substr(1);
+            if (hash){
+              $('#{{window_id}}-tabs a[href="#{{window_id}}-' + hash + '"]').tab('show');
+            }
+
         });
         </script>
         {% endblock %}
diff --git a/ishtar_common/templatetags/ishtar_helpers.py b/ishtar_common/templatetags/ishtar_helpers.py
index 88dd68b57..b6040487a 100644
--- a/ishtar_common/templatetags/ishtar_helpers.py
+++ b/ishtar_common/templatetags/ishtar_helpers.py
@@ -1,10 +1,8 @@
 #!/usr/bin/env python
 # -*- coding: utf-8 -*-
 
-from datetime import datetime
-
 from django.template import Library
-from django.utils.translation import ugettext as _
+from django.utils.text import mark_safe
 
 register = Library()
 
@@ -17,3 +15,10 @@ def or_(value1, value2):
 @register.filter
 def and_(value1, value2):
     return value1 and value2
+
+
+@register.filter
+def file_content(value):
+    if value:
+        return mark_safe(value.read())
+    return ""
diff --git a/ishtar_common/utils.py b/ishtar_common/utils.py
index eb8414ced..43603b2e1 100644
--- a/ishtar_common/utils.py
+++ b/ishtar_common/utils.py
@@ -36,6 +36,7 @@ import six
 import subprocess
 import sys
 import tempfile
+import xmltodict
 import zipfile
 
 from django import forms
@@ -1224,15 +1225,22 @@ def generate_relation_graph(obj, highlight_current=True,
     above_relations, equal_relations, styles = get_relations_for_graph(
         rel_model, obj.pk, render_above=render_above,
         render_bellow=render_bellow, full=full)
-    if not above_relations and not equal_relations:
-        obj.relation_image = None
-        obj.save()
-        return
 
     # generate dotfile
     dot_str = "digraph relations {\nnode [shape=box];\n"
     rel_str = ""
     described = []
+    if not above_relations and not equal_relations:
+        rel_str += "subgraph NoDir {\nedge [dir=none,style=dashed];\n"
+        style = 'label="{}"'.format(obj.relation_label)
+        if highlight_current:
+            style += ',style=filled,fillcolor="#C6C0C0"'
+        dot_str += u'item{}[{},href="{}"];\n'.format(
+            obj.pk, style,
+            reverse('display-item',
+                    args=[model.SLUG, obj.pk])
+        )
+        rel_str += "}\n"
     for list, directed in ((above_relations, True),
                            (equal_relations, False)):
         if directed:
@@ -1246,14 +1254,20 @@ def generate_relation_graph(obj, highlight_current=True,
                 style = 'label="{}"'.format(left.relation_label)
                 if left.pk == obj.pk and highlight_current:
                     style += ',style=filled,fillcolor="#C6C0C0"'
-                dot_str += u'item{}[{}];\n'.format(left.pk, style)
+                dot_str += u'item{}[{},href="{}"];\n'.format(
+                    left.pk, style,
+                    reverse('display-item', args=[model.SLUG, left.pk])
+                )
             if right_pk not in described:
                 described.append(right_pk)
                 right = model.objects.get(pk=right_pk)
                 style = 'label="{}"'.format(right.relation_label)
                 if right.pk == obj.pk and highlight_current:
                     style += ',style=filled,fillcolor="#C6C0C0"'
-                dot_str += u'item{}[{}];\n'.format(right.pk, style)
+                dot_str += u'item{}[{},href="{}"];\n'.format(
+                    right.pk, style,
+                    reverse('display-item', args=[model.SLUG, right.pk])
+                )
             if not directed:  # on the same level
                 rel_str += u"{{rank = same; item{}; item{};}}\n".format(
                     left_pk, right_pk)
@@ -1270,9 +1284,9 @@ def generate_relation_graph(obj, highlight_current=True,
         dot_file.write(dot_str)
 
     if not render_above:
-        suffix = "_above"
-    elif not render_bellow:
         suffix = "_bellow"
+    elif not render_bellow:
+        suffix = "_above"
     else:
         suffix = ""
 
@@ -1293,6 +1307,22 @@ def generate_relation_graph(obj, highlight_current=True,
     with open(svg_tmp_name, "w") as svg_file:
         popen = subprocess.Popen(args, stdout=svg_file)
         popen.wait()
+
+    # scale image if necessary
+    with open(svg_tmp_name, "r") as svg_file:
+        doc = xmltodict.parse(svg_file.read())
+        width = doc["svg"]["@width"]
+        if width.endswith("pt"):
+            width = float(width[:-2])
+            if width > 600:
+                doc["svg"].pop("@height")
+                doc["svg"].pop("@width")
+
+        doc["svg"]["@preserveAspectRatio"] = "xMinYMin meet"
+
+    with open(svg_tmp_name, "w") as svg_file:
+        svg_file.write(xmltodict.unparse(doc))
+
     with open(svg_tmp_name, "r") as svg_file:
         django_file = File(svg_file)
         attr = "relation_image" + suffix
-- 
cgit v1.2.3