[IMP] study_base: prevent opening of progress_status from a study

studies_base/data/studies_base_data.xml

@@ -172,7 +172,7 @@
         </record>
 
         <record id="study_questionnaire_subject_type_PRACTITIONER" model="study.questionnaire.subject.type">
-            <field name="name">Praticien</field>
+            <field name="name">Praticient</field>
             <field name="value">PRACTITIONER</field>
         </record>
 

studies_base/models/study_participant.py

@@ -34,17 +34,3 @@ class StudyParticipant(models.Model):
     updated = fields.Datetime("Date mise à jour")
 
     active = fields.Boolean("Actif", default=True)
-
-    @api.depends("write_date")
-    def _compute_updated(self):
-        for record in self:
-            ## XXXb0g : the following two lines are to be removed when all records will have been updated during migration
-            if record.updated:
-                continue
-            record.updated = record.write_date
-
-    @api.depends("create_date")
-    def _compute_created(self):
-        for record in self:
-            if not record.created:
-                record.created = record.create_date

studies_base/models/study_questionnaire.py

@@ -36,20 +36,17 @@ class StudyQuestionnaire(models.Model):
     created = fields.Datetime("Created", compute="_compute_created", readonly=True)
     date = fields.Datetime("Date", compute="_compute_updated", readonly=True)
 
-    @api.depends("write_date")
-    def _compute_updated(self):
-        for record in self:
-            ## XXXb0g : the following two lines are to be removed when all records will have been updated during migration
-            if record.updated:
-                continue
-            record.updated = record.write_date
-
     @api.depends("create_date")
     def _compute_created(self):
         for record in self:
             if not record.created:
                 record.created = record.create_date
 
+    @api.depends("write_date")
+    def _compute_updated(self):
+        for record in self:
+            record.date = record.write_date
+
     active = fields.Boolean("Actif", default=True)
 
     def copy(self, default=None):

studies_base/models/study_questionnaire_response.py

@@ -34,17 +34,3 @@ class StudyQuestionnaireResponse(models.Model):
     updated = fields.Datetime("Date mise à jour")
 
     active = fields.Boolean("Actif", default=True)
-
-    @api.depends("write_date")
-    def _compute_updated(self):
-        for record in self:
-            ## XXXb0g : the following two lines are to be removed when all records will have been updated during migration
-            if record.updated:
-                continue
-            record.updated = record.write_date
-
-    @api.depends("create_date")
-    def _compute_created(self):
-        for record in self:
-            if not record.created:
-                record.created = record.create_date

studies_base/models/study_study.py

@@ -19,7 +19,7 @@ class StudyStudy(models.Model):
         "study.progress.status",
         string="Avancement de l'étude",
         domain="[('study_id', '=', id)]",
-        compute="_compute_progress_status_id",
+        compute="_compute_progress_status_id"
     )
     progress_status = fields.One2many(
         "study.progress.status", "study_id", "All progress status"
@@ -95,30 +95,22 @@ class StudyStudy(models.Model):
     @api.depends("write_date")
     def _compute_updated(self):
         for record in self:
-            ## XXXb0g : the following two lines are to be removed when all records will have been updated during migration
-            if record.updated:
-                continue
             _logger.info(f"Record ID: {record.id}, write_date: {record.write_date}")
             record.updated = record.write_date
 
     @api.depends("progress_status")
     def _compute_progress_status_id(self):
         for record in self:
-            unfinished_progress_status = record.progress_status.filtered(
+            unfinished_progress_status = record.progress_status.filtered(lambda x: x.actual == True)
-                lambda x: x.actual == True
+            record.progress_status_id = unfinished_progress_status.id if len(unfinished_progress_status) == 1 else None
-            )
-            record.progress_status_id = (
-                unfinished_progress_status.id
-                if len(unfinished_progress_status) == 1
-                else None
-            )
 
     def copy(self, default=None):
         default = dict(default or {}, identifier_primary_id=None)
         return super().copy(default)
 
-    @api.depends("title", "name")
+
-    def name_get(self):
+@api.depends("title", "name")
+def name_get(self):
     result = []
     for study in self:
         if not study.name:

studies_base/views/study_participant_views.xml

@@ -57,8 +57,8 @@
                     </group>                    
                     <group name="technique" string="Technique">
                         <group name="technique_left">
-                            <field name="created" />
+                            <field name="create_date" />
-                            <field name="date" />
+                            <field name="write_date" />
                         </group>
                         <group name="technique_right">
                         </group>

studies_base/views/study_questionnaire_response_views.xml

@@ -61,8 +61,8 @@
                     </group>
                     <group name="technique" string="Technique">
                         <group name="technique_left">
-                            <field name="created" />
+                            <field name="create_date" />
-                            <field name="date" />
+                            <field name="write_date" />
                         </group>
                         <group name="technique_right">
                         </group>

studies_base/views/study_study_views.xml

@@ -46,7 +46,7 @@
                         <group name="study_status_left">
                             <field name="period_start" />
                             <field name="period_end" />
-                            <field name="progress_status_id" />
+                            <field name="progress_status_id" options="{'no_open': True}" />
                         </group>
                         <group name="study_status_right">
 
@@ -131,19 +131,11 @@
         <field name="model">study.study</field>
         <field name="arch" type="xml">
             <search string="Études">
-                <field name="active" string="Actif" />
-                <field name="description" string="Description" />
-                <field name="identifier_author" string="ID Plateforme" />
-                <field name="identifier_primary_id" string="Seintinelles ID"/>
-                <field name="keywords" string="Mots-clés" />
-                <field name="name" string="Acronyme" />
-                <field name="period_end" string="Date de fin" />
-                <field name="period_start" string="Date de début" />
-                <field name="phase" string="Phase" />
-                <field name="progress_status_id" string="Statut" />
-                <field name="region" string="Région" />
                 <field name="title" string="Nom de l'étude" />
-                <field name="version" string="Version" />
+                <field name="name" string="Acronyme" />
+                <field name="identifier_primary_id" string="Seintinelles ID"/>
+                <field name="period_start" string="Date de début" />
+                <field name="period_end" string="Date de fin" />
             </search>
         </field>
     </record>