From b140dc5f899be61bca36b56914d0f72320d88a17 Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Mon, 13 Mar 2023 17:04:57 +0100
Subject: [PATCH 1/3] fix without logo

---
 ereuse_devicehub/forms.py           | 3 +++
 ereuse_devicehub/inventory/forms.py | 3 +++
 ereuse_devicehub/views.py           | 3 ++-
 3 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/ereuse_devicehub/forms.py b/ereuse_devicehub/forms.py
index 4980b41e..ef383588 100644
--- a/ereuse_devicehub/forms.py
+++ b/ereuse_devicehub/forms.py
@@ -141,6 +141,9 @@ class SanitizationEntityForm(FlaskForm):
         if not is_valid:
             return False
 
+        if not self.logo.data:
+            return True
+
         extensions = ["jpg", "jpeg", "png", "gif", "svg"]
         if self.logo.data.lower().split(".")[-1] not in extensions:
             txt = "Error in Url field - accepted only .PNG, .JPG and .GIF. extensions"
diff --git a/ereuse_devicehub/inventory/forms.py b/ereuse_devicehub/inventory/forms.py
index dff4e87e..ffb8d13c 100644
--- a/ereuse_devicehub/inventory/forms.py
+++ b/ereuse_devicehub/inventory/forms.py
@@ -1565,6 +1565,9 @@ class CustomerDetailsForm(FlaskForm):
         if not is_valid:
             return is_valid
 
+        if not self.logo.data:
+            return True
+
         extensions = ["jpg", "jpeg", "png", "gif", "svg"]
         if self.logo.data.lower().split(".")[-1] not in extensions:
             txt = "Error in Url field - accepted only .PNG, .JPG and .GIF. extensions"
diff --git a/ereuse_devicehub/views.py b/ereuse_devicehub/views.py
index 65cb4d3a..8f9a5e1d 100644
--- a/ereuse_devicehub/views.py
+++ b/ereuse_devicehub/views.py
@@ -145,7 +145,8 @@ class SanitizationEntityView(View):
             messages.error('Error modifying Sanitization data!')
             if form.errors:
                 for k in form.errors.keys():
-                    txt = "{}: {}".format(k, form.errors[k])
+                    errors = ", ".join(form.errors[k])
+                    txt = "{}: {}".format(k, errors)
                     messages.error(txt)
 
         return flask.redirect(flask.url_for('core.user-profile'))

From 21d251e6c03008b878af45dc549d445ba86b576a Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Mon, 13 Mar 2023 17:52:00 +0100
Subject: [PATCH 2/3] add reopen transfer

---
 ereuse_devicehub/inventory/views.py           | 19 +++++++++++++++
 .../templates/inventory/device_list.html      | 23 +++++++++++++------
 2 files changed, 35 insertions(+), 7 deletions(-)

diff --git a/ereuse_devicehub/inventory/views.py b/ereuse_devicehub/inventory/views.py
index b03cd345..2526a850 100644
--- a/ereuse_devicehub/inventory/views.py
+++ b/ereuse_devicehub/inventory/views.py
@@ -841,6 +841,21 @@ class NewTransferView(GenericMixin):
         return flask.render_template(self.template_name, **self.context)
 
 
+class OpenTransferView(GenericMixin):
+    methods = ['GET']
+
+    def dispatch_request(self, lot_id=None):
+        lot = Lot.query.filter_by(id=lot_id).one()
+        next_url = url_for('inventory.lotdevicelist', lot_id=str(lot_id))
+
+        if hasattr(lot, 'transfer'):
+            lot.transfer.date = None
+            db.session.commit()
+            messages.success('Transfer was reopen successfully!')
+
+        return flask.redirect(next_url)
+
+
 class EditTransferView(GenericMixin):
     methods = ['POST']
     form_class = EditTransferForm
@@ -1601,3 +1616,7 @@ devices.add_url_rule(
     '/device/erasure/<int:orphans>/',
     view_func=ErasureListView.as_view('device_erasure_list_orphans'),
 )
+devices.add_url_rule(
+    '/lot/<string:lot_id>/opentransfer/',
+    view_func=OpenTransferView.as_view('open_transfer'),
+)
diff --git a/ereuse_devicehub/templates/inventory/device_list.html b/ereuse_devicehub/templates/inventory/device_list.html
index f89c93f6..7f5ac904 100644
--- a/ereuse_devicehub/templates/inventory/device_list.html
+++ b/ereuse_devicehub/templates/inventory/device_list.html
@@ -516,13 +516,13 @@
               </div>
               {% if lot and not lot.is_temporary %}
               <div id="trade-documents-list" class="tab-pane fade trade-documents-list">
-              <div class="btn-group dropdown ml-1 mt-1" uib-dropdown="">
-                <a href="{{ url_for('inventory.trade_document_add', lot_id=lot.id)}}" class="btn btn-primary">
-                  <i class="bi bi-plus"></i>
-                  Add new document
-                  <span class="caret"></span>
-                </a>
-              </div>
+                <div class="btn-group dropdown ml-1 mt-1" uib-dropdown="">
+                  <a href="{{ url_for('inventory.trade_document_add', lot_id=lot.id)}}" class="btn btn-primary">
+                    <i class="bi bi-plus"></i>
+                    Add new document
+                    <span class="caret"></span>
+                  </a>
+                </div>
 
                 <h5 class="card-title">Documents</h5>
                 <table class="table">
@@ -565,6 +565,15 @@
                 </table>
               </div>
               <div id="edit-transfer" class="tab-pane fade edit-transfer">
+                {% if form_transfer.date.data %}
+                <div class="btn-group dropdown ml-1 mt-1" uib-dropdown="">
+                  <a href="{{ url_for('inventory.open_transfer', lot_id=lot.id)}}" class="btn btn-primary">
+                    <i class="bi bi-plus"></i>
+                    Reopen a transfer
+                    <span class="caret"></span>
+                  </a>
+                </div>
+                {% endif %}
                 <h5 class="card-title">Transfer</h5>
                 <form method="post" action="{{ url_for('inventory.edit_transfer', lot_id=lot.id) }}" class="row g-3 needs-validation" novalidate>
                   {{ form_transfer.csrf_token }}

From e7833c17275f4cf0b797b21d3045d0db4ab500d2 Mon Sep 17 00:00:00 2001
From: Cayo Puigdefabregas <cayo@ribaguifi.com>
Date: Mon, 13 Mar 2023 17:59:08 +0100
Subject: [PATCH 3/3] fix test

---
 tests/test_basic.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/test_basic.py b/tests/test_basic.py
index e9927a0e..4b734079 100644
--- a/tests/test_basic.py
+++ b/tests/test_basic.py
@@ -68,6 +68,7 @@ def test_api_docs(client: Client):
         '/inventory/lot/{lot_id}/receivernote/',
         '/inventory/lot/{lot_id}/trade-document/add/',
         '/inventory/lot/{lot_id}/transfer/{type_id}/',
+        '/inventory/lot/{lot_id}/opentransfer/',
         '/inventory/lot/{lot_id}/transfer/',
         '/inventory/lot/transfer/{type_id}/',
         '/inventory/lot/{lot_id}/upload-snapshot/',