From a58feebe7ab3b388bebea4adfd9fc92c3ca3ae8b Mon Sep 17 00:00:00 2001
From: cproudlock <cproudlock@localhost>
Date: Fri, 26 Jun 2026 10:11:24 -0400
Subject: [PATCH] Fix software-compliance report + relationshiptypes zero-dates

- Add applications.isrequired (model + migration 7c02 + create/update + form
  checkbox). The software-compliance report compared required apps against
  installs but no such flag existed, so it 500'd; now it works.
- Repair relationshiptypes rows with 0000-00-00 createddate/modifieddate
  (partof/connectedto) to real timestamps.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 .../views/applications/ApplicationForm.vue    |  7 +++++
 .../versions/7c02_application_isrequired.py   | 29 +++++++++++++++++++
 shopdb/core/api/applications.py               |  3 +-
 shopdb/core/models/application.py             |  4 +++
 4 files changed, 42 insertions(+), 1 deletion(-)
 create mode 100644 migrations/versions/7c02_application_isrequired.py

diff --git a/frontend/src/views/applications/ApplicationForm.vue b/frontend/src/views/applications/ApplicationForm.vue
index af8e8cb..0594338 100644
--- a/frontend/src/views/applications/ApplicationForm.vue
+++ b/frontend/src/views/applications/ApplicationForm.vue
@@ -65,6 +65,10 @@
               <input type="checkbox" v-model="form.isprinter" />
               Printer App
             </label>
+            <label>
+              <input type="checkbox" v-model="form.isrequired" />
+              Required on all PCs
+            </label>
             <label>
               <input type="checkbox" v-model="form.ishidden" />
               Hidden
@@ -168,6 +172,7 @@ const form = ref({
   isinstallable: false,
   islicenced: false,
   isprinter: false,
+  isrequired: false,
   ishidden: false,
   applicationlink: '',
   documentationpath: '',
@@ -196,6 +201,7 @@ onMounted(async () => {
         isinstallable: app.isinstallable || false,
         islicenced: app.islicenced || false,
         isprinter: app.isprinter || false,
+        isrequired: app.isrequired || false,
         ishidden: app.ishidden || false,
         applicationlink: app.applicationlink || '',
         documentationpath: app.documentationpath || '',
@@ -224,6 +230,7 @@ async function saveApplication() {
       isinstallable: form.value.isinstallable,
       islicenced: form.value.islicenced,
       isprinter: form.value.isprinter,
+      isrequired: form.value.isrequired,
       ishidden: form.value.ishidden,
       applicationlink: form.value.applicationlink || null,
       documentationpath: form.value.documentationpath || null,
diff --git a/migrations/versions/7c02_application_isrequired.py b/migrations/versions/7c02_application_isrequired.py
new file mode 100644
index 0000000..b032899
--- /dev/null
+++ b/migrations/versions/7c02_application_isrequired.py
@@ -0,0 +1,29 @@
+"""Add applications.isrequired
+
+Backs the software-compliance report, which compares required applications
+against what is installed per PC. There was no such flag before, so the report
+errored.
+
+Revision ID: 7c02_app_isrequired
+Revises: 7c01_drop_legacy_machine
+Create Date: 2026-06-26
+
+"""
+from alembic import op
+import sqlalchemy as sa
+
+
+revision = '7c02_app_isrequired'
+down_revision = '7c01_drop_legacy_machine'
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    op.add_column('applications', sa.Column(
+        'isrequired', sa.Boolean(), nullable=True, server_default=sa.false(),
+        comment='Required on all PCs (drives the software-compliance report)'))
+
+
+def downgrade():
+    op.drop_column('applications', 'isrequired')
diff --git a/shopdb/core/api/applications.py b/shopdb/core/api/applications.py
index cc01155..8017121 100644
--- a/shopdb/core/api/applications.py
+++ b/shopdb/core/api/applications.py
@@ -131,6 +131,7 @@ def create_application():
         ishidden=data.get('ishidden', False),
         isprinter=data.get('isprinter', False),
         islicenced=data.get('islicenced', False),
+        isrequired=data.get('isrequired', False),
         image=data.get('image')
     )
 
@@ -168,7 +169,7 @@ def update_application(app_id: int):
     fields = [
         'appname', 'appdescription', 'supportteamid', 'isinstallable',
         'applicationnotes', 'installpath', 'applicationlink', 'documentationpath',
-        'ishidden', 'isprinter', 'islicenced', 'image', 'isactive'
+        'ishidden', 'isprinter', 'islicenced', 'isrequired', 'image', 'isactive'
     ]
 
     changes = {}
diff --git a/shopdb/core/models/application.py b/shopdb/core/models/application.py
index cd30e78..f7f3b5f 100644
--- a/shopdb/core/models/application.py
+++ b/shopdb/core/models/application.py
@@ -53,6 +53,10 @@ class Application(BaseModel):
     ishidden = db.Column(db.Boolean, default=False)
     isprinter = db.Column(db.Boolean, default=False)
     islicenced = db.Column(db.Boolean, default=False)
+    isrequired = db.Column(
+        db.Boolean, default=False,
+        comment='Required on all PCs (drives the software-compliance report)'
+    )
     image = db.Column(db.String(255))
 
     # Relationships