Merge branch 'master' into 456-doc-generation-active-status

.changes/unreleased/Fixed-20251118-140559.yaml

@@ -1,7 +0,0 @@
-kind: Fixed
-body: |
-    Associate activity's creator as a participant by default, and retro-actively append the creator to each activity
-time: 2025-11-18T14:05:59.904993123+01:00
-custom:
-    Issue: "466"
-    SchemaChange: Add columns or tables

.changes/unreleased/Fixed-20251119-133324.yaml

@@ -0,0 +1,6 @@
+kind: Fixed
+body: Insert name of file as the document title when uploading
+time: 2025-11-19T13:33:24.778116633+01:00
+custom:
+    Issue: ""
+    SchemaChange: No schema change

.changes/unreleased/Fixed-20251119-134802.yaml

@@ -0,0 +1,6 @@
+kind: Fixed
+body: Add missing path paramater 'id' for editing multiple participations
+time: 2025-11-19T13:48:02.078949572+01:00
+custom:
+    Issue: ""
+    SchemaChange: No schema change

.changes/unreleased/UX-20251103-161910.yaml

@@ -0,0 +1,6 @@
+kind: UX
+body: Display whether doc generation template is active or not in admin and order templates alphabetically
+time: 2025-11-03T16:19:10.051947925+01:00
+custom:
+    Issue: "456"
+    SchemaChange: No schema change

.changes/unreleased/UX-20251119-153706.yaml

@@ -1,6 +0,0 @@
-kind: UX
-body: Alphabetically order userJobs and mainLocations within user creation form
-time: 2025-11-19T15:37:06.393470745+01:00
-custom:
-    Issue: "470"
-    SchemaChange: No schema change

.changes/unreleased/UX-20251124-151115.yaml

@@ -1,6 +0,0 @@
-kind: UX
-body: Change position and color of confirm parcours button
-time: 2025-11-24T15:11:15.960279853+01:00
-custom:
-    Issue: "437"
-    SchemaChange: No schema change

.changes/unreleased/UX-20251126-133910.yaml

@@ -1,6 +0,0 @@
-kind: UX
-body: Improve accessibility of event form
-time: 2025-11-26T13:39:10.462348982+01:00
-custom:
-    Issue: "474"
-    SchemaChange: No schema change

.changes/v4.8.1.md

