Merge branch 'hotfix/V8-3313' into develop

doc/fe_users.md

@@ -0,0 +1,20 @@
+# Frontend users
+
+Les frontend users sont les utilisateurs authentifiés du côté front.
+Un administrateur de site peut créer des pages et contenus réservées aux utilisateurs authentifiés.
+
+> Pour le fonctionnement de l'authentification, voir : [auth.md](auth.md)
+
+Lorsqu'un utilisateur opentalent, déjà connecté au logiciel, envoie une requête pour une page d'un site
+Typo3, le plugin ot_connect interroge l'API Opentalent pour vérifier si cet utilisateur est bien connecté.
+
+Si oui, un enregistrement est créé ou mis à jour dans la table 'fe_user' (le lien avec la DB Opentalent
+se fait via le champ fe_user.tx_opentalent_opentalentId). À noter que l'enregistrement du fe_user ne sera 
+mis à jour que si la dernière mise à jour remonte à plus de 300s.
+
+Ces fe_users seront affectés à des groupes 'Membres' (fe_group) correspondant à la ou les structures auxquelles 
+la personne a accès.
+
+Ces groupes sont associés via leur pid à un site.
+
+Un administrateur peut alors limiter l'accès à une page ou contenu aux membres de ce groupe.

doc/installation.md

@@ -53,19 +53,17 @@ Sauvegarder les uploads et les définitions des formulaires (de l'ordre de la ce
 
 ### Mettre à jour Typo3
 
-
-
 Pour voir les dernières versions disponibles:
 
-    composer outdated
+    php composer.phar outdated
 
 Pour mettre à jour typo3:
 
-    composer update
+    php composer.phar update
 
 Pour tout mettre à jour:
 
-    composer update
+    php composer.phar update
 
 
 ### Synchroniser les données depuis prod-front sur preprod

ot_connect/Classes/Service/OtAuthenticationService.php

@@ -109,7 +109,6 @@ class OtAuthenticationService extends AbstractAuthenticationService
      */
     public function getUser()
     {
-
         // Does the user already have a session on the Opentalent API?
         $username = $this->getAuthenticatedUsername();
 
@@ -293,7 +292,6 @@ class OtAuthenticationService extends AbstractAuthenticationService
      */
     protected function createOrUpdateUser(): bool
     {
-
         // Get user's data from the API
         $userApiData = $this->getUserData();
 
@@ -315,12 +313,36 @@ class OtAuthenticationService extends AbstractAuthenticationService
             'name' => $userApiData['name'],
             'first_name' => $userApiData['first_name'],
             'description' => '[Warning: auto-generated record, do not modify] FE User',
-            'usergroup' => self::GROUP_FE_ALL_UID,
             'deleted' => 0,
             'tx_opentalent_opentalentId' => $userApiData['id'],
             'tx_opentalent_generationDate' => date('Y/m/d H:i:s')
         ];
 
+        $groupsUid = [self::GROUP_FE_ALL_UID];
+        if ($userApiData['accesses']) {
+            foreach ($userApiData['accesses'] as $accessData) {
+                $organizationId = $accessData['organizationId'];
+
+                // get the fe_group for this organization
+                $groupUid = $connection->fetchOne(
+                    "select g.uid
+                     from typo3.fe_groups g
+                     inner join (select uid, ot_website_uid from typo3.pages where is_siteroot) p 
+                     on g.pid = p.uid
+                     inner join typo3.ot_websites w on p.ot_website_uid = w.uid
+                     where w.organization_id=:organizationId;",
+                    ['organizationId' => $organizationId]
+                );
+
+                if ($groupUid) {
+                    $groupsUid[] = $groupUid;
+                } else {
+                    $this->writeLogMessage("Warning: no fe_group found for organization " . $organizationId);
+                }
+            }
+        }
+        $fe_row['usergroup'] = join(',', $groupsUid);
+
         // TODO: log a warning if a user with the same opentalentId exists (the user might have changed of username)
         $q = $connection->select(
             ['uid', 'tx_opentalent_opentalentId'],

ot_connect/ext_localconf.php

@@ -20,8 +20,9 @@ defined('TYPO3_MODE') || die();
     ]
 );
 
-# Always call the getUser method if no session is active
-$GLOBALS['TYPO3_CONF_VARS']['SVCONF']['auth']['setup']['FE_fetchUserIfNoSession'] = true;
+# Always fetch FE user, since the previous one could have been disconnected from the Opentalent API since
+$GLOBALS['TYPO3_CONF_VARS']['SVCONF']['auth']['setup']['FE_alwaysFetchUser'] = true;
+$GLOBALS['TYPO3_CONF_VARS']['SVCONF']['auth']['setup']['FE_alwaysAuthUser'] = true;
 
 // Use popup window to refresh login instead of the AJAX relogin:
 $GLOBALS['TYPO3_CONF_VARS']['BE']['showRefreshLoginPopup'] = 1;

ot_core/Classes/Domain/Repository/EventRepository.php

@@ -159,8 +159,10 @@ class EventRepository extends BaseApiRepository
         $event = new Event();
         $a = explode('/', $record['@id']);
         $event->setId(end($a));
-        $event->setOrganizationId($record['organizationId']);
-        $event->setSubdomain($record['subDomain']);
+        if ($record['organizationId'] !== null) {
+            $event->setOrganizationId($record['organizationId']);
+        }
+        $event->setSubdomain($record['subDomain'] ?? '');
         $event->setName($record['name']);
         $event->setDescription($record['description']);
         $event->setUrl($record['url']);

ot_templating/Classes/ViewHelpers/Members/GetAllViewHelper.php

@@ -95,8 +95,10 @@ class GetAllViewHelper extends OtAbstractViewHelper {
                     return 1;
                 } else if ($a->getInstrument() != $b->getInstrument()) {
                     return strcmp($a->getInstrument(), $b->getInstrument());
-                } else {
+                } else if ($a->getName() != $b->getName()) {
                     return strcmp($a->getName(), $b->getName());
+                } else {
+                    return 0;
                 }
             }
         );

ot_templating/Resources/Private/Layouts/Classic/StructureDetails.html

@@ -21,6 +21,7 @@
                     referrerpolicy="strict-origin"
                     style="border: none;"
                     onload="iFrameResize()"
+                    allow="geolocation"
             ></iframe>
         </div>
     </div>

ot_templating/Resources/Private/Layouts/Classic/Structures.html

@@ -21,6 +21,7 @@
                     referrerpolicy="strict-origin"
                     style="border: none;"
                     onload="iFrameResize()"
+                    allow="geolocation"
             ></iframe>
         </div>
     </div>

ot_templating/Resources/Private/Layouts/Modern/StructureDetails.html

@@ -21,6 +21,7 @@
                         referrerpolicy="strict-origin"
                         style="border: none;"
                         onload="iFrameResize()"
+                        allow="geolocation"
                 ></iframe>
             </div>
         </section>

ot_templating/Resources/Private/Layouts/Modern/Structures.html

@@ -21,10 +21,11 @@
                         referrerpolicy="strict-origin"
                         style="border: none;"
                         onload="iFrameResize()"
+                        allow="geolocation"
                 ></iframe>
             </div>
         </section>
-
+<!---->
         <f:comment><!-- Render the footer defined in partial/footer.html--></f:comment>
         <f:render partial="Modern/Footer" />
     </div>

ot_templating/Resources/Public/assets/Classic/style/module/_forms.scss

@@ -82,6 +82,7 @@ span.submit {
 form button[type=submit] {
   min-width: 120px;
   background-color: $forms-btn-background-color;
+  color: #000000;
 }
 
 form button[type=submit]:hover {

ot_templating/Resources/Public/assets/Classic/style/module/_structures.scss

@@ -3,9 +3,9 @@
 .ot-structures {
   iframe {
     border: none;
-    width: 100%;
+    width: 1px;
+    min-width: 100%;
     max-width: 100%;
-    height: 2400px;
   }
   @media screen and (min-width: 1100px) {
     iframe {