diff --git a/TODO.md b/TODO.md index 39acae9..7f6e3d9 100644 --- a/TODO.md +++ b/TODO.md @@ -39,6 +39,14 @@ - [x] Update `root` directive in `nginx/posterg.conf` - [x] Update `STORAGE_ROOT` production path in `app/bootstrap.php` +## Centralise Form Templates + +- [x] Extract shared fieldset partials: `fieldset-tfe-info.php`, `fieldset-academic.php`, `fieldset-files.php`, `fieldset-metadata.php`, `fieldset-licence-explanation.php` +- [x] Refactor `templates/admin/add.php` to use shared partials +- [x] Refactor `templates/admin/edit.php` to use shared partials (with edit-mode callable adapters) +- [x] Refactor `partage/index.php` `renderShareLinkForm()` to use shared partials +- [x] Add TODO comments in `partage/index.php` for student-facing explanations (intro block, per-fieldset notes, email note) + ## File Display in Forms & Recaps - [x] Add live file preview to `file-field.php` partial (`data-preview` attribute + `.file-preview-list` container) diff --git a/app/public/partage/index.php b/app/public/partage/index.php index 9d5aa34..7b0f6ce 100644 --- a/app/public/partage/index.php +++ b/app/public/partage/index.php @@ -182,17 +182,15 @@ function renderShareLinkForm(string $slug, array $link): void die('Erreur lors du chargement du formulaire.'); } - $formData = $_SESSION['form_data_share_' . $slug] ?? []; + $formData = $_SESSION['form_data_share_' . $slug] ?? []; unset($_SESSION['form_data_share_' . $slug]); // Determine allowed objet values for this link - $siteSettings = Database::getInstance()->getAllSettings(); + $siteSettings = Database::getInstance()->getAllSettings(); $objetRestriction = $link['objet_restriction'] ?? null; if ($objetRestriction !== null) { - // Link is locked to one type — always show only that $allowedObjet = [$objetRestriction]; } else { - // Build from enabled site settings $allowedObjet = ['tfe']; if (($siteSettings['objet_these_enabled'] ?? '1') === '1') $allowedObjet[] = 'thèse'; if (($siteSettings['objet_frart_enabled'] ?? '1') === '1') $allowedObjet[] = 'frart'; @@ -205,10 +203,13 @@ function renderShareLinkForm(string $slug, array $link): void } $shareCsrfToken = $_SESSION[$shareCsrfKey]; - $pageTitle = 'Soumettre un TFE'; - - // Determine if previously verified by password + $pageTitle = 'Soumettre un TFE'; $isVerified = !empty($_SESSION['share_verified_' . $slug]); + + // Build old()-compatible callable from $formData (share forms use the array variant). + $shareOldFn = fn(string $key, string $default = '') => old($formData, $key, $default); + // No autofocus in the share form — identity function. + $shareWithAutofocusFn = fn(string $field, array $attrs = []) => $attrs; ?> @@ -242,187 +243,97 @@ function renderShareLinkForm(string $slug, array $link): void + +

* Champs obligatoires

-
- Informations du TFE - - 1): ?> -
- -
- - - -
-
- - - - - - - 'name']; include APP_ROOT . '/templates/partials/form/text-field.php'; ?> - 'email']; include APP_ROOT . '/templates/partials/form/text-field.php'; ?> - -
- - Si cette case est cochée, votre contact apparaîtra sur la page publique de votre TFE. -
- -
- - -
-
+ -
- Composition du jury - - - 'jury_promoteur_ext', 'label' => 'Externe à l\'erg', 'checked' => !empty($formData['jury_promoteur_ext'])]; include APP_ROOT . '/templates/partials/form/text-field.php'; ?> - -
-

Lecteurs·rices (optionnel) :

- -
- "> - -
- -
-
+ $name, + 'is_external' => !empty($formData["jury_lecteurs_ext:$i"]) ? 1 : 0, + ]; + } + } + // TODO: Add a note explaining the jury composition to students: + // who counts as external, what role each member plays, and + // whether this information will be publicly visible. + require APP_ROOT . '/templates/partials/form/jury-fieldset.php'; + ?> -
- Cadre académique - - 2000, 'max' => date('Y') + 1]; - include APP_ROOT . '/templates/partials/form/text-field.php'; - ?> - - - - - - - - - -
+ -
- Fichiers - - - - -
+ -
- Métadonnées complémentaires + - - - - - - - $at['id'], 'name' => $at['name']]; - }, $enabledAccessTypes); - $defaultAccessType = 2; - $selectedAccessType = isset($formData['access_type_id']) - ? (int)$formData['access_type_id'] - : $defaultAccessType; - $name = 'access_type_id'; - $label = 'Visibilité / Accès :'; - $options = $accessOptions; - $selected = $selectedAccessType; - $placeholder = null; - $required = true; - $attrs = []; - include APP_ROOT . '/templates/partials/form/select-field.php'; - ?> -
- - -
- Degrés d'ouverture et licences - -
-

