From 963bfa7cb1c6ee2eebb91c44ee74b0cd25152809 Mon Sep 17 00:00:00 2001 From: Fabien BOURGEOIS Date: Tue, 6 Mar 2018 11:47:29 +0100 Subject: [PATCH] [REF]GOLEM Activity Queue : huge refactoring around activity methods + fixes --- golem_activity_queue/__manifest__.py | 2 +- golem_activity_queue/models/golem_activity.py | 254 +++++------------- .../models/golem_activity_queue.py | 2 +- 3 files changed, 70 insertions(+), 188 deletions(-) diff --git a/golem_activity_queue/__manifest__.py b/golem_activity_queue/__manifest__.py index 4f0a077..ef5e9d3 100644 --- a/golem_activity_queue/__manifest__.py +++ b/golem_activity_queue/__manifest__.py @@ -20,7 +20,7 @@ 'name': 'GOLEM activity queues', 'summary': 'GOLEM activity queues', 'description': ''' GOLEM activity queue management ''', - 'version': '10.0.1.1.2', + 'version': '10.0.1.1.3', 'category': 'GOLEM', 'author': 'Youssef El Ouahby, Fabien Bourgeois', 'license': 'AGPL-3', diff --git a/golem_activity_queue/models/golem_activity.py b/golem_activity_queue/models/golem_activity.py index 8aa7e8e..d250932 100644 --- a/golem_activity_queue/models/golem_activity.py +++ b/golem_activity_queue/models/golem_activity.py @@ -19,239 +19,121 @@ """ GOLEM Activity adaptations """ from odoo import models, fields, api, _ -from odoo.exceptions import ValidationError class GolemActivity(models.Model): """ GOLEM Activity adaptations """ _inherit = 'golem.activity' - #ajout d'un champs O2M vers activity_id de golem.activity.queue activity_queue_ids = fields.One2many('golem.activity.queue', 'activity_id', 'Pending registration') - # un booleen pour determiner si une fille d'attente est autorisé queue_allowed = fields.Boolean(default=True, readonly=True) - # un booleen pour automatisé l'inscription sur une activité depuis la file d'attente auto_registration_from_queue = fields.Boolean(default=True, readonly=True) - #ajout d'un champs pour calculer le nombre d'inscription en file d'attente queue_activity_number = fields.Integer(compute="_compute_queue_activity_number", store=True, string='Pending registration number') - #changer le mode de basculement en cas de desinctiption + @api.multi def auto_registration_toggle(self): - """ switch registration from queueu mode """ + """ switch registration from queue mode """ for activity in self: activity.auto_registration_from_queue = not activity.auto_registration_from_queue - #supprimer les personnes en attente si une inscription directement est faite @api.multi def write(self, vals): """ Override method write to delete record from queue if they register in activity""" - super(GolemActivity, self).write(vals) - #recupérer les modification au niveau des registrations - registrations = vals.get('activity_registration_ids') - if registrations: - #parcourir les registrations - for registration in registrations: - #s'il une nouvelle registration est trouvé - if registration[0] == 0: - #recupérer les données de la registration - act_id = registration[2].get('activity_id') - mem_id = registration[2].get('member_id') - domain = [('activity_id', '=', act_id), - ('member_id', '=', mem_id)] - #chercher si le meme nomre est inscrit sur lattente du meme activité - queue = self.env['golem.activity.queue'].search(domain) - if queue: - #supprimer l'inscription sur la queue - self.activity_queue_ids = [(2, queue.id, 0)] - return True + res = super(GolemActivity, self).write(vals) + registration_vals = vals.get('activity_registration_ids') + if registration_vals: + for rval in registration_vals: + if rval[0] == 0: # creation case + act_id = rval[2].get('activity_id') + mem_id = rval[2].get('member_id') + if act_id and mem_id: + domain = [('activity_id', '=', act_id), + ('member_id', '=', mem_id)] + queue = self.env['golem.activity.queue'].search(domain) + if queue: + # remove registration + queue.unlink() + # self.activity_queue_ids = [(2, queue.id, False)] + self.automated_register_from_queue() + return res - #Ajouter/supprimer une file à l'activité et afficher popup pour traitement automatisé @api.multi def queue_allowed_toggle(self): """ Toggle queue_alowed boolean """ - for activity in self: - #si l'activité contient une file - if activity.queue_allowed: - # si la queue déja contient des elements à supprimer - if len(activity.activity_queue_ids) > 0: - #parcourir et supprimer les element de la queue - for queue in activity.activity_queue_ids: - activity.activity_queue_ids = [(2, queue.id, 0)] - #changer l'état de queue allowed et automated_registration en false - activity.queue_allowed = False - activity.auto_registration_from_queue = False + self.ensure_one() + activity = self[0] + if activity.queue_allowed: + if len(activity.activity_queue_ids) > 0: + activity.activity_queue_ids.unlink() + activity.write({'queue_allowed': False, + 'auto_registration_from_queue': False}) + else: + return { + 'name': _('Choose the activity to register in'), + 'type': 'ir.actions.act_window', + 'res_model': 'golem.activity.automated.queue.activate.wizard', + 'view_mode': 'form', + 'context': {'default_activity_id' : activity.id}, + 'target': 'new', + } - else: - #lancer popup pour choisir activité à s'inscrire - self.ensure_one() - activity_id = self[0] - - return { - 'name' : _('Choose the activity to register in'), - 'type' : 'ir.actions.act_window', - 'res_model' : 'golem.activity.automated.queue.activate.wizard', - 'view_mode': 'form', - 'context' : {'default_activity_id' : activity_id.id}, - 'target': 'new', - } - - - - #contraint sur nombre d'inscription : une desincription declanche une inscription - #depuis attente mode automatique @api.multi - @api.constrains('activity_registration_ids') - def _auto_registration_from_queue(self): + def register_from_queue(self): + """ Registers from queue """ + for activity in self: + queues = activity.activity_queue_ids + if activity.queue_activity_number < activity.places_remain: + queues = queues[0:activity.queue_activity_number] + else: + queues = queues[0:activity.places_remain] + for queue in queues: + values = {'activity_id' : queue.activity_id.id, + 'member_id' : queue.member_id.id} + self.env['golem.activity.registration'].create(values) + queue.unlink() + + @api.multi + def automated_register_from_queue(self): """automated registration from queue""" for record in self: - # 1 verifier si une place est disponible sur activité - #2 verifier si la file contient des element - #3 verifier si la file est activé - #4 verifier si linscription automatique depuis la file est activé - if (len(record.activity_registration_ids) < record.places and + if (record.places_remain and record.queue_allowed and record.queue_activity_number > 0 and - record.queue_allowed and record.auto_registration_from_queue): - #recupérer la liste en file d'attente - queues = record.activity_queue_ids - #trier la liste selon l'id : récupérer l'ancien element - queues_sorted = sorted(queues, key=lambda k: k['id']) - #suppose que le membre est enrigistré - membre_registred = True - #parcourir les element sur l'attente - for queue in queues_sorted: - #inverse l'etat du memebre - membre_registred = False - #recuperer la liste des registration - registrations = record.activity_registration_ids - #parcourir les registration afin de vérifier si le memebre sur l'attente - #déja inscrit - for registration in registrations: - #compare le membre sur l'attente au membre sur l'inscription - if queue.member_id == registration.member_id: - #si membre trouvé inscrit sur l'activité on le supprime de la queue - record.activity_queue_ids = [(2, queue.id, 0)] - #si membre trouvé on mentionne enregistré, on passe au - #registration suivante - membre_registred = True - break - #à la sortie de la boucle si le membre nest pas sur inscription faire une - if not membre_registred: - #valeures pour creer une inscritpion apartir de la file - values = { - 'activity_id' : queue.activity_id, - 'member_id' : queue.member_id - } - # creation d'inscription - record.activity_registration_ids = [(0, 0, values)] - #suppression de l'element de la file d'attente - record.activity_queue_ids = [(2, queue.id, 0)] - #sortir de la boucle parcourissante la queue puisque inscription faite - break + record.register_from_queue() - - - #calculer le nombre d'inscription sur la file d'attente - @api.multi @api.depends('activity_queue_ids') def _compute_queue_activity_number(self): """ compute number of queue registration for activity""" for activity in self: activity.queue_activity_number = len(activity.activity_queue_ids) - #fonction enregistrement du premier element de la liste d'ttente en inscription : mode manuel - @api.multi - def register_from_queue(self): - """ register member from queue""" - for record in self: - #recupérer la liste en file d'attente - queues = record.activity_queue_ids - #trier la liste selon l'id : récupérer l'ancien element - queues_sorted = sorted(queues, key=lambda k: k['id']) - #suppose que le membre est enrigistré - membre_registred = True - #parcourir les element sur l'attente - for queue in queues_sorted: - #inverse l'etat du memebre - membre_registred = False - #recuperer la liste des registration - registrations = record.activity_registration_ids - #parcourir les registration afin de vérifier si le memebre sur - #l'attente déja inscrit - for registration in registrations: - #compare le membre sur l'attente au membre sur l'inscription - if queue.member_id == registration.member_id: - #si membre trouvé inscrit sur l'activité on le supprime de la queue - record.activity_queue_ids = [(2, queue.id, 0)] - # on mentionne enregistré, on passe au registration suivante - membre_registred = True - #on sort de la boucle de registration et on passe à - #l'element suivant de l'attente - break - #à la sortie de la boucle si le membre nest pas sur inscription faire une - if not membre_registred: - #valeures pour creer une inscritpion apartir de la file - values = { - 'activity_id' : queue.activity_id, - 'member_id' : queue.member_id - } - # creation d'inscription - record.activity_registration_ids = [(0, 0, values)] - #suppression de l'element de la file d'attente - record.activity_queue_ids = [(2, queue.id, 0)] - #sortir de la boucle parcourissante la queue puisque inscription faite - break - #si member_registred est true donc soit membre déja inscrit ou aucun membre sur queue - if membre_registred: - message = _('there is no member to register for this activity' - ' from queue.') - raise ValidationError(message) - # 1 verifie si ajout insctiption donc nombre d'inscription depasse place donc proposer queue - #2 verifier si desincription donc place disponible pour queue(automatique ou manuel) @api.multi @api.onchange('activity_registration_ids') def _check_registration_number(self): - current_activity = self._origin - for record in self: - #warning au cas ou le nombre d'inscription depasse le nombre de place - if (len(record.activity_registration_ids) > record.places and - record.queue_allowed): + for activity in self: + places_remain = activity.places - activity.places_used + if places_remain == 0 and activity.queue_allowed: message = _('No remaining place for the activity : {}, please' - ' discard changes and register in the queue using' - ' the button bellow') + ' discard changes and register in the queue.') return { 'warning' : { 'title' : _('Warning'), - 'message': message.format(record.name), + 'message': message.format(activity.name), } } - elif (len(record.activity_registration_ids) < len(current_activity.activity_registration_ids) and - len(current_activity.activity_registration_ids) == record.places and - record.queue_activity_number > 0): - #si le mode d'inscription depuis attente est activé - if record.auto_registration_from_queue: - #lancer un warning informant que l'inscription automatique aura - #lieu apres sauvegarde + elif places_remain > 0 and activity.queue_activity_number > 0: + if activity.auto_registration_from_queue: warning_message = _('There is a free place for the activity' ' : {}, once you save it will be filled' - ' by the first membre from queue') - return { - 'warning' : { - 'title' : _('Warning'), - 'message': warning_message.format(record.name) - } - } - - #traitement manuel pour le passage de la file d'attente en inscription - # : button sur queue + ' by the first member from queue') else: warning_message = _('There is a free place for the activity' - ' : {}, you can fill it from the queue' - ' using the button on queue tab') - return { - 'warning' : { - 'title' : _('Warning'), - 'message': warning_message.format(record.name) - } + ' : {}, you may fill it manually from ' + 'the queue') + return { + 'warning' : { + 'title' : _('Warning'), + 'message': warning_message.format(activity.name) } + } diff --git a/golem_activity_queue/models/golem_activity_queue.py b/golem_activity_queue/models/golem_activity_queue.py index 3e78cc1..3d993e5 100644 --- a/golem_activity_queue/models/golem_activity_queue.py +++ b/golem_activity_queue/models/golem_activity_queue.py @@ -24,7 +24,7 @@ from odoo.exceptions import ValidationError class GolemActivityQueue(models.Model): """ GOLEM Activity Queue """ _name = 'golem.activity.queue' - _order = "sequence" + _order = 'sequence asc, id desc' _description = 'GOLEM Activity Queue' _sql_constraints = [('golem_activity_queue_uniq', 'UNIQUE (member_id, activity_id)', _('This member has already been registered for the queue.'))]