From 830e2cd0459bd5f9910060b6143475a2b6593234 Mon Sep 17 00:00:00 2001
From: Guillaume Pellerin <guillaume.pellerin@parisson.com>
Date: Mon, 21 Jul 2025 13:39:33 +0200
Subject: [PATCH] update webclass display and admin fields management

---
 teleforma/webclass/admin.py                    |  4 ++--
 teleforma/webclass/models.py                   | 18 +++++++++---------
 .../templates/webclass/webclass_record.html    | 12 ++++++++----
 3 files changed, 19 insertions(+), 15 deletions(-)

diff --git a/teleforma/webclass/admin.py b/teleforma/webclass/admin.py
index a7b34e5e..d4614eb6 100644
--- a/teleforma/webclass/admin.py
+++ b/teleforma/webclass/admin.py
@@ -23,8 +23,8 @@ class WebclassAdmin(admin.ModelAdmin):
 
 class WebclassRecordAdmin(admin.ModelAdmin):
     list_filter = ('period', 'course', 'session', 'professor', 'category', 'created', 'status')
-    list_display = ('period', 'course', 'session', 'professor', 'category', 'created', 'status')
-    search_fields = ['id', 'course__code', 'course__title']
+    list_display = ('period', 'course', 'session', 'record_id', 'professor', 'category', 'created', 'status')
+    search_fields = ['id', 'course__code', 'course__title', 'record_id']
     actions = ['publish',]
 
     # def get_form(self, request, obj=None, **kwargs):
diff --git a/teleforma/webclass/models.py b/teleforma/webclass/models.py
index e80c20c2..bb9df45d 100644
--- a/teleforma/webclass/models.py
+++ b/teleforma/webclass/models.py
@@ -64,8 +64,9 @@ def get_records_from_bbb(**kwargs):
                 url = str(url)
             else:
                 continue
-            url = url.replace('bbb2.parisson.com', 'bbb3.parisson.com')
-            url = url.replace("http://", "https://")
+
+            url = url.replace('bbb2.parisson.com', 'bbb3.parisson.com') #HACK
+            url = url.replace("http://", "https://") #HACK
             start = int(str(recording['startTime'])[:-3])
             end = int(str(recording['endTime'])[:-3])
             preview = ""
@@ -150,6 +151,7 @@ class WebclassPublishedManager(models.Manager):
     def get_queryset(self):
         return super(WebclassPublishedManager, self).get_queryset().filter(status=3).exclude(end_date__lt=date.today())
 
+
 class Webclass(models.Model):
 
     department = models.ForeignKey('teleforma.Department', related_name='webclass', verbose_name=_(
@@ -220,7 +222,7 @@ class WebclassSlot(models.Model):
     participants = models.ManyToManyField(User, related_name="webclass_slot", verbose_name=_('participants'),
                                           blank=True)
     room_id = models.CharField(
-        'id de la conférence BBB (généré automatiquement)', blank=True, null=True, max_length=255)
+        'ID Room BBB (généré automatiquement)', blank=True, null=True, max_length=255)
     room_password = models.CharField(
         'password du modérateur (généré automatiquement)', blank=True, null=True, max_length=255)
 
@@ -430,22 +432,20 @@ class WebclassRecord(models.Model):
     period = models.ForeignKey('teleforma.Period', verbose_name=_('period'), on_delete=models.CASCADE)
     course = models.ForeignKey(
         'teleforma.Course', related_name='webclass_records', verbose_name=_('course'), on_delete=models.CASCADE)
-    record_id = models.CharField("Enregistrement BBB", max_length=255, blank=True, null=True)
+    record_id = models.CharField("ID Enregistrement BBB", max_length=255, blank=True, null=True)
+    # Used when conference are created from webclass by teachers to retrive the record id
+    room_id = models.CharField(
+        'ID Room BBB (généré automatiquement)', blank=True, null=True, max_length=255, unique=True)
     # not used for now, but may be handy if we need to optimize performance
     bbb_server = models.ForeignKey(
         'BBBServer', related_name='webclass_records', verbose_name='Serveur BBB', on_delete=models.CASCADE)
     created = models.DateTimeField("Date de la conférence", auto_now_add=True)
     session = models.CharField(
         _('session'), choices=session_choices, max_length=16, blank=True, null=True)
-    
-    # Used when conference are created from webclass by teachers to retrive the record id
-    room_id = models.CharField(
-        'id de la conférence BBB (généré automatiquement)', blank=True, null=True, max_length=255, unique=True)
     professor = models.ForeignKey('teleforma.Professor', related_name='webclass_records', verbose_name=_('professor'), on_delete=models.SET_NULL, blank=True, null=True)
     status = models.IntegerField(
         _('status'), choices=STATUS_CHOICES, default=2)
     
-
     WEBCLASS = 'WC'
     CORRECTION = 'CC'
     CATEGORY_CHOICES = [
diff --git a/teleforma/webclass/templates/webclass/webclass_record.html b/teleforma/webclass/templates/webclass/webclass_record.html
index 3d730d27..17221a70 100644
--- a/teleforma/webclass/templates/webclass/webclass_record.html
+++ b/teleforma/webclass/templates/webclass/webclass_record.html
@@ -17,14 +17,18 @@
         <div>
             <dl class="listing" style="font-size: 1.2em;">
                 {% if record.slot %}
-                <dt>{% trans "Professor" %}</dt>
-                <dd>{{ record.slot.professor }}</dd>
+                    <dt>{% trans "Professor" %}</dt>
+                    <dd>{{ record.slot.professor }}</dd>
                 {% endif %}
                 <dt>{% trans "Begin" %}</dt>
                 <dd>{{ record.start_date }}</dd>
                 {% if record.obj.session %}
-                <dt>{% trans "Session" %}</dt>
-                <dd>{{ record.obj.session }}</dd>
+                    <dt>{% trans "Session" %}</dt>
+                    <dd>{{ record.obj.session }}</dd>
+                {% endif %}
+                {% if user.is_staff or user.professor.count %}
+                    <dt>{% trans "ID" %}</dt>
+                    <dd>{{ record.obj.record_id }}</dd>
                 {% endif %}
             </dl>
         </div>
-- 
2.39.5