fix some stuff the pycharm code checker found

it also thought the db param of init_db is not needed, but it is, added a comment there

conftest.py

@@ -30,7 +30,7 @@ from django.utils.translation import activate
 
 # Note: fixture must be "function" scope (default), see https://github.com/pelme/pytest_django/issues/33
 @pytest.fixture(autouse=True)
-def db_init(db):
+def db_init(db):  # note: db is a predefined fixture and required here to have the db available
     """
     Init the database contents for testing, so we have a service domain, ...
     """

nsupdate/api/views.py

@@ -1,7 +1,5 @@
 # -*- coding: utf-8 -*-
 
-import os
-
 import logging
 logger = logging.getLogger(__name__)
 

nsupdate/main/dnstools.py

@@ -27,7 +27,6 @@ import dns.update
 import dns.tsig
 import dns.tsigkeyring
 
-from django.conf import settings
 from django.utils.timezone import now
 
 

nsupdate/main/templates/main/host.html

@@ -36,17 +36,17 @@
                We populated the input fields for the new addresses with your currently detected remote addresses.
             </p>
                 <div class="form-group">
-                    <label>Current IPv4 address from master nameserver</label>
+                    <label for="current_ipv4">Current IPv4 address from master nameserver</label>
                     <input class="form-control" type="text" id="current_ipv4" name="current_ipv4" value="{{ host.getIPv4 }}" disabled>
-                    <label>Set new IPv4 address</label>
+                    <label for="myipv4">Set new IPv4 address</label>
                     <input class="form-control" type="text" id="myipv4" name="myipv4" placeholder="give new ipv4 address" value="{{ request.session.ipv4 }}" required>
                 </div>
                 <button type="button" class="btn btn-primary" onclick="update_host_v4('{{ host.get_fqdn }}')">Apply</button> <span id="update_result_v4"></span>
                 <p></p>
                 <div class="form-group">
-                    <label>Current IPv6 address from master nameserver</label>
+                    <label for="current_ipv6">Current IPv6 address from master nameserver</label>
                     <input class="form-control" type="text" id="current_ipv6" name="current_ipv6" value="{{ host.getIPv6 }}" disabled>
-                    <label>Set new IPv6 address</label>
+                    <label for="myipv6">Set new IPv6 address</label>
                     <input class="form-control" type="text" id="myipv6" name="myipv6" placeholder="give new ipv6 address" value="{{ request.session.ipv6 }}" required>
                 </div>
                 <button type="button" class="btn btn-primary" onclick="update_host_v6('{{ host.get_fqdn }}')">Apply</button> <span id="update_result_v6"></span>
@@ -59,7 +59,7 @@
                       }).fail(function( data ) {
                         $('#update_result_v4').text('error');
                       });
-                }
+                };
                 update_host_v6 = function(host) {
                     $('#update_result_v6').text('updating...');
                     $.get( "{% url 'nic_update_authorized' %}", { myip: $('#myipv6').val(), hostname: host } )
@@ -68,7 +68,7 @@
                       }).fail(function( data ) {
                         $('#update_result_v6').text('error');
                       });
-                }
+                };
             </script>
         </div>
     </div>

nsupdate/main/views.py

@@ -1,7 +1,5 @@
 # -*- coding: utf-8 -*-
 
-import dns.inet
-
 from django.db.models import Q
 from django.views.generic import TemplateView, CreateView
 from django.views.generic.edit import UpdateView, DeleteView

nsupdate/settings.py

@@ -3,7 +3,6 @@ Django settings for nsupdate project
 """
 
 import os
-import dns.tsig
 import django.conf.global_settings as DEFAULT_SETTINGS
 
 # Use a unique, long, random, secret string here.