2865-email-bdm-deletion
-
438 440 traditional: true, 439 441 contentType: "application/json; charset=utf-8", 440 442 success: function(data) { 441 parentName = data.member.parent_barcode_base + ' - ' + data.member.parent_name; 442 childName = data.member.barcode_base + ' - ' + data.member.name; 443 parentName = data.member.parent_barcode_base + ' - ' + data.member.parent_name -
Developer
En préambule à mon commentaire :
Variable globale = variable accessible dans tout le code. Par convention on les déclare en début de fichier.
Variable locale = variable accessible dans le contexte dans lequel elle est définie. Par exemple, une variable locale définie dans une fonction ne sera accessible que dans cette fonction.
Ici tu crée une variable non déclarée. En js, il faut toujours déclarer ses variables, je t'invite à lire cette page sur le sujet : https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Statements/var
En l'état ton code marche car une variable non déclarée en js est une variable globale.
Si tu les déclare ici, elles ne seront pas accessible dans l'autre fonction qui est appelée en dessous (
confirmDeletion) !Je te conseille donc de passer l'objet
data.memberen paramètre de la fonctionconfirmDeletionpour pouvoir accéder aux données dans cette fonction. -
-
-
438 440 traditional: true, 439 441 contentType: "application/json; charset=utf-8", 440 442 success: function(data) { 441 parentName = data.member.parent_barcode_base + ' - ' + data.member.parent_name; 442 childName = data.member.barcode_base + ' - ' + data.member.name; 443 parentName = data.member.parent_barcode_base + ' - ' + data.member.parent_name 444 parentEmail = data.member.email -
Developer
parentEmailetchildEmailreçoivent toutes les deuxdata.member.email!Il faut récupérer également l'email de l'attaché quand est appelé
/members/get_member_info/. -
-
-
Developer
- Je vois que plusieurs lignes n'ont pas de
;: pense à linter le code avant de commit ton code ! - J'ai toujours une ligne avec les mail à False et d'autres lignes qui font simplement planter l'appli, je te propose que je me penche dessus quand tu aura fini, j'ai détecté plusieurs soucis dans le code qui datent d'avant que tu te mette dessus. Ça résoudra peut-être ces problèmes.
- Je vois que plusieurs lignes n'ont pas de
-
-
556 556 member = member[0] 557 557 parent = None 558 558 if member['parent_id']: 559 parent = api.search_read('res.partner', [['id', '=', int(member['parent_id'][0])]], ['barcode_base'])[0] 560 member['parent_barcode_base'] = parent['barcode_base'] 559 parent_barcode_base = api.search_read('res.partner', [['id', '=', int(member['parent_id'][0])]], ['barcode_base'])[0] 560 member['parent_barcode_base'] = parent_barcode_base['barcode_base'] 561 parent_email = api.search_read('res.partner', [['id', '=', int(member['parent_id'][0])]], ['email'])[0] -
Developer
Alors, presque ! La méthode search_read prend en paramètres, dans l'ordre : 1) le modèle 2) les conditions 3) les champs à retourner
Vu qu'on parle du même modèle avec les mêmes conditions, peux donc faire un seul appel en recherchant
barcode_baseetemail -
-
-
-
-
Toggle commit list
-
-
merged
Toggle commit list
