Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] easy_my_coop: use firstname and lastname #15

Open
wants to merge 1 commit into
base: 9.0
Choose a base branch
from

Conversation

robinkeunen
Copy link
Collaborator

@robinkeunen robinkeunen commented Dec 3, 2018

Cleaning up first/last names

Contexte

Avant ce commit:
beescoop/Obeesdoo@51e1c1e#diff-4d07185a6ca5e1a4c22f21e06740b293
dans la bees nous utilisions les champs first_name et last_name. Le module partner_firstname de l'oca définit lui par contre les champs firstname et lastname.

J'ai donc dans les modules bees changer pour ces deux champs, toujours dans l'idée d'être compatible. Sauf que j'ai remarqué (à l'époque) que easy_my_coop malgré qu'il dépend de partner_firstname utilisait les champs first_name et last_name... du coup dans le module que j'ai écris j'ai créer les deux champs l'un étant related de l'autre histoire qu'on puisse indifféremment, lire et écrire l'un ou l'autre.

Idéalement tout le code, qui utilise le module partner_firstname (OCA ou le mien) devrait uiliser les champs firstname et lastname. c'est déjà le cas pour les module de la beescoop.

- Mouloud

Effectivement, dans houssine78/partner_firstname, copie du module de l'OCA:

class ResPartner(models.Model):  
    """Adds last name and first name; name becomes a stored function field."""  
   _inherit = 'res.partner'  
  
   firstname = fields.Char("First name")  
    lastname = fields.Char("Last name")

Et dans obeesdoo/partner_fistname:

class Partner(models.Model):
    _inherit = 'res.partner'
    firstname = fields.Char('First Name')  
    lastname = fields.Char('Last Name', required=True, default="/")  
    ...
    #Compatibility with old name use in beedoo  
    last_name = fields.Char(related='lastname')  
    first_name = fields.Char(related='firstname')

Todo

  • Remplacer dans le code easymycoop les champs first_name et last_name par firstname et lastname.

@robinkeunen robinkeunen changed the title [FIX] usesy_my_coop: firstname and lastname [FIX] easy_my_coop: use firstname and lastname Dec 3, 2018
@remytms
Copy link
Collaborator

remytms commented May 31, 2019

@robinkeunen Est-ce que c'est toujours d'acualité ? Sinon, on peut la fermer et si oui, alors on va la porter vers le dépot coopiteasy. :)

@robinkeunen
Copy link
Collaborator Author

Oui, il faut harmoniser l'usage de firstname et lastname. :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants