User: Add ordering logic for courses and sessions list - refs #1655

src/CoreBundle/DataProvider/Extension/CourseRelUserExtension.php

@@ -60,22 +60,26 @@ final class CourseRelUserExtension implements QueryCollectionExtensionInterface
             }
         }
 
-        if ($this->security->isGranted('ROLE_ADMIN')) {
-            return;
-        }
-
         if (CourseRelUser::class === $resourceClass) {
             if ('collection_query' === $operation?->getName()) {
+                $rootAlias = $queryBuilder->getRootAliases()[0];
+                $queryBuilder->leftJoin("$rootAlias.course", 'c');
+                $queryBuilder
+                    ->orderBy('c.title', 'ASC')
+                    ->addOrderBy("$rootAlias.sort", 'ASC')
+                    ->addOrderBy("$rootAlias.userCourseCat", 'ASC');
+
+                if (!$this->security->isGranted('ROLE_ADMIN')) {
                     /** @var User|null $user */
                     if (null === $user = $this->security->getUser()) {
                         throw new AccessDeniedException('Access Denied.');
                     }
 
-                $rootAlias = $queryBuilder->getRootAliases()[0];
                     $queryBuilder->andWhere(\sprintf('%s.user = :current_user', $rootAlias));
                     $queryBuilder->setParameter('current_user', $user->getId());
                 }
             }
+        }
 
         $this->addWhere($queryBuilder, $resourceClass);
     }

src/CoreBundle/Repository/SessionRepository.php

@@ -78,7 +78,9 @@ class SessionRepository extends ServiceEntityRepository
                 'user' => $user,
                 'url' => $url,
             ])
-        ;
+            ->orderBy('s.displayStartDate', 'ASC')
+            ->addOrderBy('s.title', 'ASC')
+            ->addOrderBy('s.position', 'ASC');
 
         return $qb;
     }