adminmpmcz/patchcenter
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 2 Wiki Activity

feat(snapshots): dropdown Intervenant alimente depuis users actifs (hors admin) au pageload

Browse Source 
- Router: charge la liste des users (is_active=true AND role <> 'admin')
  depuis la table users + passe au contexte
- Template: dropdown rempli en Jinja avec username + (toi) si current user + display_name
  si different. Si user connecte non present en BDD (cas rare), ajoute une entree
  en bonus
- JS: supprime rebuildIntervenantDropdown (la liste reste figee, plus simple,
  predictible). Note conservee pour explication.
This commit is contained in:
Pierre & Lumière 2026-05-07 21:04:00 +02:00
parent 46b80474c2
commit cefddd2ea0
2 changed files with 18 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
app/routers/snapshots.py
View File

@ -49,12 +49,19 @@ async def snapshots_page(request: Request, db=Depends(get_db)):
vcenters = db.execute(text(
"SELECT id, name, endpoint FROM vcenters WHERE is_active = true ORDER BY name"
)).fetchall()
# Liste des intervenants disponibles : users actifs non-admin (cf table users)
intervenants = db.execute(text("""
SELECT username, display_name FROM users
WHERE is_active = true AND role <> 'admin'
ORDER BY username
""")).fetchall()
ctx = base_context(request, db, user)
ctx.update({
"app_name": APP_NAME,
"intervenant_default": intervenant,
"vcenters": vcenters,
"intervenants_list": intervenants,
"can_delete": _can_delete_snaps(perms),
})
return templates.TemplateResponse("snapshots.html", ctx)

37
app/templates/snapshots.html
View File

@ -58,9 +58,14 @@
<div class="col-span-3">
<label class="text-xs text-gray-500">Intervenant</label>
<select id="f-intervenant" class="w-full">
<option value="{{ intervenant_default }}" selected>{{ intervenant_default or '(toi)' }}</option>
{% for u in intervenants_list %}
<option value="{{ u.username }}"{% if u.username == intervenant_default %} selected{% endif %}>{{ u.username }}{% if u.username == intervenant_default %} (toi){% endif %}{% if u.display_name and u.display_name != u.username %} — {{ u.display_name }}{% endif %}</option>
{% endfor %}
{% if intervenant_default and intervenant_default not in intervenants_list|map(attribute='username')|list %}
<option value="{{ intervenant_default }}" selected>{{ intervenant_default }} (toi, hors liste)</option>
{% endif %}
</select>
<p class="text-xs text-gray-500 mt-1">Liste alimentée après chargement (auteurs détectés dans les snapshots PatchCenter).</p>
<p class="text-xs text-gray-500 mt-1">Users actifs (hors admins) — défaut = toi.</p>
</div>
<div class="col-span-3">
<label class="text-xs text-gray-500">Âge minimum (jours)</label>
@ -150,29 +155,10 @@
});
}
function rebuildIntervenantDropdown() {
// Liste des auteurs distincts des snapshots PatchCenter trouvés
const authors = Array.from(new Set(
allSnaps.filter(s => s.origin === 'patchcenter' && s.author).map(s => s.author)
)).sort((a,b) => a.localeCompare(b, 'fr', {sensitivity:'base'}));
// Conserve la sélection courante (ou défaut user connecté)
const current = fIntervenant.value || intervenantDefault;
let opts = '';
// Si user connecté pas dans la liste (= aucun snap encore), on l'ajoute en tête
const haveCurrent = authors.some(a => a.toLowerCase() === current.toLowerCase());
if (intervenantDefault && !authors.some(a => a.toLowerCase() === intervenantDefault.toLowerCase())) {
opts += `<option value="${intervenantDefault}">${intervenantDefault} (toi, aucun snap)</option>`;
}
for (const a of authors) {
const isYou = a.toLowerCase() === intervenantDefault.toLowerCase();
const sel = (a.toLowerCase() === current.toLowerCase()) ? ' selected' : '';
opts += `<option value="${a}"${sel}>${a}${isYou ? ' (toi)' : ''}</option>`;
}
if (!opts) opts = '<option value="">(aucun snap PatchCenter trouvé)</option>';
fIntervenant.innerHTML = opts;
// Re-sélectionne current ou défaut si présent
if (haveCurrent) fIntervenant.value = current;
}
// Note: la liste des intervenants est désormais figée au pageload (cf template,
// alimentée depuis la table users actifs hors admin). On ne la reconstruit plus
// dynamiquement après scan vCenter — un user qui aurait fait un snapshot mais
// serait inactif/admin ne sera pas filtrable nominativement, tant pis pour ce cas.
function fmtDateFR(iso) {
// ISO 8601 -> 'jj/mm/aaaa HH:MM' (heure locale du navigateur)
@ -296,7 +282,6 @@
(errs ? ` <span class="text-cyber-yellow">⚠ ${escapeHTML(errs)}</span>` : '');
status.dataset.baseMsg = baseMsg;
status.innerHTML = baseMsg;
rebuildIntervenantDropdown();
render();
} catch (e) {
status.innerHTML = '<span class="text-cyber-red">Erreur réseau : ' + escapeHTML(String(e)) + '</span>';