Merge pull request 'mijn eerste aanpassing' (#2) from import_eigendom into main

Reviewed-on: #2

DATABASE.md

@@ -118,6 +118,7 @@ python beheer.py rechten jan sg-antwerpen alles | geen
 | `koppelingen` | Koppelingen van deze school |
 | `gebruikersbeheer` | Gebruikers & rechten van deze school |
 | `database_toestellen` | Beheer › Database: toestellen importeren uit Excel |
+| `database_eigendom` | Beheer › Database: eigendom (toestel ↔ leerling) importeren uit Excel |
 
 Een nieuw onderdeel voeg je toe in `rechten.py › ONDERDELEN`. Bij het opstarten komt het automatisch in de tabel `Onderdelen`.
 

blueprints/database_bp.py

@@ -6,6 +6,8 @@ Toestellen:
     GET  /beheer/database/toestellen/sjabloon.xlsx → sjabloon downloaden
     POST /beheer/database/toestellen/controleren   → Excel lezen en voorbeeld tonen (niets bewaard)
     POST /beheer/database/toestellen/importeren    → bevestigen: het voorbeeld uitvoeren
+
+Eigendom (toekenningen toestel ↔ leerling): dezelfde vier routes onder /beheer/database/eigendom
 """
 import json
 
@@ -14,6 +16,7 @@ from flask import Blueprint, render_template, request, Response, redirect, url_f
 from blueprints.auth import login_required
 from database import execute_query
 from rechten import recht_vereist, log
+from services import eigendom_import as ei
 from services import toestellen_import as ti
 from tenancy import huidige_school
 
@@ -98,3 +101,86 @@ def toestellen_importeren():
     if t['fout']:
         flash(f"{t['fout']} rij(en) met een fout werden overgeslagen.", "warning")
     return terug
+
+
+# ---------------------------------------------------------------------------
+# Eigendom
+# ---------------------------------------------------------------------------
+@database_bp.route('/eigendom', methods=['GET'])
+@login_required
+@recht_vereist('database_eigendom')
+def eigendom():
+    try:
+        aantal = execute_query("SELECT COUNT(*) AS n FROM Eigendom WHERE Datum_inlevering IS NULL")[0]['n']
+    except Exception:
+        aantal = None
+    return render_template('database_eigendom.html', plan=None, aantal=aantal, kolommen=ei.KOLOMMEN,
+                           max_rijen=ei.MAX_RIJEN)
+
+
+@database_bp.route('/eigendom/sjabloon.xlsx', methods=['GET'])
+@login_required
+@recht_vereist('database_eigendom')
+def eigendom_sjabloon():
+    school = huidige_school()
+    data = ei.maak_sjabloon(school.naam)
+    return Response(data, mimetype='application/vnd.openxmlformats-officedocument.spreadsheetml.sheet',
+                    headers={'Content-Disposition': f"attachment; filename=Sjabloon_eigendom_{school.code}.xlsx"})
+
+
+@database_bp.route('/eigendom/controleren', methods=['POST'])
+@login_required
+@recht_vereist('database_eigendom')
+def eigendom_controleren():
+    terug = redirect(url_for('database_beheer.eigendom'))
+    bestand = request.files.get('bestand')
+    if not bestand or not bestand.filename:
+        flash("Kies eerst een Excel-bestand.", "error")
+        return terug
+    if not bestand.filename.lower().endswith(('.xlsx', '.xlsm')):
+        flash("Enkel Excel-bestanden (.xlsx) kunnen ingelezen worden. Sla een .xls of .csv eerst op als .xlsx.", "error")
+        return terug
+    modus = 'overschrijven' if request.form.get('modus') == 'overschrijven' else 'aanvullen'
+    try:
+        rijen, onbekend = ei.lees_excel(bestand)
+    except ei.ImportFout as e:
+        flash(str(e), "error")
+        return terug
+    if not rijen:
+        flash("Het bestand bevat geen toekenningen (enkel kolomkoppen of lege rijen).", "warning")
+        return terug
+    try:
+        plan = ei.maak_plan(rijen, modus)
+    except Exception as e:
+        flash(f"Controleren mislukt: {e}", "error")
+        return terug
+    return render_template('database_eigendom.html', plan=plan, bestandsnaam=bestand.filename, onbekend=onbekend,
+                           rijen_json=json.dumps(rijen, ensure_ascii=False), kolommen=ei.KOLOMMEN,
+                           max_rijen=ei.MAX_RIJEN, aantal=None)
+
+
+@database_bp.route('/eigendom/importeren', methods=['POST'])
+@login_required
+@recht_vereist('database_eigendom')
+def eigendom_importeren():
+    terug = redirect(url_for('database_beheer.eigendom'))
+    modus = 'overschrijven' if request.form.get('modus') == 'overschrijven' else 'aanvullen'
+    try:
+        rijen = json.loads(request.form.get('rijen') or '[]')
+        if not isinstance(rijen, list) or len(rijen) > ei.MAX_RIJEN or not all(isinstance(r, dict) for r in rijen):
+            raise ValueError
+    except ValueError:
+        flash("De gegevens van het voorbeeld zijn ongeldig. Upload het bestand opnieuw.", "error")
+        return terug
+    try:
+        plan, nieuw, bijgewerkt = ei.voer_uit(rijen, modus, session.get('user'))
+    except Exception as e:
+        flash(f"Importeren mislukt, er is niets bewaard: {e}", "error")
+        return terug
+    t = plan['tellers']
+    log(f"Eigendom geïmporteerd uit Excel: {nieuw} nieuw, {bijgewerkt} bijgewerkt ({modus})")
+    ongewijzigd = f", {t['ongewijzigd']} ongewijzigd" if t['ongewijzigd'] else ""
+    flash(f"Import klaar: {nieuw} toekenning(en) toegevoegd, {bijgewerkt} bijgewerkt{ongewijzigd}.", "success")
+    if t['fout']:
+        flash(f"{t['fout']} rij(en) met een fout werden overgeslagen.", "warning")
+    return terug

desktop.ini

@@ -0,0 +1,6 @@
+[.ShellClassInfo]
+IconResource=C:\Windows\System32\SHELL32.dll,33
+[ViewState]
+Mode=
+Vid=
+FolderType=Documents

rechten.py

@@ -88,6 +88,7 @@ ONDERDELEN = [
     ('nieuw_schooljaar',         'Nieuw schooljaar (accounts & toestellen)',        'Beheer'),
     ('ip_whitelist',             'IP-adressen toestaan voor deze school',           'Beheer'),
     ('database_toestellen',      'Database: toestellen importeren uit Excel',       'Beheer'),
+    ('database_eigendom',        'Database: eigendom importeren uit Excel',         'Beheer'),
     # --- PIVOT ---
     ('gebruikersbeheer',         'Gebruikers & rechten van deze school',            'PIVOT'),
     ('koppelingen',              'Koppelingen van deze school',                     'PIVOT'),

services/eigendom_import.py

@@ -0,0 +1,431 @@
+"""
+Beheer › Database › Eigendom: toekenningen toestel ↔ leerling (tabel Eigendom) aanvullen vanuit een Excel-bestand.
+
+Zelfde werkwijze als de toestellen-import (services/toestellen_import.py):
+    1. lees_excel()   : het geüploade bestand → ruwe rijen (enkel tekst / ISO-datums)
+    2. maak_plan()    : ruwe rijen → per rij 'nieuw', 'bijwerken', 'ongewijzigd' of 'fout' (niets wordt bewaard)
+    3. voer_uit()     : het plan in één transactie uitvoeren (het plan wordt opnieuw gemaakt, nooit blind vertrouwd)
+
+Elke rij is één toekenning: een leerling (internnummer, Smartschool-nummer of gebruikersnaam) en een toestel
+(toestelnummer of serienummer) die al in PIVOT bestaan, met een datum van toekenning en eventueel van inlevering.
+    - Zonder datum inlevering heeft de leerling het toestel nu (toestel → 'Uitgeleend').
+    - Met datum inlevering is het geschiedenis (de status van het toestel verandert niet).
+    - Bestaat dezelfde toekenning al (zelfde leerling, toestel en datum toekenning; of de huidige als die datum
+      leeg is), dan wordt enkel een lege datum inlevering aangevuld (= inleveren, toestel → 'Beschikbaar').
+    - Een toestel kan nooit tegelijk bij twee leerlingen zijn. Staat het nu bij een andere leerling, dan is dat
+      een fout, of (keuze 'overschrijven') wordt die toekenning eerst afgesloten op de nieuwe datum toekenning.
+Er wordt nooit iets verwijderd.
+"""
+import io
+from datetime import date, datetime
+
+from database import execute_query, get_db_connection
+from services.toestellen_import import ImportFout, _norm, _cel, _datum_tekst, _toon
+
+MAX_RIJEN = 5000
+
+# (sleutel, kolomkop in het sjabloon, max. lengte, uitleg, andere namen die ook herkend worden)
+KOLOMMEN = [
+    ('leerling', 'Leerling', 255,
+     "Internnummer, Smartschool-nummer of gebruikersnaam van een leerling die al in PIVOT staat. Verplicht.",
+     ('internnummer', 'intern nummer', 'leerlingnummer', 'ss nummer', 'smartschool nummer', 'gebruikersnaam')),
+    ('toestel', 'Toestel', 100,
+     "Toestelnummer (Schooleigen_ID) of serienummer van een toestel dat al in PIVOT staat. Verplicht.",
+     ('toestelnummer', 'schooleigen id', 'kamxm nummer', 'kamxm', 'serienummer', 'serial', 'serial number')),
+    ('datum_toekenning', 'Datum toekenning', None,
+     "Datum (dd-mm-jjjj) waarop de leerling het toestel kreeg. Leeg = vandaag "
+     "(verplicht als de datum inlevering ingevuld is).",
+     ('toekenning', 'datum uitdeling', 'uitgedeeld', 'uitgedeeld op', 'toegekend op')),
+    ('datum_inlevering', 'Datum inlevering', None,
+     "Datum (dd-mm-jjjj) waarop het toestel terug ingeleverd werd. Leeg = de leerling heeft het toestel nu.",
+     ('inlevering', 'ingeleverd', 'ingeleverd op', 'datum inleveren')),
+]
+SLEUTELS = [k[0] for k in KOLOMMEN]
+KOPPEN = {k[0]: k[1] for k in KOLOMMEN}
+
+_HERKEN = {}
+for _k, _kop, _m, _u, _alias in KOLOMMEN:
+    for _naam in (_k, _kop) + tuple(_alias):
+        _HERKEN[_norm(_naam)] = _k
+
+
+# ---------------------------------------------------------------------------
+# Sjabloon
+# ---------------------------------------------------------------------------
+def maak_sjabloon(schoolnaam=''):
+    """Excel-sjabloon (bytes): tabblad Eigendom (in te vullen) en Uitleg."""
+    from openpyxl import Workbook
+    from openpyxl.styles import Font, PatternFill, Alignment
+    from openpyxl.worksheet.datavalidation import DataValidation
+
+    wb = Workbook()
+    ws = wb.active
+    ws.title = 'Eigendom'
+    ws.append([k[1] for k in KOLOMMEN])
+    kop_vulling = PatternFill('solid', fgColor='1E6FA5')
+    for c in ws[1]:
+        c.font = Font(bold=True, color='FFFFFF')
+        c.fill = kop_vulling
+        c.alignment = Alignment(vertical='center')
+    breedtes = {'Leerling': 20, 'Toestel': 20, 'Datum toekenning': 18, 'Datum inlevering': 18}
+    for i, k in enumerate(KOLOMMEN):
+        letter = ws.cell(row=1, column=i + 1).column_letter
+        ws.column_dimensions[letter].width = breedtes.get(k[1], 16)
+    ws.freeze_panes = 'A2'
+    ws.row_dimensions[1].height = 22
+
+    kol = {k[0]: ws.cell(row=1, column=i + 1).column_letter for i, k in enumerate(KOLOMMEN)}
+    # Nummers als tekst (anders maakt Excel van 00123 → 123)
+    for sleutel in ('leerling', 'toestel'):
+        for r in range(2, MAX_RIJEN + 2):
+            ws[f"{kol[sleutel]}{r}"].number_format = '@'
+    for sleutel in ('datum_toekenning', 'datum_inlevering'):
+        for r in range(2, MAX_RIJEN + 2):
+            ws[f"{kol[sleutel]}{r}"].number_format = 'DD-MM-YYYY'
+        dv_datum = DataValidation(type='date', operator='greaterThan', formula1='1', allow_blank=True,
+                                  showErrorMessage=True, errorTitle=KOPPEN[sleutel],
+                                  error='Vul een datum in, bv. 01-09-2026.')
+        dv_datum.add(f"{kol[sleutel]}2:{kol[sleutel]}{MAX_RIJEN + 1}")
+        ws.add_data_validation(dv_datum)
+
+    # Uitleg
+    wu = wb.create_sheet('Uitleg')
+    wu.column_dimensions['A'].width = 18
+    wu.column_dimensions['B'].width = 95
+    wu.append([f"Eigendom importeren in PIVOT{' – ' + schoolnaam if schoolnaam else ''}"])
+    wu['A1'].font = Font(bold=True, size=14)
+    regels = [
+        "Vul het tabblad 'Eigendom' in: één toekenning (leerling + toestel) per rij, vanaf rij 2. "
+        "Laat de kolomkoppen in rij 1 staan.",
+        "De leerling en het toestel moeten al in PIVOT staan. Nieuwe toestellen voeg je eerst toe via "
+        "Database › Toestellen importeren.",
+        "Zonder datum inlevering heeft de leerling het toestel nu: het toestel krijgt de status Uitgeleend.",
+        "Met een datum inlevering wordt de toekenning als geschiedenis bewaard.",
+        "Bestaat de toekenning al, dan wordt enkel een lege datum inlevering aangevuld (= het toestel inleveren).",
+        "Een toestel kan niet tegelijk bij twee leerlingen zijn. Staat het nu bij een andere leerling, dan kies je bij "
+        "het uploaden of dat een fout is of dat die toekenning eerst afgesloten wordt.",
+        f"Er wordt nooit iets verwijderd. Je krijgt eerst een voorbeeld te zien. Maximaal {MAX_RIJEN} rijen per bestand.",
+    ]
+    for t in regels:
+        wu.append(['', t])
+    wu.append([])
+    wu.append(['Kolom', 'Uitleg'])
+    for c in wu[wu.max_row]:
+        c.font = Font(bold=True)
+    for _k, kop, maxlen, uitleg, _a in KOLOMMEN:
+        wu.append([kop, uitleg])
+    for rij in wu.iter_rows(min_row=2):
+        for c in rij:
+            c.alignment = Alignment(wrap_text=True, vertical='top')
+    wu.append([])
+    wu.append(['Voorbeeld', 'Leerling 12345 · Toestel CB-0123 · Datum toekenning 01-09-2026 · Datum inlevering leeg'])
+
+    wb.active = 0
+    buf = io.BytesIO()
+    wb.save(buf)
+    return buf.getvalue()
+
+
+# ---------------------------------------------------------------------------
+# Excel lezen
+# ---------------------------------------------------------------------------
+def lees_excel(bestand):
+    """Leest een .xlsx (bestandsobject of bytes). Geeft (rijen, onbekende_kolommen).
+    rijen = [{'rij': <rijnummer in Excel>, 'leerling': '...', ...}] (enkel niet-lege rijen)."""
+    from openpyxl import load_workbook
+    data = bestand if isinstance(bestand, (bytes, bytearray)) else bestand.read()
+    if not data:
+        raise ImportFout("Het bestand is leeg.")
+    try:
+        wb = load_workbook(io.BytesIO(data), read_only=True, data_only=True)
+    except Exception:
+        raise ImportFout("Dit bestand kan niet gelezen worden. Gebruik een Excel-bestand (.xlsx), "
+                         "bv. het sjabloon van deze pagina.")
+    try:
+        ws = wb['Eigendom'] if 'Eigendom' in wb.sheetnames else wb.worksheets[0]
+        rijen_iter = ws.iter_rows(values_only=True)
+        # Kopregel zoeken in de eerste 10 rijen: de eerste rij met een herkende kolom
+        kop_index, onbekend, rijnr = None, [], 0
+        for rij in rijen_iter:
+            rijnr += 1
+            gevonden = {i: _HERKEN[_norm(v)] for i, v in enumerate(rij or ()) if _norm(v) in _HERKEN}
+            if gevonden:
+                kop_index = {}
+                for i, sleutel in gevonden.items():
+                    kop_index.setdefault(sleutel, i)      # dubbele kolom: de eerste telt
+                onbekend = [_cel(v) for i, v in enumerate(rij) if _cel(v) and i not in gevonden]
+                break
+            if rijnr >= 10:
+                break
+        if not kop_index:
+            raise ImportFout("Geen kolomkoppen gevonden. Rij 1 moet de kolomnamen van het sjabloon bevatten "
+                             f"({', '.join(KOPPEN.values())}).")
+        ontbreekt = [KOPPEN[s] for s in ('leerling', 'toestel') if s not in kop_index]
+        if ontbreekt:
+            raise ImportFout(f"De kolom {' en '.join(ontbreekt)} ontbreekt.")
+
+        uit = []
+        for rij in rijen_iter:
+            rijnr += 1
+            waarden = {s: _cel(rij[i]) if rij and i < len(rij) else '' for s, i in kop_index.items()}
+            if not any(waarden.values()):
+                continue
+            if len(uit) >= MAX_RIJEN:
+                raise ImportFout(f"Het bestand bevat meer dan {MAX_RIJEN} rijen. Splits het op in kleinere bestanden.")
+            uit.append({'rij': rijnr, **{s: waarden.get(s, '') for s in SLEUTELS}})
+        return uit, onbekend
+    finally:
+        wb.close()
+
+
+# ---------------------------------------------------------------------------
+# Plan maken (niets bewaren)
+# ---------------------------------------------------------------------------
+def _d(w):
+    return w.date() if isinstance(w, datetime) else w
+
+
+def _overlapt(begin1, einde1, begin2, einde2):
+    """Overlappen twee periodes? Einde None = nog niet ingeleverd. Inleveren en opnieuw uitdelen op dezelfde dag mag."""
+    return (einde2 is None or begin1 < einde2) and (einde1 is None or begin2 < einde1)
+
+
+def _naam(l):
+    naam = ' '.join(x for x in (l.get('Voornaam'), l.get('Naam')) if x)
+    return f"{naam} ({l['Internnummer']})" if naam else l['Internnummer']
+
+
+def _toestel_label(t):
+    return t.get('Schooleigen_ID') or t.get('Serienummer') or str(t['Toestel_ID'])
+
+
+def maak_plan(rijen, modus='aanvullen', vandaag=None):
+    """Bepaalt per rij wat er zou gebeuren. modus: 'aanvullen' (toestel bij een andere leerling = fout) of
+    'overschrijven' (die toekenning eerst afsluiten; ook een andere datum inlevering wordt dan overschreven).
+    Geeft {'rijen': [...], 'tellers': {...}}; elke rij heeft 'actie', 'meldingen', 'wijzigingen' en wat er in de
+    database moet gebeuren ('waarden', 'eigendom_id', 'sluit', 'toestel_status')."""
+    overschrijven = modus == 'overschrijven'
+    vandaag = vandaag or date.today()
+
+    # Leerlingen: internnummer gaat voor Smartschool-nummer, dat gaat voor gebruikersnaam
+    leerlingen = execute_query("SELECT Internnummer, SS_nummer, Gebruikersnaam, Naam, Voornaam, Vertrokken_op "
+                               "FROM Leerlingen") or []
+    op_leerling = {}
+    for veld in ('Gebruikersnaam', 'SS_nummer', 'Internnummer'):
+        for l in leerlingen:
+            w = str(l[veld] or '').strip().upper()
+            if w:
+                op_leerling[w] = l
+    toestellen = [dict(t) for t in execute_query("SELECT * FROM Toestellen") or []]
+    op_nummer = {(t['Schooleigen_ID'] or '').strip().upper(): t for t in toestellen if (t['Schooleigen_ID'] or '').strip()}
+    op_serie = {}
+    for t in toestellen:
+        s = (t['Serienummer'] or '').strip().upper()
+        if s:
+            op_serie.setdefault(s, t)
+    toestel_op_id = {t['Toestel_ID']: t for t in toestellen}
+
+    # Bestaande toekenningen per toestel (kopie: het plan past ze aan zodat latere rijen ermee rekening houden)
+    per_toestel, actief_per_leerling = {}, {}
+    for e in execute_query("SELECT Eigendom_ID, Internnummer, Toestel_ID, Datum_toekenning, Datum_inlevering "
+                           "FROM Eigendom") or []:
+        e = {**e, 'Datum_toekenning': _d(e['Datum_toekenning']), 'Datum_inlevering': _d(e['Datum_inlevering']),
+             'rij': None}
+        per_toestel.setdefault(e['Toestel_ID'], []).append(e)
+        if e['Datum_inlevering'] is None:
+            actief_per_leerling.setdefault(e['Internnummer'], set()).add(e['Toestel_ID'])
+
+    uit = []
+    tellers = {'nieuw': 0, 'bijwerken': 0, 'ongewijzigd': 0, 'fout': 0}
+
+    for invoer in rijen:
+        r = {'rij': invoer.get('rij'), 'invoer': {s: str(invoer.get(s) or '').strip() for s in SLEUTELS},
+             'meldingen': [], 'wijzigingen': [], 'actie': None, 'label': None,
+             'waarden': {}, 'toestel_id': None, 'eigendom_id': None, 'sluit': None, 'toestel_status': None}
+        v = r['invoer']
+        fouten = []
+
+        # --- waarden controleren ---
+        leerling = toestel = None
+        if not v['leerling']:
+            fouten.append("Leerling is verplicht.")
+        else:
+            leerling = op_leerling.get(v['leerling'].upper())
+            if not leerling:
+                fouten.append(f"Leerling '{v['leerling']}' staat niet in PIVOT.")
+        if not v['toestel']:
+            fouten.append("Toestel is verplicht.")
+        else:
+            toestel = op_nummer.get(v['toestel'].upper()) or op_serie.get(v['toestel'].upper())
+            if not toestel:
+                fouten.append(f"Toestel '{v['toestel']}' staat niet in PIVOT. Voeg het eerst toe via "
+                              f"Database › Toestellen.")
+        begin = einde = None
+        for s in ('datum_toekenning', 'datum_inlevering'):
+            if v[s]:
+                try:
+                    d = _datum_tekst(v[s])
+                except ValueError:
+                    fouten.append(f"{KOPPEN[s]} '{v[s]}' is geen geldige datum (gebruik dd-mm-jjjj).")
+                    continue
+                if d > vandaag:
+                    fouten.append(f"{KOPPEN[s]} {_toon(d)} ligt in de toekomst.")
+                elif s == 'datum_toekenning':
+                    begin = d
+                else:
+                    einde = d
+        if v['datum_inlevering'] and not v['datum_toekenning']:
+            fouten.append("Datum toekenning is verplicht als de datum inlevering ingevuld is.")
+        if begin and einde and einde < begin:
+            fouten.append("Datum inlevering ligt vóór de datum toekenning.")
+
+        if not fouten:
+            intern = leerling['Internnummer']
+            tid = r['toestel_id'] = toestel['Toestel_ID']
+            r['label'] = f"{_naam(leerling)} · toestel {_toestel_label(toestel)}"
+            lijst = per_toestel.setdefault(tid, [])
+
+            # --- bestaat deze toekenning al? ---
+            if begin:
+                bestaand = next((e for e in lijst if e['Internnummer'] == intern and e['Datum_toekenning'] == begin),
+                                None)
+            else:
+                bestaand = next((e for e in lijst if e['Internnummer'] == intern and e['Datum_inlevering'] is None),
+                                None)
+            if not bestaand and not einde:
+                # de leerling heeft dit toestel al (sinds een andere datum)
+                bestaand = next((e for e in lijst if e['Internnummer'] == intern and e['Datum_inlevering'] is None),
+                                None)
+            if bestaand:
+                if bestaand['rij']:
+                    fouten.append(f"Deze toekenning staat al op rij {bestaand['rij']}.")
+                elif einde and einde != bestaand['Datum_inlevering'] and (bestaand['Datum_inlevering'] is None
+                                                                          or overschrijven):
+                    andere = [e for e in lijst if e is not bestaand]
+                    botsing = next((e for e in andere if _overlapt(bestaand['Datum_toekenning'], einde,
+                                                                   e['Datum_toekenning'], e['Datum_inlevering'])), None)
+                    if botsing:
+                        fouten.append(f"Het toestel was vanaf {_toon(botsing['Datum_toekenning'])} al bij "
+                                      f"leerling {botsing['Internnummer']}.")
+                    else:
+                        r['actie'] = 'bijwerken'
+                        r['eigendom_id'] = bestaand['Eigendom_ID']
+                        r['waarden'] = {'Datum_inlevering': einde}
+                        r['wijzigingen'].append(f"Datum inlevering: {_toon(bestaand['Datum_inlevering']) or '–'} "
+                                                f"→ {_toon(einde)}")
+                        if bestaand['Datum_inlevering'] is None:
+                            actief_per_leerling.get(intern, set()).discard(tid)
+                            if toestel['Status'] == 'Uitgeleend':
+                                r['toestel_status'] = 'Beschikbaar'
+                                r['wijzigingen'].append("Toestel: Uitgeleend → Beschikbaar")
+                                toestel['Status'] = 'Beschikbaar'
+                        bestaand['Datum_inlevering'] = einde
+                        bestaand['rij'] = r['rij']
+                else:
+                    r['actie'] = 'ongewijzigd'
+                    if einde and bestaand['Datum_inlevering'] and einde != bestaand['Datum_inlevering']:
+                        r['meldingen'].append(f"Niet overschreven: datum inlevering "
+                                              f"({_toon(bestaand['Datum_inlevering'])}).")
+                    elif bestaand['Datum_inlevering'] is None:
+                        r['meldingen'].append(f"Heeft dit toestel al sinds {_toon(bestaand['Datum_toekenning'])}.")
+                    else:
+                        r['meldingen'].append("Deze toekenning staat al in PIVOT.")
+
+            # --- nieuwe toekenning ---
+            else:
+                begin = begin or vandaag
+                sluit = None
+                botsingen = [e for e in lijst if _overlapt(begin, einde, e['Datum_toekenning'], e['Datum_inlevering'])]
+                if (overschrijven and not einde and len(botsingen) == 1 and botsingen[0]['Datum_inlevering'] is None
+                        and botsingen[0]['Datum_toekenning'] <= begin and botsingen[0]['rij'] is None):
+                    sluit = botsingen.pop()
+                for e in botsingen:
+                    waar = f"op rij {e['rij']}" if e['rij'] else "in PIVOT"
+                    if e['Datum_inlevering'] is None:
+                        tip = '' if overschrijven or e['rij'] else ' Kies "Vorige toekenning afsluiten" of lever het eerst in.'
+                        fouten.append(f"Het toestel staat {waar} sinds {_toon(e['Datum_toekenning'])} bij leerling "
+                                      f"{e['Internnummer']}.{tip}")
+                    else:
+                        fouten.append(f"Het toestel was {waar} van {_toon(e['Datum_toekenning'])} tot "
+                                      f"{_toon(e['Datum_inlevering'])} bij leerling {e['Internnummer']}.")
+                    break
+                if not fouten:
+                    r['actie'] = 'nieuw'
+                    r['waarden'] = {'Internnummer': intern, 'Toestel_ID': tid,
+                                    'Status': 1 if int(toestel.get('Geblokkeerd') or 0) else 0,
+                                    'Datum_toekenning': begin, 'Datum_inlevering': einde}
+                    if not v['datum_toekenning']:
+                        r['wijzigingen'].append(f"Datum toekenning: vandaag ({_toon(begin)})")
+                    if sluit:
+                        r['sluit'] = {'eigendom_id': sluit['Eigendom_ID'], 'datum': begin}
+                        r['wijzigingen'].append(f"Vorige toekenning bij leerling {sluit['Internnummer']} afgesloten "
+                                                f"op {_toon(begin)}")
+                        actief_per_leerling.get(sluit['Internnummer'], set()).discard(tid)
+                        sluit['Datum_inlevering'] = begin
+                        sluit['rij'] = r['rij']
+                    if not einde:
+                        andere = [toestel_op_id[t] for t in actief_per_leerling.get(intern, ()) if t in toestel_op_id]
+                        if andere:
+                            r['meldingen'].append("Heeft ook nog toestel " +
+                                                  ', '.join(_toestel_label(t) for t in andere) + '.')
+                        actief_per_leerling.setdefault(intern, set()).add(tid)
+                        if toestel['Status'] != 'Uitgeleend':
+                            r['toestel_status'] = 'Uitgeleend'
+                            r['wijzigingen'].append(f"Toestel: {toestel['Status'] or '–'} → Uitgeleend")
+                            toestel['Status'] = 'Uitgeleend'
+                    lijst.append({'Eigendom_ID': None, 'Internnummer': intern, 'Toestel_ID': tid,
+                                  'Datum_toekenning': begin, 'Datum_inlevering': einde, 'rij': r['rij']})
+
+            if not fouten and leerling.get('Vertrokken_op'):
+                r['meldingen'].append(f"Let op: leerling vertrokken op {_toon(_d(leerling['Vertrokken_op']))}.")
+
+        if fouten:
+            r['actie'] = 'fout'
+            r['meldingen'] = fouten
+            r['wijzigingen'] = []
+            r['waarden'], r['sluit'], r['toestel_status'], r['eigendom_id'] = {}, None, None, None
+        tellers[r['actie']] += 1
+        uit.append(r)
+
+    return {'rijen': uit, 'tellers': tellers, 'modus': 'overschrijven' if overschrijven else 'aanvullen'}
+
+
+# ---------------------------------------------------------------------------
+# Uitvoeren
+# ---------------------------------------------------------------------------
+def voer_uit(rijen, modus, gebruiker=None):
+    """Maakt het plan opnieuw en voert 'nieuw' en 'bijwerken' uit in één transactie.
+    Geeft (plan, aantal_nieuw, aantal_bijgewerkt). Bij een databasefout wordt niets bewaard."""
+    plan = maak_plan(rijen, modus)
+    conn = get_db_connection()
+    cur = conn.cursor()
+    nieuw = bijgewerkt = 0
+    try:
+        conn.start_transaction()
+        for r in plan['rijen']:
+            if r['actie'] not in ('nieuw', 'bijwerken'):
+                continue
+            if r['sluit']:
+                cur.execute("UPDATE Eigendom SET Datum_inlevering = %s WHERE Eigendom_ID = %s",
+                            (r['sluit']['datum'], r['sluit']['eigendom_id']))
+            w = r['waarden']
+            if r['actie'] == 'nieuw':
+                kolommen = list(w.keys())
+                cur.execute(f"INSERT INTO Eigendom ({', '.join(kolommen)}) "
+                            f"VALUES ({', '.join(['%s'] * len(kolommen))})", tuple(w[k] for k in kolommen))
+                nieuw += 1
+            else:
+                cur.execute("UPDATE Eigendom SET Datum_inlevering = %s WHERE Eigendom_ID = %s",
+                            (w['Datum_inlevering'], r['eigendom_id']))
+                bijgewerkt += 1
+            if r['toestel_status']:
+                cur.execute("UPDATE Toestellen SET Status = %s WHERE Toestel_ID = %s",
+                            (r['toestel_status'], r['toestel_id']))
+        conn.commit()
+    except Exception:
+        conn.rollback()
+        raise
+    finally:
+        cur.close()
+        conn.close()
+    return plan, nieuw, bijgewerkt

templates/dashboard.html

@@ -210,7 +210,7 @@
     {% endif %}
 </div>
 
-{% if mag('co2', 'koppelingen', 'gebruikersbeheer', 'nieuw_schooljaar', 'ip_whitelist', 'database_toestellen') or is_admin or is_superbeheerder %}
+{% if mag('co2', 'koppelingen', 'gebruikersbeheer', 'nieuw_schooljaar', 'ip_whitelist', 'database_toestellen', 'database_eigendom') or is_admin or is_superbeheerder %}
 <div class="dash-section-title admin">⚙️ Beheer</div>
 
 <div class="tile-grid">
@@ -255,11 +255,12 @@
     </section>
     {% endif %}
 
-    {% if mag('database_toestellen') %}
+    {% if mag('database_toestellen', 'database_eigendom') %}
     <section class="tile tile-admin">
         <div class="tile-head"><span class="tile-icon">🗄️</span><h3>Database</h3></div>
         <ul class="tile-links">
-            <li><a href="{{ url_for('database_beheer.toestellen') }}">💻 Toestellen importeren (Excel)</a></li>
+            {% if mag('database_toestellen') %}<li><a href="{{ url_for('database_beheer.toestellen') }}">💻 Toestellen importeren (Excel)</a></li>{% endif %}
+            {% if mag('database_eigendom') %}<li><a href="{{ url_for('database_beheer.eigendom') }}">🎒 Eigendom importeren (Excel)</a></li>{% endif %}
         </ul>
     </section>
     {% endif %}

templates/database_eigendom.html

@@ -0,0 +1,162 @@
+{% extends "base.html" %}
+
+{% block title %}Eigendom importeren{% endblock %}
+{% block page_title %}Database › Eigendom{% endblock %}
+{% block loader_text %}Bestand controleren...{% endblock %}
+
+{% block extra_css %}
+<style>
+    .stappen { display: grid; grid-template-columns: 1fr 1fr; gap: 20px; align-items: start; }
+    .stappen > .card + .card { margin-top: 0; }
+    @media (max-width: 860px) { .stappen { grid-template-columns: 1fr; } }
+    .stap-nr { display: inline-grid; place-items: center; width: 26px; height: 26px; border-radius: 50%;
+               background: var(--primary-color); color: #fff; font-size: 14px; margin-right: 6px; }
+    .kolom-lijst { margin: 10px 0 0; padding: 0; list-style: none; font-size: 14px; }
+    .kolom-lijst li { padding: 6px 0; border-bottom: 1px solid var(--border); }
+    .kolom-lijst li:last-child { border-bottom: 0; }
+    .kolom-lijst strong { display: inline-block; min-width: 120px; }
+    .modus-keuze { display: grid; gap: 8px; margin: 4px 0 16px; }
+    .modus-keuze label { display: flex; gap: 10px; align-items: flex-start; padding: 10px 12px; border: 1px solid var(--border);
+                         border-radius: var(--radius-sm); font-weight: 500; cursor: pointer; background: var(--surface-muted); }
+    .modus-keuze label:has(input:checked) { border-color: var(--primary-color); background: var(--primary-light); }
+    .modus-keuze input { margin-top: 2px; flex-shrink: 0; }
+    .modus-keuze small { display: block; color: var(--text-muted); font-weight: 400; }
+    .tellers { display: flex; flex-wrap: wrap; gap: 10px; margin: 6px 0 14px; }
+    .teller-knop { all: unset; cursor: pointer; display: inline-flex; gap: 8px; align-items: center; padding: 8px 14px;
+                   border-radius: var(--radius); border: 1px solid var(--border); background: #fff; font-weight: 600; font-size: 14px; }
+    .teller-knop.actief { border-color: var(--primary-color); box-shadow: 0 0 0 2px rgba(30,111,165,.18); }
+    .teller-knop .n { font-size: 18px; }
+    tr.rij-fout td { background: #fff7f7; }
+    td.melding { font-size: 13px; color: var(--text-muted); min-width: 220px; }
+    td.melding .fout { color: var(--danger-text); }
+    td.melding .wijz { color: var(--text); }
+    .bevestig { display: flex; flex-wrap: wrap; gap: 10px; align-items: center; justify-content: space-between; margin-top: 16px; }
+</style>
+{% endblock %}
+
+{% block content %}
+{% if not plan %}
+<div class="stappen">
+    <div class="card">
+        <h2 class="card-title"><span class="stap-nr">1</span>Sjabloon downloaden</h2>
+        <p class="muted">Vul het sjabloon in met één toekenning (leerling + toestel) per rij. Leerling en toestel moeten al
+            in PIVOT staan{% if mag('database_toestellen') %}; nieuwe toestellen voeg je eerst toe via
+            <a href="{{ url_for('database_beheer.toestellen') }}">Toestellen importeren</a>{% endif %}.</p>
+        <a class="btn" href="{{ url_for('database_beheer.eigendom_sjabloon') }}">⬇️ Sjabloon downloaden (.xlsx)</a>
+        <ul class="kolom-lijst">
+            {% for k in kolommen %}
+            <li><strong>{{ k[1] }}</strong> <span class="muted">{{ k[3] }}</span></li>
+            {% endfor %}
+        </ul>
+    </div>
+
+    <div class="card">
+        <h2 class="card-title"><span class="stap-nr">2</span>Excel uploaden</h2>
+        <p class="muted">Je krijgt eerst een voorbeeld te zien. Er wordt pas iets bewaard als je bevestigt.
+            {% if aantal is not none %}Er zijn nu <strong>{{ aantal }}</strong> toestellen uitgeleend.{% endif %}</p>
+        <form method="POST" action="{{ url_for('database_beheer.eigendom_controleren') }}" enctype="multipart/form-data"
+              onsubmit="showLoader('Bestand controleren...')">
+            <div class="field">
+                <label for="bestand">Excel-bestand (.xlsx)</label>
+                <input type="file" id="bestand" name="bestand" accept=".xlsx,.xlsm,application/vnd.openxmlformats-officedocument.spreadsheetml.sheet" required>
+                <div class="field-hint">Max. {{ max_rijen }} rijen. Het tabblad <em>Eigendom</em> wordt gelezen (of anders het eerste tabblad).</div>
+            </div>
+            <label>Toestel staat nu bij een andere leerling</label>
+            <div class="modus-keuze">
+                <label><input type="radio" name="modus" value="aanvullen" checked>
+                    <span>Als fout tonen <small>De rij wordt overgeslagen; de huidige toekenning blijft staan. (Aanbevolen)</small></span></label>
+                <label><input type="radio" name="modus" value="overschrijven">
+                    <span>Vorige toekenning afsluiten <small>Het toestel wordt bij de vorige leerling ingeleverd op de datum toekenning uit Excel. Ook een andere datum inlevering wordt dan overschreven.</small></span></label>
+            </div>
+            <button type="submit">🔎 Controleren</button>
+        </form>
+    </div>
+</div>
+
+{% else %}
+{% set t = plan.tellers %}
+<div class="card">
+    <div class="page-head">
+        <h2>🔎 Voorbeeld: {{ bestandsnaam }}</h2>
+        <a class="btn btn-light" href="{{ url_for('database_beheer.eigendom') }}">↩️ Ander bestand</a>
+    </div>
+    <p class="muted" style="margin-top:-8px">Nog niets bewaard. Toestel bij een andere leerling:
+        <strong>{{ 'vorige toekenning afsluiten' if plan.modus == 'overschrijven' else 'als fout tonen' }}</strong>.</p>
+    {% if onbekend %}
+    <div class="alert alert-warning">Deze kolommen worden genegeerd: {{ onbekend|join(', ') }}.</div>
+    {% endif %}
+
+    <div class="tellers" role="group" aria-label="Filter">
+        <button type="button" class="teller-knop actief" data-filter="">Alle <span class="n">{{ plan.rijen|length }}</span></button>
+        <button type="button" class="teller-knop" data-filter="nieuw"><span class="badge badge-success">➕ Nieuw</span> <span class="n">{{ t.nieuw }}</span></button>
+        <button type="button" class="teller-knop" data-filter="bijwerken"><span class="badge badge-info">✏️ Bijwerken</span> <span class="n">{{ t.bijwerken }}</span></button>
+        <button type="button" class="teller-knop" data-filter="ongewijzigd"><span class="badge">= Ongewijzigd</span> <span class="n">{{ t.ongewijzigd }}</span></button>
+        <button type="button" class="teller-knop" data-filter="fout"><span class="badge badge-danger">⚠️ Fout</span> <span class="n">{{ t.fout }}</span></button>
+    </div>
+
+    <div class="table-wrap">
+        <table id="voorbeeld">
+            <thead>
+                <tr>
+                    <th>Rij</th><th>Actie</th>
+                    {% for k in kolommen %}<th>{{ k[1] }}</th>{% endfor %}
+                    <th>Wat gebeurt er</th>
+                </tr>
+            </thead>
+            <tbody>
+            {% for r in plan.rijen %}
+                <tr data-actie="{{ r.actie }}" class="{{ 'rij-fout' if r.actie == 'fout' }}">
+                    <td class="muted">{{ r.rij }}</td>
+                    <td>
+                        {% if r.actie == 'nieuw' %}<span class="badge badge-success">➕ Nieuw</span>
+                        {% elif r.actie == 'bijwerken' %}<span class="badge badge-info">✏️ Bijwerken</span>
+                        {% elif r.actie == 'ongewijzigd' %}<span class="badge">= Ongewijzigd</span>
+                        {% else %}<span class="badge badge-danger">⚠️ Fout</span>{% endif %}
+                    </td>
+                    {% for k in kolommen %}<td class="{{ 'mono' if k[0] in ('leerling', 'toestel') }}">{{ r.invoer[k[0]] }}</td>{% endfor %}
+                    <td class="melding">
+                        {% if r.actie == 'fout' %}{% for m in r.meldingen %}<div class="fout">{{ m }}</div>{% endfor %}
+                        {% else %}
+                            {% if r.label %}<div>{{ r.label }}</div>{% endif %}
+                            {% if r.actie == 'nieuw' %}<div class="wijz">Wordt toegevoegd{{ ' als geschiedenis' if r.invoer.datum_inlevering }}.</div>{% endif %}
+                            {% for w in r.wijzigingen %}<div class="wijz">{{ w }}</div>{% endfor %}
+                            {% for m in r.meldingen %}<div>{{ m }}</div>{% endfor %}
+                        {% endif %}
+                    </td>
+                </tr>
+            {% endfor %}
+            </tbody>
+        </table>
+    </div>
+
+    {% set te_doen = t.nieuw + t.bijwerken %}
+    <form method="POST" action="{{ url_for('database_beheer.eigendom_importeren') }}" class="bevestig"
+          onsubmit="return confirm({{ ('Nu ' ~ t.nieuw ~ ' toekenning(en) toevoegen en ' ~ t.bijwerken ~ ' bijwerken?' ~ (' De ' ~ t.fout ~ ' rij(en) met een fout worden overgeslagen.' if t.fout else ''))|tojson|forceescape }}) && (showLoader('Importeren...'), true)">
+        <input type="hidden" name="modus" value="{{ plan.modus }}">
+        <input type="hidden" name="rijen" value="{{ rijen_json }}">
+        <span class="muted">
+            {% if te_doen %}{{ t.nieuw }} nieuw en {{ t.bijwerken }} bij te werken toekenning(en).
+                {% if t.fout %}Rijen met een fout worden overgeslagen; verbeter ze in Excel en upload opnieuw.{% endif %}
+            {% else %}Er is niets te importeren.{% endif %}
+        </span>
+        <span style="display:flex;gap:8px">
+            <a class="btn btn-light" href="{{ url_for('database_beheer.eigendom') }}">Annuleren</a>
+            <button type="submit" class="btn-success" {{ 'disabled' if not te_doen }}>✅ Importeren</button>
+        </span>
+    </form>
+</div>
+{% endif %}
+{% endblock %}
+
+{% block scripts %}
+<script>
+    // Voorbeeld filteren op actie (nieuw / bijwerken / ongewijzigd / fout)
+    document.querySelectorAll('.teller-knop').forEach(knop => knop.addEventListener('click', () => {
+        document.querySelectorAll('.teller-knop').forEach(k => k.classList.toggle('actief', k === knop));
+        const f = knop.dataset.filter;
+        document.querySelectorAll('#voorbeeld tbody tr').forEach(tr => {
+            tr.style.display = !f || tr.dataset.actie === f ? '' : 'none';
+        });
+    }));
+</script>
+{% endblock %}