From 4ebb312e59833721fda647e5c4cb11eee78d7e6b Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Thu, 3 Feb 2022 11:27:16 +0100
Subject: [PATCH 01/10] first step for datawipe form

---
 ereuse_devicehub/inventory/forms.py | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/ereuse_devicehub/inventory/forms.py b/ereuse_devicehub/inventory/forms.py
index 83efbd89..198d762c 100644
--- a/ereuse_devicehub/inventory/forms.py
+++ b/ereuse_devicehub/inventory/forms.py
@@ -529,3 +529,22 @@ class AllocateForm(NewActionForm):
             is_valid = False
 
         return is_valid
+
+
+class DataWipeForm(NewActionForm):
+    url = URL(required= False, )
+    success = Boolean(required=False, default=False)
+    software = StringField()
+    date = DateField(u'endTime', 
+                    required=False, 
+    id_document = StringField(u'documentId', 
+    file_name = StringField(u'filename',
+                             validate=validate.Length(max=100))
+    file_hash = StringField(u'hash',
+                             default='',
+                                                          validate=validate.Length(max=64))
+
+    def validate(self, extra_validators=None):
+        is_valid = super().validate(extra_validators)
+
+        return is_valid

From 8b467b95c3571691841ac55d684a17112c642254 Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Mon, 7 Feb 2022 14:01:38 +0100
Subject: [PATCH 02/10] template for data wipe

---
 ereuse_devicehub/static/js/main_inventory.js  | 25 +++++++-
 .../templates/inventory/data_wipe.html        | 63 +++++++++++++++++++
 .../templates/inventory/device_list.html      |  4 +-
 3 files changed, 88 insertions(+), 4 deletions(-)
 create mode 100644 ereuse_devicehub/templates/inventory/data_wipe.html

diff --git a/ereuse_devicehub/static/js/main_inventory.js b/ereuse_devicehub/static/js/main_inventory.js
index 05833fe5..1eb680fd 100644
--- a/ereuse_devicehub/static/js/main_inventory.js
+++ b/ereuse_devicehub/static/js/main_inventory.js
@@ -1,8 +1,12 @@
 $(document).ready(function() {
-    var show_action_form = $("#allocateModal").data('show-action-form');
-    if (show_action_form != "None") {
+    var show_allocate_form = $("#allocateModal").data('show-action-form');
+    var show_datawipe_form = $("#datawipeModal").data('show-action-form');
+    if (show_allocate_form != "None") {
         $("#allocateModal .btn-primary").show();
-        newAllocate(show_action_form);
+        newAllocate(show_allocate_form);
+    } else if (show_datawipe_form != "None") {
+        $("#datawipeModal .btn-primary").show();
+        newDataWipe(show_datawipe_form);
     } else {
         $(".deviceSelect").on("change", deviceSelect);
     }
@@ -26,6 +30,9 @@ function deviceSelect() {
 
         $("#allocateModal .pol").show();
         $("#allocateModal .btn-primary").hide();
+
+        $("#datawipeModal .pol").show();
+        $("#datawipeModal .btn-primary").hide();
     } else {
         $("#addingLotModal .pol").hide();
         $("#addingLotModal .btn-primary").show();
@@ -39,6 +46,9 @@ function deviceSelect() {
         $("#allocateModal .pol").hide();
         $("#allocateModal .btn-primary").show();
 
+        $("#datawipeModal .pol").hide();
+        $("#datawipeModal .btn-primary").show();
+
         $("#addingTagModal .pol").hide();
     }
 }
@@ -69,11 +79,20 @@ function newAllocate(action) {
     $("#activeAllocateModal").click();
 }
 
+function newDataWipe(action) {
+    $("#datawipeModal #type").val(action);
+    $("#datawipeModal #title-action").html(action);
+    get_device_list();
+    deviceSelect();
+    $("#activeDatawipeModal").click();
+}
+
 function get_device_list() {
     var devices = $(".deviceSelect").filter(':checked');
 
     /* Insert the correct count of devices in actions form */
     var devices_count = devices.length;
+    $("#datawipeModal .devices-count").html(devices_count);
     $("#allocateModal .devices-count").html(devices_count);
     $("#actionModal .devices-count").html(devices_count);
 
diff --git a/ereuse_devicehub/templates/inventory/data_wipe.html b/ereuse_devicehub/templates/inventory/data_wipe.html
new file mode 100644
index 00000000..8ce124ad
--- /dev/null
+++ b/ereuse_devicehub/templates/inventory/data_wipe.html
@@ -0,0 +1,63 @@
+<div class="modal fade" id="datawipeModal" tabindex="-1" style="display: none;" aria-hidden="true"
+    data-show-action-form="{{ form_new_datawipe.type.data }}">
+  <div class="modal-dialog modal-lg">
+    <div class="modal-content">
+
+      <div class="modal-header">
+        <h5 class="modal-title">New Action <span id="title-action"></span></h5>
+        <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button>
+      </div>
+
+      <form action="{{ url_for('inventory.devices.datawipe_add') }}" method="post" enctype="multipart/form-data">
+      {{ form_new_datawipe.csrf_token }}
+        <div class="modal-body">
+          {% for field in form_new_datawipe %}
+          {% if field != form_new_datawipe.csrf_token %}
+          {% if field == form_new_datawipe.devices  %}
+          <div class="col-12">
+              {{ field.label(class_="form-label") }}: <span class="devices-count"></span>
+              {{ field(class_="devicesList") }}
+              <p class="text-danger pol" style="display: none;">
+                  You need select first some device before to do one action
+              </p>
+              <p class="enumeration-devices"></p>
+          </div>
+          {% elif field == form_new_datawipe.lot %}
+            {{ field }}
+          {% elif field == form_new_datawipe.type %}
+            {{ field }}
+          {% else %}
+          <div class="col-12">
+              {{ field.label(class_="form-label") }}
+              {% if field == form_new_datawipe.success %}
+                <div class="form-check form-switch">
+                  {{ field(class_="form-check-input") }}
+                  <small class="text-muted">{{ field.description }}</small>
+                </div>
+              {% else %}
+                {{ field(class_="form-control") }}
+                <small class="text-muted">{{ field.description }}</small>
+              {% endif %}
+              {% if field.errors %}
+              <p class="text-danger">
+                {% for error in field.errors %}
+                  {{ error }}<br/>
+                {% endfor %}
+              </p>
+              {% endif %}
+          </div>
+          {% endif %}
+          {% endif %}
+          {% endfor %}
+        </div>
+
+
+        <div class="modal-footer">
+          <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">Close</button>
+          <input type="submit" class="btn btn-primary" style="display: none;" value="Create" />
+        </div>
+      </form>
+
+    </div>
+  </div>
+</div>
diff --git a/ereuse_devicehub/templates/inventory/device_list.html b/ereuse_devicehub/templates/inventory/device_list.html
index 97291791..a165e387 100644
--- a/ereuse_devicehub/templates/inventory/device_list.html
+++ b/ereuse_devicehub/templates/inventory/device_list.html
@@ -80,6 +80,7 @@
               </button>
               <span class="d-none" id="activeActionModal" data-bs-toggle="modal" data-bs-target="#actionModal"></span>
               <span class="d-none" id="activeAllocateModal" data-bs-toggle="modal" data-bs-target="#allocateModal"></span>
+              <span class="d-none" id="activeDatawipeModal" data-bs-toggle="modal" data-bs-target="#datawipeModal"></span>
               <ul class="dropdown-menu" aria-labelledby="btnActions">
                 <li>
                     Status actions
@@ -139,7 +140,7 @@
                   </a>
                 </li>
                 <li>
-                  <a href="javascript:void()" class="dropdown-item">
+                  <a href="javascript:newDataWipe('DataWipe')" class="dropdown-item">
                     <i class="bi bi-eraser-fill"></i>
                     DataWipe
                   </a>
@@ -280,6 +281,7 @@
 {% include "inventory/removelot.html" %}
 {% include "inventory/actions.html" %}
 {% include "inventory/allocate.html" %}
+{% include "inventory/data_wipe.html" %}
 
 <!-- CDN -->
 <script src="https://cdn.jsdelivr.net/npm/simple-datatables@latest"></script>

From 94604dd029a047a6838566569d1e4dd8765cee14 Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Mon, 7 Feb 2022 14:01:56 +0100
Subject: [PATCH 03/10] view for data wipe

---
 ereuse_devicehub/inventory/views.py | 35 ++++++++++++++++++++---------
 1 file changed, 24 insertions(+), 11 deletions(-)

diff --git a/ereuse_devicehub/inventory/views.py b/ereuse_devicehub/inventory/views.py
index bd52eb77..c9b69091 100644
--- a/ereuse_devicehub/inventory/views.py
+++ b/ereuse_devicehub/inventory/views.py
@@ -9,7 +9,7 @@ from ereuse_devicehub.inventory.forms import (AllocateForm, LotDeviceForm,
                                               LotForm, NewActionForm,
                                               NewDeviceForm, TagDeviceForm,
                                               TagForm, TagUnnamedForm,
-                                              UploadSnapshotForm)
+                                              UploadSnapshotForm, DataWipeForm)
 from ereuse_devicehub.resources.device.models import Device
 from ereuse_devicehub.resources.lot.models import Lot
 from ereuse_devicehub.resources.tag.model import Tag
@@ -37,6 +37,7 @@ class DeviceListMix(View):
             devices = sorted(devices, key=lambda x: x.updated, reverse=True)
             form_new_action = NewActionForm(lot=lot.id)
             form_new_allocate = AllocateForm(lot=lot.id)
+            form_new_datawipe = DataWipeForm(lot=lot.id)
         else:
             devices = Device.query.filter(
                 Device.owner_id == current_user.id).filter(
@@ -44,6 +45,7 @@ class DeviceListMix(View):
                     Device.updated.desc())
             form_new_action = NewActionForm()
             form_new_allocate = AllocateForm()
+            form_new_datawipe = DataWipeForm()
 
         action_devices = form_new_action.devices.data
         list_devices = []
@@ -57,6 +59,7 @@ class DeviceListMix(View):
             'form_tag_device': TagDeviceForm(),
             'form_new_action': form_new_action,
             'form_new_allocate': form_new_allocate,
+            'form_new_datawipe': form_new_datawipe,
             'lot': lot,
             'tags': tags,
             'list_devices': list_devices
@@ -310,25 +313,35 @@ class NewAllocateView(NewActionView, DeviceListMix):
     methods = ['POST']
     _form = AllocateForm
 
-    def dispatch_request(self, lot_id=None):
-        self.form = self._form()
-
-        next_url = url_for('inventory.devices.devicelist')
+    def dispatch_request(self):
         lot_id = self.form.lot.data
-        if lot_id:
-            next_url = url_for('inventory.devices.lotdevicelist', lot_id=lot_id)
-
-        if self.form.validate_on_submit():
-            self.form.save()
-            return flask.redirect(next_url)
+        dispatch = super().dispatch_request()
+        if dispatch:
+            return dispatch
 
         self.get_context(lot_id)
         self.context['form_new_allocate'] = self.form
         return flask.render_template(self.template_name, **self.context)
 
 
+class NewDataWipeView(NewActionView, DeviceListMix):
+    methods = ['POST']
+    _form = DataWipeForm
+
+    def dispatch_request(self):
+        dispatch = super().dispatch_request()
+        if dispatch:
+            return dispatch
+
+        lot_id = self.form.lot.data
+        self.get_context(lot_id)
+        self.context['form_new_datawipe'] = self.form
+        return flask.render_template(self.template_name, **self.context)
+
+
 devices.add_url_rule('/action/add/', view_func=NewActionView.as_view('action_add'))
 devices.add_url_rule('/action/allocate/add/', view_func=NewAllocateView.as_view('allocate_add'))
+devices.add_url_rule('/action/datawipe/add/', view_func=NewDataWipeView.as_view('datawipe_add'))
 devices.add_url_rule('/device/', view_func=DeviceListView.as_view('devicelist'))
 devices.add_url_rule('/device/<string:id>/', view_func=DeviceDetailView.as_view('device_details'))
 devices.add_url_rule('/lot/<string:lot_id>/device/', view_func=DeviceListView.as_view('lotdevicelist'))

From 2a92b1f7477f29d832ce798ee4f6c8517a466270 Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Mon, 7 Feb 2022 14:02:28 +0100
Subject: [PATCH 04/10] add form for data wipe

---
 ereuse_devicehub/inventory/forms.py | 84 +++++++++++++++++++++++------
 1 file changed, 67 insertions(+), 17 deletions(-)

diff --git a/ereuse_devicehub/inventory/forms.py b/ereuse_devicehub/inventory/forms.py
index 9716ebaf..63aa0983 100644
--- a/ereuse_devicehub/inventory/forms.py
+++ b/ereuse_devicehub/inventory/forms.py
@@ -5,12 +5,15 @@ from flask import g, request
 from flask_wtf import FlaskForm
 from sqlalchemy.util import OrderedSet
 from wtforms import (DateField, FloatField, HiddenField, IntegerField,
-                     MultipleFileField, SelectField, StringField,
-                     TextAreaField, validators)
+                     MultipleFileField, FileField, SelectField, StringField,
+                     TextAreaField, BooleanField, URLField, validators)
 
 from ereuse_devicehub.db import db
+from ereuse_devicehub.resources.hash_reports import insert_hash
+from ereuse_devicehub.resources.documents.models import DataWipeDocument
 from ereuse_devicehub.resources.action.models import (Action, RateComputer,
-                                                      Snapshot, VisualTest)
+                                                      Snapshot, VisualTest,
+                                                      DataWipe)
 from ereuse_devicehub.resources.action.rate.v1_0 import CannotRate
 from ereuse_devicehub.resources.action.schemas import \
     Snapshot as SnapshotSchema
@@ -472,10 +475,18 @@ class TagDeviceForm(FlaskForm):
 
 
 class NewActionForm(FlaskForm):
-    name = StringField(u'Name', [validators.length(max=50)])
+    name = StringField(u'Name', [validators.length(max=50)],
+                       description="A name or title of the event. Something to look for.")
     devices = HiddenField()
-    date = DateField(u'Date', validators=(validators.Optional(),))
-    severity = SelectField(u'Severity', choices=[(v.name, v.name) for v in Severity])
+    date = DateField(u'Date', [validators.Optional()],
+                     description="""When the action ends. For some actions like booking
+                                    the time when it expires, for others like renting the
+                                    time that the end rents. For specific actions, it is the
+                                    time in which they are carried out; differs from created
+                                    in that created is where the system receives the action.""")
+    severity = SelectField(u'Severity', choices=[(v.name, v.name) for v in Severity],
+                           description="""An indicator that evaluates the execution of the event.
+                                          For example, failed events are set to Error""")
     description = TextAreaField(u'Description')
     lot = HiddenField()
     type = HiddenField()
@@ -540,19 +551,58 @@ class AllocateForm(NewActionForm):
 
 
 class DataWipeForm(NewActionForm):
-    url = URL(required= False, )
-    success = Boolean(required=False, default=False)
-    software = StringField()
-    date = DateField(u'endTime', 
-                    required=False, 
-    id_document = StringField(u'documentId', 
-    file_name = StringField(u'filename',
-                             validate=validate.Length(max=100))
-    file_hash = StringField(u'hash',
-                             default='',
-                                                          validate=validate.Length(max=64))
+    url = URLField(u'Url', [validators.Optional()],
+                   description="Url where the document resides")
+    success = BooleanField(u'Success', [validators.Optional()],
+                   description="The erase was success or not?")
+    software = StringField(u'Software', [validators.Optional()],
+                   description="Which software has you use for erase the disks")
+    id_document = StringField(u'Document Id', [validators.Optional()],
+                   description="Identification number of document")
+    file_name = FileField(u'File', [validators.DataRequired()],
+                   description="""This file is not stored on our servers, it is only used to
+                                  generate a digital signature and obtain the name of the file.""")
 
     def validate(self, extra_validators=None):
         is_valid = super().validate(extra_validators)
 
         return is_valid
+
+    def save(self):
+        file_name = ''
+        file_hash = ''
+        if self.file_name.data:
+            file_name = self.file_name.data.filename
+            file_hash = insert_hash(self.file_name.data.read(), commit=False)
+
+        document = DataWipeDocument(
+            date=self.date.data,
+            id_document=self.id_document.data,
+            file_name=file_name,
+            file_hash=file_hash,
+            url=self.url.data,
+            software=self.software.data,
+            success=self.success.data
+        )
+
+        db.session.add(document)
+
+        Model = db.Model._decl_class_registry.data[self.type.data]()
+        self.instance = Model()
+        devices = self.devices.data
+        severity = self.severity.data
+        self.devices.data = self._devices
+        self.severity.data = Severity[self.severity.data]
+        self.instance.document = document
+        self.instance.file_name = file_name
+
+        # import pdb; pdb.set_trace()
+        #TODO AttributeError: can't set attribute
+        self.populate_obj(self.instance)
+        db.session.add(self.instance)
+        db.session.commit()
+
+        self.devices.data = devices
+        self.severity.data = severity
+
+        return self.instance

From 2542719378066cb64631781c68e8d21b7a97dadf Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Mon, 7 Feb 2022 14:03:01 +0100
Subject: [PATCH 05/10] fix insert hash for force not to do a commit

---
 ereuse_devicehub/resources/hash_reports.py | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/ereuse_devicehub/resources/hash_reports.py b/ereuse_devicehub/resources/hash_reports.py
index f44a79be..1cb7f491 100644
--- a/ereuse_devicehub/resources/hash_reports.py
+++ b/ereuse_devicehub/resources/hash_reports.py
@@ -26,12 +26,15 @@ class ReportHash(db.Model):
     hash3.comment = """The normalized name of the hash."""
 
 
-def insert_hash(bfile):
+def insert_hash(bfile, commit=True):
     hash3 = hashlib.sha3_256(bfile).hexdigest()
     db_hash = ReportHash(hash3=hash3)
     db.session.add(db_hash)
-    db.session.commit()
-    db.session.flush()
+    if commit:
+        db.session.commit()
+        db.session.flush()
+
+    return hash3
 
 
 def verify_hash(bfile):

From 764d749db0b70d75a7c00e6732a11b7c7c9e298c Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Tue, 8 Feb 2022 13:59:50 +0100
Subject: [PATCH 06/10] datawipe form example as 2 forms in one

---
 ereuse_devicehub/inventory/forms.py | 45 +++++++++++++++++++----------
 1 file changed, 29 insertions(+), 16 deletions(-)

diff --git a/ereuse_devicehub/inventory/forms.py b/ereuse_devicehub/inventory/forms.py
index 63aa0983..a90ecf1b 100644
--- a/ereuse_devicehub/inventory/forms.py
+++ b/ereuse_devicehub/inventory/forms.py
@@ -1,13 +1,16 @@
 import json
+import copy
 from json.decoder import JSONDecodeError
 
 from flask import g, request
 from flask_wtf import FlaskForm
 from sqlalchemy.util import OrderedSet
+from wtforms.fields import FormField
 from wtforms import (DateField, FloatField, HiddenField, IntegerField,
                      MultipleFileField, FileField, SelectField, StringField,
-                     TextAreaField, BooleanField, URLField, validators)
+                     TextAreaField, BooleanField, URLField, validators, Form)
 
+from boltons.urlutils import URL
 from ereuse_devicehub.db import db
 from ereuse_devicehub.resources.hash_reports import insert_hash
 from ereuse_devicehub.resources.documents.models import DataWipeDocument
@@ -550,7 +553,9 @@ class AllocateForm(NewActionForm):
         return is_valid
 
 
-class DataWipeForm(NewActionForm):
+class DataWipeDocumentForm(Form):
+    date = DateField(u'Date', [validators.Optional()],
+                   description="Date when was data wipe")
     url = URLField(u'Url', [validators.Optional()],
                    description="Url where the document resides")
     success = BooleanField(u'Success', [validators.Optional()],
@@ -568,24 +573,32 @@ class DataWipeForm(NewActionForm):
 
         return is_valid
 
-    def save(self):
+    def save(self, commit=True):
         file_name = ''
         file_hash = ''
         if self.file_name.data:
             file_name = self.file_name.data.filename
             file_hash = insert_hash(self.file_name.data.read(), commit=False)
 
-        document = DataWipeDocument(
-            date=self.date.data,
-            id_document=self.id_document.data,
-            file_name=file_name,
-            file_hash=file_hash,
-            url=self.url.data,
-            software=self.software.data,
-            success=self.success.data
+        self.url.data = URL(self.url.data)
+        self._obj = DataWipeDocument(
+            document_type='DataWipeDocument',
         )
+        self.populate_obj(self._obj)
+        self._obj.file_name = file_name
+        self._obj.file_hash = file_hash
+        db.session.add(self._obj)
+        if commit:
+            db.session.commit()
 
-        db.session.add(document)
+        return self._obj
+
+
+class DataWipeForm(NewActionForm):
+    document = FormField(DataWipeDocumentForm)
+
+    def save(self):
+        self.document.form.save(commit=False)
 
         Model = db.Model._decl_class_registry.data[self.type.data]()
         self.instance = Model()
@@ -593,16 +606,16 @@ class DataWipeForm(NewActionForm):
         severity = self.severity.data
         self.devices.data = self._devices
         self.severity.data = Severity[self.severity.data]
-        self.instance.document = document
-        self.instance.file_name = file_name
 
-        # import pdb; pdb.set_trace()
-        #TODO AttributeError: can't set attribute
+        document = copy.copy(self.document)
+        del self.document
         self.populate_obj(self.instance)
+        self.instance.document = document.form._obj
         db.session.add(self.instance)
         db.session.commit()
 
         self.devices.data = devices
         self.severity.data = severity
+        self.document = document
 
         return self.instance

From 1ed5d394c5718034dbf2aba3a32f7ab4a520e3c6 Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Wed, 9 Feb 2022 11:27:08 +0100
Subject: [PATCH 07/10] fix the new version of dispatch for NewDataWipeView

---
 ereuse_devicehub/inventory/views.py | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/ereuse_devicehub/inventory/views.py b/ereuse_devicehub/inventory/views.py
index 01bbe1db..1eae4de2 100644
--- a/ereuse_devicehub/inventory/views.py
+++ b/ereuse_devicehub/inventory/views.py
@@ -336,12 +336,17 @@ class NewAllocateView(NewActionView, DeviceListMix):
 
 class NewDataWipeView(NewActionView, DeviceListMix):
     methods = ['POST']
-    _form = DataWipeForm
+    form_class = DataWipeForm
 
     def dispatch_request(self):
-        dispatch = super().dispatch_request()
-        if dispatch:
-            return dispatch
+        self.form = self.form_class()
+
+        if self.form.validate_on_submit():
+            instance = self.form.save()
+            messages.success('Action "{}" created successfully!'.format(instance.type))
+
+            next_url = self.get_next_url()
+            return flask.redirect(next_url)
 
         lot_id = self.form.lot.data
         self.get_context(lot_id)

From f2d6c198cc7d4aafb6224ef7dfd178d629c9163e Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Wed, 9 Feb 2022 11:28:57 +0100
Subject: [PATCH 08/10] fix css for form documents into data_wipe html

---
 .../templates/inventory/data_wipe.html        | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/ereuse_devicehub/templates/inventory/data_wipe.html b/ereuse_devicehub/templates/inventory/data_wipe.html
index 8ce124ad..a44130ec 100644
--- a/ereuse_devicehub/templates/inventory/data_wipe.html
+++ b/ereuse_devicehub/templates/inventory/data_wipe.html
@@ -26,6 +26,28 @@
             {{ field }}
           {% elif field == form_new_datawipe.type %}
             {{ field }}
+          {% elif field == form_new_datawipe.document %}
+            {% for _field in field %}
+              <div class="col-12">
+                  {{ _field.label(class_="form-label") }}
+                  {% if _field == field.success %}
+                    <div class="form-check form-switch">
+                      {{ _field(class_="form-check-input") }}
+                      <small class="text-muted">{{ _field.description }}</small>
+                    </div>
+                  {% else %}
+                    {{ _field(class_="form-control") }}
+                    <small class="text-muted">{{ _field.description }}</small>
+                  {% endif %}
+                  {% if _field.errors %}
+                  <p class="text-danger">
+                    {% for error in _field.errors %}
+                      {{ error }}<br/>
+                    {% endfor %}
+                  </p>
+                  {% endif %}
+              </div>
+            {% endfor %}
           {% else %}
           <div class="col-12">
               {{ field.label(class_="form-label") }}

From d66e2b77fdaaa87e98d894d1e4d7d9a3fd28a728 Mon Sep 17 00:00:00 2001
From: Santiago Lamora <santiago@ribaguifi.com>
Date: Thu, 10 Feb 2022 11:16:57 +0100
Subject: [PATCH 09/10] Run flake8 linter on github workflow

---
 .github/workflows/flask.yml | 14 +++++++++++---
 tox.ini                     |  8 ++++++++
 2 files changed, 19 insertions(+), 3 deletions(-)
 create mode 100644 tox.ini

diff --git a/.github/workflows/flask.yml b/.github/workflows/flask.yml
index 151cc26d..328533a8 100644
--- a/.github/workflows/flask.yml
+++ b/.github/workflows/flask.yml
@@ -47,9 +47,6 @@ jobs:
         sudo apt-get update -qy
         sudo apt-get -y install postgresql-client
         python -m pip install --upgrade pip
-        pip install virtualenv
-        virtualenv env
-        source env/bin/activate
         pip install flake8 pytest coverage
         pip install -r requirements.txt
 
@@ -65,6 +62,17 @@ jobs:
         psql -h "localhost" -U "$POSTGRES_USER" -d "$POSTGRES_DB" -c "CREATE EXTENSION citext SCHEMA public;"
         psql -h "localhost" -U "$POSTGRES_USER" -d "$POSTGRES_DB" -c "CREATE EXTENSION pg_trgm SCHEMA public;"
 
+    - name: Lint with flake8
+      run: |
+        # stop the build if:
+        # - E9,F63,F7,F82: Python syntax errors or undefined names
+        # - E501: line longer than 120 characters
+        # - C901: complexity greater than 10
+        # - F401: modules imported but unused
+        # See: https://flake8.pycqa.org/en/latest/user/error-codes.html
+        flake8 . --select=E9,F63,F7,F82,E501,C901,F401
+        flake8 . --exit-zero
+
     - name: Run Tests
       run: |
         export SECRET_KEY=`python3 -c 'import secrets; print(secrets.token_hex())'`
diff --git a/tox.ini b/tox.ini
new file mode 100644
index 00000000..2f5d68fc
--- /dev/null
+++ b/tox.ini
@@ -0,0 +1,8 @@
+[flake8]
+count = True
+exclude =
+    migrations
+max-complexity = 10
+max-line-length = 120
+show-source = True
+statistics = True

From df4c6f740732d80f73ca0bb481ce59a61f53e8e0 Mon Sep 17 00:00:00 2001
From: Santiago Lamora <santiago@ribaguifi.com>
Date: Thu, 10 Feb 2022 11:29:47 +0100
Subject: [PATCH 10/10] flake8: remove unused imports & sort them

---
 ereuse_devicehub/inventory/forms.py | 35 +++++++++++++----------------
 1 file changed, 15 insertions(+), 20 deletions(-)

diff --git a/ereuse_devicehub/inventory/forms.py b/ereuse_devicehub/inventory/forms.py
index a90ecf1b..8959ae93 100644
--- a/ereuse_devicehub/inventory/forms.py
+++ b/ereuse_devicehub/inventory/forms.py
@@ -1,34 +1,29 @@
-import json
 import copy
+import json
 from json.decoder import JSONDecodeError
 
-from flask import g, request
-from flask_wtf import FlaskForm
-from sqlalchemy.util import OrderedSet
-from wtforms.fields import FormField
-from wtforms import (DateField, FloatField, HiddenField, IntegerField,
-                     MultipleFileField, FileField, SelectField, StringField,
-                     TextAreaField, BooleanField, URLField, validators, Form)
-
 from boltons.urlutils import URL
 from ereuse_devicehub.db import db
-from ereuse_devicehub.resources.hash_reports import insert_hash
-from ereuse_devicehub.resources.documents.models import DataWipeDocument
-from ereuse_devicehub.resources.action.models import (Action, RateComputer,
-                                                      Snapshot, VisualTest,
-                                                      DataWipe)
+from ereuse_devicehub.resources.action.models import RateComputer, Snapshot
 from ereuse_devicehub.resources.action.rate.v1_0 import CannotRate
 from ereuse_devicehub.resources.action.schemas import \
     Snapshot as SnapshotSchema
-from ereuse_devicehub.resources.action.views.snapshot import (move_json,
-                                                              save_json)
+from ereuse_devicehub.resources.action.views.snapshot import move_json, save_json
 from ereuse_devicehub.resources.device.models import (SAI, Cellphone, Computer,
-                                                      Device, Keyboard,
-                                                      MemoryCardReader,
-                                                      Monitor, Mouse,
-                                                      Smartphone, Tablet)
+                                                      Device, Keyboard, MemoryCardReader,
+                                                      Monitor, Mouse, Smartphone, Tablet)
+from flask import g, request
+from flask_wtf import FlaskForm
+from sqlalchemy.util import OrderedSet
+from wtforms import (BooleanField, DateField, FileField, FloatField, Form,
+                     HiddenField, IntegerField, MultipleFileField, SelectField,
+                     StringField, TextAreaField, URLField, validators)
+from wtforms.fields import FormField
+
 from ereuse_devicehub.resources.device.sync import Sync
+from ereuse_devicehub.resources.documents.models import DataWipeDocument
 from ereuse_devicehub.resources.enums import Severity, SnapshotSoftware
+from ereuse_devicehub.resources.hash_reports import insert_hash
 from ereuse_devicehub.resources.lot.models import Lot
 from ereuse_devicehub.resources.tag.model import Tag
 from ereuse_devicehub.resources.user.exceptions import InsufficientPermission