@@ -1,6 +0,0 @@
-## v4.8.1 - 2025-11-20
-### Fixed
-* Insert name of file as the document title when uploading   
-* Add missing path paramater 'id' for editing multiple participations   
-* ([#471](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/471)) Hide the display of inactive user groups in the api
-   

CHANGELOG.md

@@ -6,13 +6,6 @@ adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html),
 and is generated by [Changie](https://github.com/miniscruff/changie).
 
 
-## v4.8.1 - 2025-11-20
-### Fixed
-* Insert name of file as the document title when uploading   
-* Add missing path paramater 'id' for editing multiple participations   
-* ([#471](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/471)) Hide the display of inactive user groups in the api
-   
-
 ## v4.8.0 - 2025-11-17
 ### Feature
 * ([#461](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/461)) Make a calendar item on the 'mes rendez-vous' page clickable. Clicking will navigate to the edit page of the calendar item.   

src/Bundle/ChillActivityBundle/Controller/ActivityController.php

@@ -382,7 +382,6 @@ final class ActivityController extends AbstractController
 
         $entity = new Activity();
         $entity->setUser($this->security->getUser());
-        $entity->addUser($this->security->getUser());
 
         if ($person instanceof Person) {
             $entity->setPerson($person);

src/Bundle/ChillActivityBundle/migrations/Version20251118124241.php

@@ -1,50 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-/*
- * Chill is a software for social workers
- *
- * For the full copyright and license information, please view
- * the LICENSE file that was distributed with this source code.
- */
-
-namespace Chill\Migrations\Activity;
-
-use Doctrine\DBAL\Schema\Schema;
-use Doctrine\Migrations\AbstractMigration;
-
-/**
- * Migration fixing the automatic association of users to activities (exchanges).
- *
- * Originally, the user who created an exchange was not automatically associated
- * to it (the "TMS" column), which led to incomplete data and biased statistics.
- *
- * This migration:
- *  - retroactively associates the creator of each exchange to the corresponding
- *    activity;
- *  - flags these backfilled associations with a temporary column so it is clear
- *    they were added by this data correction and can be safely cleaned up later.
- */
-final class Version20251118124241 extends AbstractMigration
-{
-    public function getDescription(): string
-    {
-        return 'Insert the creator of activity into the activity_user table';
-    }
-
-    public function up(Schema $schema): void
-    {
-        $this->addSql('ALTER TABLE activity_user ADD COLUMN by_migration BOOL DEFAULT FALSE');
-        $this->addSql("COMMENT ON COLUMN activity_user.by_migration IS 'For backup purpose - can be safely deleted after a while. See migration \\Chill\\Migrations\\Activity\\Version20251118124241'");
-
-        $this->addSql('INSERT INTO activity_user (activity_id, user_id, by_migration)
-            SELECT id, user_id, true FROM activity
-            ON CONFLICT DO NOTHING');
-    }
-
-    public function down(Schema $schema): void
-    {
-        $this->addSql('ALTER TABLE activity_user DROP COLUMN by_migration');
-    }
-}

src/Bundle/ChillDocGeneratorBundle/Controller/AdminDocGeneratorTemplateController.php

@@ -87,7 +87,7 @@ class AdminDocGeneratorTemplateController extends CRUDController
      */
     protected function orderQuery(string $action, $query, Request $request, PaginatorInterface $paginator)
     {
-        return $query->addSelect('JSON_EXTRACT(e.name, :lang) AS HIDDEN name_lang')
+        return $query->addSelect('REPLACE(JSON_EXTRACT(e.name, :lang), \' \', \'~\') AS HIDDEN name_lang')
             ->setParameter('lang', $request->getLocale())
             ->addOrderBy('name_lang', 'ASC');
     }

src/Bundle/ChillDocGeneratorBundle/Resources/views/DocGeneratorTemplate/index.html.twig

@@ -25,12 +25,24 @@
                             <div class="item-bloc">
                                 <div class="item-row">
                                     <div class="item-col" style="flex-basis:100%;">
-                                        <h2>{{ entity.name|localize_translatable_string }}</h2>
+                                        <h2>{{ entity.name|localize_translatable_string }} </h2>
+                                        <p style="margin-left: 1rem;"><span class="badge bg-chill-gray">
+                                            {% if entity.active %}
+                                                {{ 'admin.active'|trans }}
+                                            {% else %}
+                                                {{ 'admin.not active'|trans }}
+                                            {% endif %}
+                                        </span></p>
                                     </div>
                                 </div>
                                 <div class="item-row">
                                     <p><span class="badge bg-chill-green-dark">{{ contextManager.getContextByKey(entity.context).name|trans }}</span></p>
                                 </div>
+{#                                <div class="item-row">#}
+{#                                    <div class="item-col" style="flex-basis:100%;">#}
+{##}
+{#                                    </div>#}
+{#                                </div>#}
                                 <div class="item-row">
                                     <div class="item-col"></div>
                                     <ul class="record_actions item-col flex-shrink-1">

src/Bundle/ChillDocGeneratorBundle/translations/messages.fr.yml

@@ -49,3 +49,7 @@ crud:
 
 
 Template file: Fichier modèle
+
+admin:
+    active: Actif
+    not active: Non-actif

src/Bundle/ChillEventBundle/Controller/ParticipationController.php

@@ -486,7 +486,7 @@ final class ParticipationController extends AbstractController
 
             return $this->redirectToRoute(
                 'chill_event__event_show',
-                ['id' => $event->getId()]
+                ['event_id' => $event->getId()]
             );
         }
 

src/Bundle/ChillEventBundle/Resources/views/Event/edit.html.twig

@@ -15,6 +15,7 @@
     {{ form_row(edit_form.circle) }}
     {{ form_row(edit_form.name) }}
     {{ form_row(edit_form.date) }}
+
     {{ form_row(edit_form.type, { label: "Event type" }) }}
     {{ form_row(edit_form.themes) }}
     {{ form_row(edit_form.moderator) }}

src/Bundle/ChillMainBundle/Controller/UserGroupApiController.php

@@ -12,12 +12,5 @@ declare(strict_types=1);
 namespace Chill\MainBundle\Controller;
 
 use Chill\MainBundle\CRUD\Controller\ApiController;
-use Symfony\Component\HttpFoundation\Request;
 
-class UserGroupApiController extends ApiController
-{
-    protected function customizeQuery(string $action, Request $request, $query): void
-    {
-        $query->andWhere('e.active = TRUE');
-    }
-}
+class UserGroupApiController extends ApiController {}

src/Bundle/ChillMainBundle/Form/Type/ChillDateTimeType.php

@@ -37,9 +37,4 @@ class ChillDateTimeType extends AbstractType
     {
         return DateTimeType::class;
     }
-
-    public function getBlockPrefix(): string
-    {
-        return 'chill_datetime';
-    }
 }

src/Bundle/ChillMainBundle/Form/UserType.php

@@ -18,7 +18,6 @@ use Chill\MainBundle\Entity\UserJob;
 use Chill\MainBundle\Form\Type\ChillDateType;
 use Chill\MainBundle\Form\Type\ChillPhoneNumberType;
 use Chill\MainBundle\Form\Type\PickCivilityType;
-use Chill\MainBundle\Repository\UserJobRepository;
 use Chill\MainBundle\Templating\TranslatableStringHelper;
 use Doctrine\ORM\EntityRepository;
 use Symfony\Bridge\Doctrine\Form\Type\EntityType;
@@ -37,7 +36,7 @@ use Symfony\Component\Validator\Constraints\Regex;
 
 class UserType extends AbstractType
 {
-    public function __construct(private readonly TranslatableStringHelper $translatableStringHelper, protected ParameterBagInterface $parameterBag, private readonly UserJobRepository $userJobRepository) {}
+    public function __construct(private readonly TranslatableStringHelper $translatableStringHelper, protected ParameterBagInterface $parameterBag) {}
 
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
@@ -81,7 +80,12 @@ class UserType extends AbstractType
                 'placeholder' => 'choose a job',
                 'class' => UserJob::class,
                 'choice_label' => fn (UserJob $c) => $this->translatableStringHelper->localize($c->getLabel()),
-                'choices' => $this->loadAndSortUserJobs(),
+                'query_builder' => static function (EntityRepository $er) {
+                    $qb = $er->createQueryBuilder('uj');
+                    $qb->where('uj.active = TRUE');
+
+                    return $qb;
+                },
             ])
             ->add('mainLocation', EntityType::class, [
                 'label' => 'Main location',
@@ -92,7 +96,6 @@ class UserType extends AbstractType
                 'query_builder' => static function (EntityRepository $er) {
                     $qb = $er->createQueryBuilder('l');
                     $qb->orderBy('l.locationType');
-                    $qb->orderBy('l.name', 'ASC');
                     $qb->where('l.availableForUsers = TRUE');
 
                     return $qb;
@@ -152,20 +155,6 @@ class UserType extends AbstractType
         }
     }
 
-    private function loadAndSortUserJobs(): array
-    {
-        $items = $this->userJobRepository->findBy(['active' => true]);
-
-        usort(
-            $items,
-            fn ($a, $b) => mb_strtolower((string) $this->translatableStringHelper->localize($a->getLabel()))
-            <=>
-            mb_strtolower((string) $this->translatableStringHelper->localize($b->getLabel()))
-        );
-
-        return $items;
-    }
-
     /**
      * @param OptionsResolverInterface $resolver
      */

src/Bundle/ChillMainBundle/Resources/views/Form/fields.html.twig

@@ -311,32 +311,3 @@
         </div>
     </div>
 {% endblock %}
-
-{% block chill_datetime_label %}
-    <label for="{{ form.date.vars.id }}" class="col-form-label col-sm-4 required">
-        {{ "Date"|trans }}
-    </label>
-{% endblock %}
-
-{% block chill_datetime_widget %}
-    <div class="col-sm-8 d-flex align-items-start gap-2">
-        {#date#}
-        {{ form_widget(form.date, {
-            attr: { class: 'form-control', style: 'flex: 1 1 auto;' }
-        }) }}
-        {#time#}
-        {{ form_widget(form.time, {
-            attr: {
-                class: 'form-select',
-                style: 'flex: 0 0 200px; max-width: 200px; white-space: nowrap; padding:0;'
-            }
-        }) }}
-    </div>
-{% endblock %}
-
-{% block chill_datetime_row %}
-    <div class="mb-3 row">
-        {{ block('chill_datetime_label') }}
-        {{ block('chill_datetime_widget') }}
-    </div>
-{% endblock %}

src/Bundle/ChillPersonBundle/Resources/public/vuejs/AccompanyingCourse/components/Confirm.vue

@@ -20,16 +20,16 @@
                     </ul>
                 </div>
                 <ul class="record_actions">
-                    <li>
-                        <a class="btn btn-delete" :href="deleteLink">
-                            {{ $t("confirm.delete") }}
-                        </a>
-                    </li>
                     <li>
                         <button class="btn btn-save" disabled>
                             {{ $t("confirm.ok") }}
                         </button>
                     </li>
+                    <li>
+                        <a class="btn btn-delete" :href="deleteLink">
+                            {{ $t("confirm.delete") }}
+                        </a>
+                    </li>
                 </ul>
             </div>
 
@@ -40,11 +40,6 @@
                     "
                 />
                 <ul class="record_actions">
-                    <li>
-                        <a class="btn btn-delete" :href="deleteLink">
-                            {{ $t("confirm.delete") }}
-                        </a>
-                    </li>
                     <li>
                         <button
                             class="btn btn-save"
@@ -53,6 +48,11 @@
                             {{ $t("confirm.ok") }}
                         </button>
                     </li>
+                    <li>
+                        <a class="btn btn-delete" :href="deleteLink">
+                            {{ $t("confirm.delete") }}
+                        </a>
+                    </li>
                 </ul>
             </div>
         </div>
@@ -112,7 +112,7 @@
                 </template>
                 <template #footer>
                     <button
-                        class="btn btn-save"
+                        class="btn btn-danger"
                         :disabled="disableConfirm"
                         @click="confirmCourse"
                     >