[IMP] make event data more dynamic

Author: emjay0921

spp_event_data/models/__init__.py

@@ -1,4 +1,6 @@
 # Part of OpenSPP. See LICENSE file for full copyright and licensing details.
 
 from . import event_data
+from . import event_mixin
+from . import ir_model
 from . import registrant

spp_event_data/models/event_data.py

@@ -85,7 +85,16 @@ def open_form(self):
                 "edit": False,
             }
             res_model = rec.model
-            view_id = self.env[res_model].get_view_id()
+
+            # Get view_id - check if model has get_view_id method or find it directly
+            view_id = False
+            if hasattr(self.env[res_model], "get_view_id"):
+                view_id = self.env[res_model].get_view_id()
+            else:
+                # Fallback for dynamic models without the mixin
+                view = self.env["ir.ui.view"].search([("model", "=", res_model), ("type", "=", "form")], limit=1)
+                view_id = view.id if view else False
+
             return {
                 "name": rec.name,
                 "view_mode": "form",

spp_event_data/models/event_mixin.py

@@ -0,0 +1,31 @@
+# Part of OpenSPP. See LICENSE file for full copyright and licensing details.
+
+from odoo import api, models
+
+
+class SPPEventMixin(models.AbstractModel):
+    """
+    Mixin for event type models.
+    Provides common functionality for all event types.
+    """
+
+    _name = "spp.event.mixin"
+    _description = "SPP Event Mixin"
+
+    @api.model
+    def _register_hook(self):
+        """
+        Mark this model as an event model automatically.
+        All models inheriting from this mixin will be registered as event types.
+        """
+        super()._register_hook()
+        ir_model = self.env["ir.model"].search([("model", "=", self._name)], limit=1)
+        if ir_model and not ir_model.is_event_model:
+            ir_model.sudo().write({"is_event_model": True})
+
+    def get_view_id(self):
+        """
+        Retrieve the form view ID for this event model.
+        This is used by the event wizard to open the appropriate form.
+        """
+        return self.env["ir.ui.view"].search([("model", "=", self._name), ("type", "=", "form")], limit=1).id

spp_event_data/models/ir_model.py

@@ -0,0 +1,13 @@
+# Part of OpenSPP. See LICENSE file for full copyright and licensing details.
+
+from odoo import fields, models
+
+
+class IrModel(models.Model):
+    _inherit = "ir.model"
+
+    is_event_model = fields.Boolean(
+        string="Is Event Model",
+        default=False,
+        help="Indicates if this model represents an event type for the event tracking system",
+    )

spp_event_data/wizard/create_event_wizard.py

@@ -1,17 +1,42 @@
 # Part of OpenG2P. See LICENSE file for full copyright and licensing details.
 
+import logging
 from datetime import date
 
-from odoo import _, fields, models
+from odoo import _, api, fields, models
+
+_logger = logging.getLogger(__name__)
 
 
 class SPPCreateEventWizard(models.TransientModel):
     _name = "spp.create.event.wizard"
     _description = "Create Event Wizard"
 
+    @api.model
+    def _get_event_data_model_selection(self):
+        """
+        Dynamically get event types from models marked as event models.
+        Returns a list of tuples (model_name, model_display_name).
+        """
+        # Start with default
+        selection = [("default", "None")]
+
+        # Query all models marked as event models
+        event_models = self.env["ir.model"].search([("is_event_model", "=", True)], order="name")
+
+        for event_model in event_models:
+            selection.append((event_model.model, event_model.name))
+            _logger.debug(
+                "Added event type to wizard: %s (%s)",
+                event_model.model,
+                event_model.name,
+            )
+
+        return selection
+
     event_data_model = fields.Selection(
-        [("default", "None")],
-        "Event Type",
+        selection="_get_event_data_model_selection",
+        string="Event Type",
         default="default",
     )
     partner_id = fields.Many2one("res.partner", domain=[("is_registrant", "=", True)])
@@ -50,6 +75,27 @@ def next_page(self):
                 for split_wizard in wizard_list:
                     wizard_model += "%s." % split_wizard
                 wizard_model += "wizard"
+
+                # Check if specific wizard exists
+                wizard_exists = wizard_model in self.env
+
+                if not wizard_exists:
+                    # Try to find a generic wizard (for dynamic models)
+                    generic_wizard = "%s.create.dynamic.event.wizard" % wizard_list[0]
+                    if generic_wizard in self.env:
+                        wizard_model = generic_wizard
+                        _logger.info(
+                            "Using generic wizard %s for model %s",
+                            wizard_model,
+                            model_name,
+                        )
+                    else:
+                        _logger.warning(
+                            "No wizard found for model %s, skipping wizard step",
+                            model_name,
+                        )
+                        return
+
                 view_id = rec.get_view_id(wizard_model)
 
                 # create the event data and pass it to event_data_model wizard

spp_event_demo/models/house_visit.py

@@ -6,6 +6,7 @@
 
 class OpenSPPHouseVisit(models.Model):
     _name = "spp.event.house.visit"
+    _inherit = "spp.event.mixin"
     _description = "House Visit"
 
     summary = fields.Char()
@@ -17,9 +18,3 @@ class OpenSPPHouseVisit(models.Model):
     number_of_cows = fields.Integer()
     no_food_stock = fields.Integer()
     disabled = fields.Boolean(default=False)
-
-    def get_view_id(self):
-        """
-        This retrieves the View ID of this model
-        """
-        return self.env["ir.ui.view"].search([("model", "=", self._name), ("type", "=", "form")], limit=1).id

spp_event_demo/models/phone_survey.py

@@ -6,13 +6,8 @@
 
 class OpenSPPPhoneSurvey(models.Model):
     _name = "spp.event.phone.survey"
+    _inherit = "spp.event.mixin"
     _description = "Phone Survey"
 
     summary = fields.Char()
     description = fields.Text()
-
-    def get_view_id(self):
-        """
-        This retrieves the View ID of this model
-        """
-        return self.env["ir.ui.view"].search([("model", "=", self._name), ("type", "=", "form")], limit=1).id

spp_event_demo/models/school_enrolment.py

@@ -6,14 +6,9 @@
 
 class OpenSPPSchoolEnrolmentRecord(models.Model):
     _name = "spp.event.schoolenrolment.record"
+    _inherit = "spp.event.mixin"
     _description = "School Enrolment Record"
 
     school_name = fields.Char()
     enrolment_type = fields.Char()
     date = fields.Date()
-
-    def get_view_id(self):
-        """
-        This retrieves the View ID of this model
-        """
-        return self.env["ir.ui.view"].search([("model", "=", self._name), ("type", "=", "form")], limit=1).id

spp_event_demo/wizard/__init__.py

@@ -1,4 +1,3 @@
-from . import create_event_wizard
 from . import create_event_house_visit_wizard
 from . import create_event_phone_survey_wizard
 from . import create_event_school_enrolment_wizard