Je veux que mon TFE soit disponible sous les conditions suivantes :

- -
-

🔓 Libre

-

Mon TFE est en libre accès à tout le monde sur la plateforme des TFE ainsi que dans la bibliothèque de l'erg. Je suis conscient·e des responsabilités et obligations légales qui viennent avec une diffusion externe – et acquiesce avoir lu la documentation prévue à cet effet par l'erg, ainsi qu'avoir discuté des enjeux d'une publication avec l'équipe pédagogique. J'accepte de partager mes droits de diffusion avec l'erg, ce uniquement dans le cadre d'une diffusion sur la plateforme xamxam.

-
    -
    • -
    • -
-

Au moins une des deux cases doit être cochée pour le degré Libre.

-
- -
-

🔒 Interne

-

Mon TFE et ma note d'intention ne sont accessibles que sur place en physique ainsi que sur la plateforme xamxam par la communauté erg. Une note descriptive est disponible sur le site à toustes. J'autorise une (ré-)utilisation et diffusion dans un contexte académique et didactique au sein de l'erg.

-

La diffusion limitée est protégée par le cadre académique/didactique, le travail pourrait donc être diffusé en interne et être cité par d'autres étudiant·es sans implications légales pour l'auteur·ice ni pour l'école.

-
    -
    • -
    • -
-

Au moins une des deux cases doit être cochée.

-
- -
-

🚫 Interdit

-

Mon TFE n'est pas disponible en physique ni sur le site. Une note descriptive est disponible sur le site.

-
-
- -
-

Généralités

