Merge branch 'master' of github.com:chamilo/chamilo-lms

ywarnier · ywarnier · commit 1d5aab574bb0 · 2025-04-03T17:21:34.000+02:00
diff --git a/public/main/inc/lib/sessionmanager.lib.php b/public/main/inc/lib/sessionmanager.lib.php
@@ -211,8 +211,9 @@ public static function create_session(
                     return get_lang('Session title already exists');
                 }
             } else {
-                $rs = Database::query("SELECT 1 FROM $tbl_session WHERE title='".$name."'");
-                if (Database::num_rows($rs)) {
+                $sessionRepo = Database::getManager()->getRepository(Session::class);
+                $existingSession = $sessionRepo->findOneBy(['title' => $name]);
+                if ($existingSession !== null) {
                     return get_lang('Session title already exists');
                 }
                 $ready_to_create = true;

Original file line number	Diff line number	Diff line change
`@@ -211,8 +211,9 @@ public static function create_session(`
`211`	`211`	`return get_lang('Session title already exists');`
`212`	`212`	`}`
`213`	`213`	`} else {`
`214`		`- $rs = Database::query("SELECT 1 FROM $tbl_session WHERE title='".$name."'");`
`215`		`- if (Database::num_rows($rs)) {`
	`214`	`+ $sessionRepo = Database::getManager()->getRepository(Session::class);`
	`215`	`+ $existingSession = $sessionRepo->findOneBy(['title' => $name]);`
	`216`	`+ if ($existingSession !== null) {`
`216`	`217`	`return get_lang('Session title already exists');`
`217`	`218`	`}`
`218`	`219`	`$ready_to_create = true;`