From 6240586d25e8f1473c8156642e8cbcf8bce791cb Mon Sep 17 00:00:00 2001
From: koksnuss <koksnuss@arcor.de>
Date: Fri, 1 May 2020 22:57:43 +0200
Subject: [PATCH] get rid of generic views, fix some controlbar bugs, make it
 work in chromium as well

---
 coinmanager/coinc/static/coinc/main.js        |  18 ++++++++++--
 .../coinc/templates/coinc/country.html        |   2 +-
 .../{navigation.html => filter_country.html}  |   0
 coinmanager/coinc/templates/coinc/index.html  |  17 +++--------
 .../coinc/templates/coinc/username.html       |   2 +-
 coinmanager/coinc/urls.py                     |   2 +-
 coinmanager/coinc/views.py                    |  27 +++++++-----------
 coinmanager/db.sqlite3                        | Bin 188416 -> 188416 bytes
 8 files changed, 32 insertions(+), 36 deletions(-)
 rename coinmanager/coinc/templates/coinc/{navigation.html => filter_country.html} (100%)

diff --git a/coinmanager/coinc/static/coinc/main.js b/coinmanager/coinc/static/coinc/main.js
index 594e453..78af51a 100644
--- a/coinmanager/coinc/static/coinc/main.js
+++ b/coinmanager/coinc/static/coinc/main.js
@@ -34,7 +34,7 @@ $(document).ready(function() {
     let option = $('div.username option[value=' + name + ']');
     if (option.length == 0) {
       $('div.username select').append($('<option>', {
-        value: name, label: color, html: name
+        value: name, color: color, html: name
       }));
     }
 
@@ -55,9 +55,10 @@ $(document).ready(function() {
   }
 
 
+  // select username on change
   $('div.username select').on('change', function() {
-
-    color = $('div.username option[value=' + this.value + ']').attr('label');
+    let color = $('div.username option[value=' + this.value + ']').attr('color');
+    $('div.username option[selected=selected]').removeAttr('selected');
     render_login(this.value, color);
   });
 
@@ -117,5 +118,16 @@ $(document).ready(function() {
 
   if (name != '') {
     render_login(name, color);
+  } else {
+  //#TODO log in as default if any
+   // let option = $('div.username option[value=' + name + ']');
+   // if (option.length == 0) {
+   //   $('div.username select').append($('<option>', {
+   //     value: name, label: color, html: name
+   //   }));
+ //   }
+
+ //   $('div.username option[value=' + name + ']').attr('selected', 'selected');
+
   }
 });
diff --git a/coinmanager/coinc/templates/coinc/country.html b/coinmanager/coinc/templates/coinc/country.html
index a50d212..f8bd08d 100644
--- a/coinmanager/coinc/templates/coinc/country.html
+++ b/coinmanager/coinc/templates/coinc/country.html
@@ -8,7 +8,7 @@
           <th colspan="12">
             <a name="{{ country.name_iso }}"></a>
             <img class="country" src="/static/coinc/images/{{ country.name_iso }}.png" />
-            <a href="{% url 'coinc:country' country.name_iso %}">{{ country.name }}</a>{% if country.comment %} - {{ country.comment }}{% endif %}
+            <a href="{% url 'coinc:country' country.name_iso %}">{{ country.name }}</a>{% if country.comment %}<small> - {{ country.comment }}</small>{% endif %}
           </th>
         </tr>
         <tr>
diff --git a/coinmanager/coinc/templates/coinc/navigation.html b/coinmanager/coinc/templates/coinc/filter_country.html
similarity index 100%
rename from coinmanager/coinc/templates/coinc/navigation.html
rename to coinmanager/coinc/templates/coinc/filter_country.html
diff --git a/coinmanager/coinc/templates/coinc/index.html b/coinmanager/coinc/templates/coinc/index.html
index 7f23ffd..062d27f 100644
--- a/coinmanager/coinc/templates/coinc/index.html
+++ b/coinmanager/coinc/templates/coinc/index.html
@@ -1,19 +1,10 @@
-{% if not showed_header %}
-	{% include "header.html" with title="Münzsammlung" %}
-	{% url 'bar.html' as showed_header %}
-{% endif %}
-{% if not showed_usernames %}
-	{% include 'coinc/username.html' with users=users %}
-	{% url 'foo.html' as showed_usernames %}
-{% endif %}
+{% include 'header.html' with title='Münzsammlung' %}
+{% include 'coinc/username.html' with users=users %}
     {% if countrys %}
-      {% show_navigation countrys %}
+      {% include 'coinc/filter_country.html' with countrys=countrys %}
       {% for country in countrys %}
         {% show_country country %}
       {% endfor %}
     {% else %}
     <p>Trage zuerst Länder im Adminbereich ein.</p>{% endif %}
-{% if not showed_footer %}
-	{% include "footer.html" with title="Münzsammlung" %}
-	{% url 'baz.html' as showed_footer %}
-{% endif %}
+{% include 'footer.html' with title='Münzsammlung' %}
diff --git a/coinmanager/coinc/templates/coinc/username.html b/coinmanager/coinc/templates/coinc/username.html
index 9493a24..1b2d48d 100644
--- a/coinmanager/coinc/templates/coinc/username.html
+++ b/coinmanager/coinc/templates/coinc/username.html
@@ -2,7 +2,7 @@
   <label for="select_usernames" id="for_select_usernames">Name</label>
   <select id="select_usernames" name="username" form="username_form" class="form-control">
   {% for user in users %}
-    <option value="{{ user.name }}" label="{{ user.color }}">{{ user.name }}</option>
+    <option value="{{ user.name }}" color="{{ user.color }}">{{ user.name }}</option>
   {% endfor %}
   </select>
   <span id="user_message"><button id="show_add_user" class="btn btn-info">+ Namen hinzufügen</button></span>
diff --git a/coinmanager/coinc/urls.py b/coinmanager/coinc/urls.py
index 7b72f86..63c7c22 100644
--- a/coinmanager/coinc/urls.py
+++ b/coinmanager/coinc/urls.py
@@ -25,7 +25,7 @@ from . import views
 
 app_name = 'coinc'
 urlpatterns = [
-    path('', views.IndexView.as_view(), name='index'),
+    path('', views.index, name='index'),
     path('statistik', views.statistic, name='statistic'),
     path('<str:country>', views.detail_country, name='country'),
     path('add/user/<str:username>/color/<str:color>', views.add_user, name='add_user')
diff --git a/coinmanager/coinc/views.py b/coinmanager/coinc/views.py
index 5272f2b..e481b6c 100644
--- a/coinmanager/coinc/views.py
+++ b/coinmanager/coinc/views.py
@@ -17,28 +17,28 @@
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 
 
-
 from json import dumps
 from datetime import datetime
 
 from django.http import HttpResponse
 from django.template import loader
 from django.template.defaultfilters import register
-from django.views import generic
-from django.shortcuts import redirect
 
 from .models import Country, Stamp, Coin, User
 
 
+def index(request):
+    ''' index view '''
 
-class IndexView(generic.ListView):
-    template_name = 'coinc/index.html'
-    context_object_name = 'countrys'
-
-    def get_queryset(self):
-        ''' return all countries ordered by name '''
-        return Country.objects.order_by('name_iso')
+    template = loader.get_template('coinc/index.html')
+    countrys = Country.objects.order_by('name_iso')
+    users = User.objects.order_by('id')
+    context = {
+        'countrys': countrys,
+        'users': users
+    }
 
+    return HttpResponse(template.render(context, request))
 
 
 def detail_country(request, country):
@@ -47,13 +47,6 @@ def detail_country(request, country):
     return HttpResponse(template.render(show_country(c, single_country=True), request))
 
 
-
-@register.inclusion_tag('coinc/navigation.html')
-def show_navigation(countrys):
-    return {'countrys': countrys}
-
-
-
 @register.inclusion_tag('coinc/country.html')
 def show_country(country, single_country=False):
     c = {
diff --git a/coinmanager/db.sqlite3 b/coinmanager/db.sqlite3
index d6dbabc7066c80b7de41ba41c5041662b5e8315a..56c6dc66337f3b96ad2eb67e1bc71c390f7febd2 100644
GIT binary patch
delta 343
zcmZoTz};|wdxA9Mx`{H*jO#WgFzK_!i?cEtGfw~L#3;PETEERemOq+-li!Vj|0DlB
z{&W2M_}B2y<Zt6I;CJIU<Tu>dD9mpUR3XZM4uqvy7-XfrQWNu1OG;9eO%g54QZ0ar
zFjY1e<Zmy?XPjcd9M5}qI{N}fO%~oqymz-7EMQ#8JKbO(BMVzR3o|EU9EcsY-Eke`
zVMgY7KDFuW8yI)9@M-g@ZEQ?HQMetb@XAGI8TO3n>=T$wS=eW=XKZZLU{BDC=VWjc
zl;!2*WHd4`GSD?J)ip3wFf_9=vam8W)HAX$H!(LbvVggoSuYN^dJ{uELo*Xg10y3d
NxccoUrZZW10075*QC|Q6

delta 210
zcmZoTz};|wdxA9Mnu#*bjB7R~FzK_!inB5sGfw~L#3;PETEERemVtqRo8ORu|0DlB
z{&W2M_}B2y<Zt6I;E(2a<2T&cD9mp!EW^SeEA5q<n3q~olB#TyXknIWAuNqePzsA+
zb3y+0f_%m)2F$U%&!@95VAR}hvVd_V@5BJU=?424qqaM)V?4~r9Lr}goqYr2?v0Ik
eyxW1|S1vM3v7eaEK7q+}V`C5d_7l^YEIa@Vy+GFh