Skills: Fix htaccess url for badge - refs BT#15374

.htaccess

@@ -53,12 +53,12 @@ RewriteRule ^session/(\d{1,})/about/?$ main/session/about.php?session_id=$1 [L]
 RewriteRule ^course/(\d{1,})/about/?$ main/course_info/about.php?course_id=$1 [L]
 
 # Issued individual badge friendly URL
-RewriteRule ^badge/(\d{1,}) main/badge/issued.php?issue=$1 [L]
+RewriteRule ^badge/(\d{1,})/?$ main/badge/issued.php?issue=$1 [L]
 
 # Issued badges friendly URL
-RewriteRule ^skill/(\d{1,})/user/(\d{1,}) main/badge/issued_all.php?skill=$1&user=$2 [L]
+RewriteRule ^skill/(\d{1,})/user/(\d{1,})/?$ main/badge/issued_all.php?skill=$1&user=$2 [L]
 # Support deprecated URL (avoid 404)
-RewriteRule ^badge/(\d{1,})/user/(\d{1,}) main/badge/issued_all.php?skill=$1&user=$2 [L]
+RewriteRule ^badge/(\d{1,})/user/(\d{1,})/?$ main/badge/issued_all.php?skill=$1&user=$2 [L]
 
 # Support old URLs using the exercice (with a c) folder rather than exercise
 RewriteRule ^main/exercice/(.*)$ main/exercise/$1 [QSA,L]

main/badge/issued.php

@@ -16,7 +16,6 @@ use SkillRelUser as SkillRelUserManager;
 require_once __DIR__.'/../inc/global.inc.php';
 
 $issue = isset($_REQUEST['issue']) ? (int) $_REQUEST['issue'] : 0;
-$userId = isset($_REQUEST['user']) ? (int) $_REQUEST['user'] : 0;
 
 if (empty($issue)) {
     api_not_allowed(true);