-
    -
  • L'auteur·ice peut décider entre trois degrés de partage de son travail : libre, interne, interdit.
    • -
  • L'auteur·ice peut, à tout moment, décider de restreindre le degré d'accès à son travail. Il ne peut néanmoins pas l'ouvrir davantage.
    • -
  • Le choix effectué dans ce formulaire sera d'application une semaine après la soutenance orale de l'auteur·ice. Celui-ci peut donc décider de restreindre ce choix avant sa publication (mais pas l'ouvrir).
    • -
  • L'erg se réserve le droit de restreindre le degré d'ouverture du TFE – ce en accord avec le règlement.
    • -
  • Dans tous les cas, l'auteur·ice garde les droits d'auteurs, de diffusion, d'utilisation, etc. de son travail – sauf si la licence choisie restreindrait ses droits.
    • -
  • La diffusion « xamxam » est indépendante de la diffusion à la BAIU.
    • -
-
-
+ +
E-mail de confirmation - + +
diff --git a/app/templates/admin/add.php b/app/templates/admin/add.php index e9eca13..4503d51 100644 --- a/app/templates/admin/add.php +++ b/app/templates/admin/add.php @@ -8,95 +8,39 @@ "> -
- Informations du TFE - - - - 'name']); include APP_ROOT . '/templates/partials/form/text-field.php'; ?> - 'email']; include APP_ROOT . '/templates/partials/form/text-field.php'; ?> - -
- - Si cette case est cochée, votre contact apparaîtra sur la page publique de votre TFE. -
- -
- - -
-
+ -
- Cadre académique - - 2000, 'max' => date('Y') + 1]); - include APP_ROOT . '/templates/partials/form/text-field.php'; - ?> - - - - - - - - - -
+ -
- Fichiers - - - - -
+ -
- Métadonnées complémentaires + - + + - - - - - $at['id'], 'name' => $at['name']]; - }, $enabledAccessTypes); - $defaultAccessType = 2; - $selectedAccessType = isset($formData['access_type_id']) - ? (int)$formData['access_type_id'] - : $defaultAccessType; - $name = 'access_type_id'; - $label = 'Visibilité / Accès :'; - $options = $accessOptions; - $selected = $selectedAccessType; - $placeholder = null; - $required = true; - $attrs = []; - include APP_ROOT . '/templates/partials/form/select-field.php'; - ?> -
- - +
E-mail de confirmation diff --git a/app/templates/admin/edit.php b/app/templates/admin/edit.php index 1b651dd..04dfafd 100644 --- a/app/templates/admin/edit.php +++ b/app/templates/admin/edit.php @@ -5,97 +5,77 @@ - 'name'], $autofocusField === 'auteurice' ? ['autofocus' => true] : []); include APP_ROOT . '/templates/partials/form/text-field.php'; ?> - 'email']; include APP_ROOT . '/templates/partials/form/text-field.php'; ?> - - -
- - Si cette case est cochée, le contact apparaît sur la page publique du TFE. -
- + true] : []; - include APP_ROOT . '/templates/partials/form/text-field.php'; + // Build a merged $formData that includes contact_public for the fieldset partial. + $formData = array_merge($formData ?? [], ['contact_public' => $currentAuthorShowContact ?? false]); + // Build old()-compatible callable from the already-loaded $thesis data. + $editOldFn = function (string $key, string $default = '') use ($thesis, $formData, $currentAuthorEmail) { + if (!empty($formData[$key])) return htmlspecialchars($formData[$key]); + $map = [ + 'titre' => htmlspecialchars($thesis['title']), + 'subtitle' => htmlspecialchars($thesis['subtitle'] ?? ''), + 'auteurice'=> htmlspecialchars($thesis['authors'] ?? ''), + 'mail' => htmlspecialchars($currentAuthorEmail ?? ''), + 'synopsis' => htmlspecialchars($thesis['synopsis'] ?? ''), + ]; + return $map[$key] ?? $default; + }; + $editWithAutofocusFn = function (string $field, array $attrs = []) use ($autofocusField) { + if ($autofocusField === $field) $attrs['autofocus'] = true; + return $attrs; + }; + $allowedObjet = []; + $synopsisExtra = ''; + $oldFn = $editOldFn; + $withAutofocusFn = $editWithAutofocusFn; + include APP_ROOT . '/templates/partials/form/fieldset-tfe-info.php'; + // Restore $formData to original for downstream partials. + $formData = $_SESSION['form_data'] ?? []; ?> - - - - - - - + - + $at['id'], 'name' => $label]; - }, $accessTypes); - $name = 'access_type_id'; $label = 'Visibilité / Accès :'; $options = $accessOptions; $selected = $currentAccessTypeId; $placeholder = '- Non défini -'; - include APP_ROOT . '/templates/partials/form/select-field.php'; + $editFormData = [ + 'année' => $thesis['year'], + 'orientation' => $thesis['orientation'], + 'ap' => $thesis['ap_program'], + 'finality' => $thesis['finality_type'], + 'languages' => $currentLanguages, + 'formats' => $currentFormats, + 'tag' => $thesis['keywords'] ?? '', + ]; + $editAcademicOldFn = function (string $key, string $default = '') use ($editFormData) { + return isset($editFormData[$key]) && !is_array($editFormData[$key]) + ? htmlspecialchars((string)$editFormData[$key]) + : $default; + }; + $oldFn = $editAcademicOldFn; + $withAutofocusFn = $editWithAutofocusFn; + $formData = $editFormData; + include APP_ROOT . '/templates/partials/form/fieldset-academic.php'; ?> - -
- -
- - Visible publiquement pour les TFE Interne ou Interdit. Max 1 500 caractères. -
-
- - - - true] : []; include APP_ROOT . '/templates/partials/form/text-field.php'; ?> - - - -
- - -
- - - - - - - - - - - - +
Fichiers @@ -175,18 +155,53 @@
- - -
- - -
+ + $currentLicenseId, + 'duration_info' => $thesis['file_size_info'] ?? '', + 'lien' => $thesis['baiu_link'] ?? '', + 'access_type_id' => $currentAccessTypeId, + ]; + $editMetaOldFn = function (string $key, string $default = '') use ($editMetaFormData) { + return isset($editMetaFormData[$key]) + ? htmlspecialchars((string)$editMetaFormData[$key]) + : $default; + }; + $oldFn = $editMetaOldFn; + $withAutofocusFn = $editWithAutofocusFn; + $formData = $editMetaFormData; + $defaultAccessTypeId = $currentAccessTypeId; + $showDescription = true; + include APP_ROOT . '/templates/partials/form/fieldset-metadata.php'; + ?> + + +
+ Note contextuelle +
+ +
+ + Visible publiquement pour les TFE Interne ou Interdit. Max 1 500 caractères. +
+
+
+ + +
+ Publication +
+ +
+
diff --git a/app/templates/partials/form/fieldset-academic.php b/app/templates/partials/form/fieldset-academic.php new file mode 100644 index 0000000..44d7f87 --- /dev/null +++ b/app/templates/partials/form/fieldset-academic.php @@ -0,0 +1,62 @@ + $d); +$withAutofocusFn = $withAutofocusFn ?? fn($field, $attrs = []) => $attrs; +$formData = $formData ?? []; +?> +
+ Cadre académique + + 2000, 'max' => date('Y') + 1]); + include APP_ROOT . '/templates/partials/form/text-field.php'; + ?> + + + + + + + + + +
+ +
+ Fichiers + + + + +
diff --git a/app/templates/partials/form/fieldset-licence-explanation.php b/app/templates/partials/form/fieldset-licence-explanation.php new file mode 100644 index 0000000..4f9f219 --- /dev/null +++ b/app/templates/partials/form/fieldset-licence-explanation.php @@ -0,0 +1,61 @@ + +
+ Degrés d'ouverture et licences + +
+

Je veux que mon TFE soit disponible sous les conditions suivantes :

+ +
+

🔓 Libre

+

Mon TFE est en libre accès à tout le monde sur la plateforme des TFE ainsi que dans la bibliothèque de l'erg. Je suis conscient·e des responsabilités et obligations légales qui viennent avec une diffusion externe – et acquiesce avoir lu la documentation prévue à cet effet par l'erg, ainsi qu'avoir discuté des enjeux d'une publication avec l'équipe pédagogique. J'accepte de partager mes droits de diffusion avec l'erg, ce uniquement dans le cadre d'une diffusion sur la plateforme xamxam.

+
    +
    • +
    • +
+

Au moins une des deux cases doit être cochée pour le degré Libre.

+
+ +
+

🔒 Interne

+

Mon TFE et ma note d'intention ne sont accessibles que sur place en physique ainsi que sur la plateforme xamxam par la communauté erg. Une note descriptive est disponible sur le site à toustes. J'autorise une (ré-)utilisation et diffusion dans un contexte académique et didactique au sein de l'erg.

+

La diffusion limitée est protégée par le cadre académique/didactique, le travail pourrait donc être diffusé en interne et être cité par d'autres étudiant·es sans implications légales pour l'auteur·ice ni pour l'école.

+
    +
    • +
    • +
+

Au moins une des deux cases doit être cochée.

+
+ +
+

🚫 Interdit

+

Mon TFE n'est pas disponible en physique ni sur le site. Une note descriptive est disponible sur le site.

+
+
+ +
+

Généralités

+
    +
  • L'auteur·ice peut décider entre trois degrés de partage de son travail : libre, interne, interdit.
    • +
  • L'auteur·ice peut, à tout moment, décider de restreindre le degré d'accès à son travail. Il ne peut néanmoins pas l'ouvrir davantage.
    • +
  • Le choix effectué dans ce formulaire sera d'application une semaine après la soutenance orale de l'auteur·ice. Celui-ci peut donc décider de restreindre ce choix avant sa publication (mais pas l'ouvrir).
    • +
  • L'erg se réserve le droit de restreindre le degré d'ouverture du TFE – ce en accord avec le règlement.
    • +
  • Dans tous les cas, l'auteur·ice garde les droits d'auteurs, de diffusion, d'utilisation, etc. de son travail – sauf si la licence choisie restreindrait ses droits.
    • +
  • La diffusion « xamxam » est indépendante de la diffusion à la BAIU.
    • +
+
+
+ $d); +$withAutofocusFn = $withAutofocusFn ?? fn($field, $attrs = []) => $attrs; +$formData = $formData ?? []; +$defaultAccessTypeId = $defaultAccessTypeId ?? 2; +$showDescription = $showDescription ?? false; + +$accessOptions = array_map(function ($at) use ($showDescription) { + $label = $at['name']; + if ($showDescription && !empty($at['description'])) { + $label .= ' — ' . $at['description']; + } + return ['id' => $at['id'], 'name' => $label]; +}, $enabledAccessTypes); + +$selectedAccessType = isset($formData['access_type_id']) + ? (int) $formData['access_type_id'] + : $defaultAccessTypeId; +?> +
+ Métadonnées complémentaires + + + + + + + + +
+ 1 a radio group + * is shown; when count === 1 a hidden input is emitted instead. + * Omit or pass [] to suppress the objet selector entirely. + * array $formData — raw form data array (used for contact_public checkbox state). + * string $synopsisExtra — optional HTML block injected after the synopsis label + * (use for student-facing explanations). + * bool $isAdminMode — when true the form uses admin-style helpers; default false. + */ + +$oldFn = $oldFn ?? (function_exists('old') ? 'old' : fn($k, $d = '') => $d); +$withAutofocusFn = $withAutofocusFn ?? fn($field, $attrs = []) => $attrs; +$allowedObjet = $allowedObjet ?? []; +$formData = $formData ?? []; +$synopsisExtra = $synopsisExtra ?? ''; +?> +
+ Informations du TFE + + + 1): ?> +
+ +
+ + + +
+
+ + + + + + + + 'name']); + include APP_ROOT . '/templates/partials/form/text-field.php'; + ?> + 'email']; + include APP_ROOT . '/templates/partials/form/text-field.php'; + ?> + +
+ + Si cette case est cochée, votre contact apparaîtra sur la page publique de votre TFE. +
+ +
+ + + +
+
+